[프로그래머스] level1. 실패율 ( javascript)

문제

슈퍼 게임 개발자 오렐리는 큰 고민에 빠졌다. 그녀가 만든 프랜즈 오천성이 대성공을 거뒀지만, 요즘 신규 사용자의 수가 급감한 것이다. 원인은 신규 사용자와 기존 사용자 사이에 스테이지 차이가 너무 큰 것이 문제였다.

이 문제를 어떻게 할까 고민 한 그녀는 동적으로 게임 시간을 늘려서 난이도를 조절하기로 했다. 역시 슈퍼 개발자라 대부분의 로직은 쉽게 구현했지만, 실패율을 구하는 부분에서 위기에 빠지고 말았다. 오렐리를 위해 실패율을 구하는 코드를 완성하라.

실패율은 다음과 같이 정의한다.
스테이지에 도달했으나 아직 클리어하지 못한 플레이어의 수 / 스테이지에 도달한 플레이어 수
전체 스테이지의 개수 N, 게임을 이용하는 사용자가 현재 멈춰있는 스테이지의 번호가 담긴 배열 stages가 매개변수로 주어질 때, 실패율이 높은 스테이지부터 내림차순으로 스테이지의 번호가 담겨있는 배열을 return 하도록 solution 함수를 완성하라.

나의 풀이

function solution(N, stages) {
    var answer = [];
    const arr = [];
    for (let i =0 ; i< N+1;i++){
        arr.push(0)
    }
    // console.log(arr)

    for (let i =1 ; i< N+2;i++){
        for(let j =0; j< stages.length;j++){
            if(stages[j] ==i){
            arr[i-1] += 1
            }
        }
   }
    // console.log("stagecount : ",arr)

    const arr2 =[];
    for (let i =0 ; i< arr.length-1;i++){
        let num = arr[i];
        let num1 = 0;
        for (let j =i ; j< arr.length;j++){
            num1 += arr[j]
        }
        // console.log("cnt",num1);
        let fail = (num1 ==0 )? 0 :(num) / (num1)
        arr2.push(fail)

    }
    // console.log(arr2)

    for(let i =0;i < arr2.length;i++){
        let index = arr2.indexOf(Math.max(...arr2))
        answer.push(index+1)
        // console.log(index)
        arr2[index] = -1
    }

    // console.log(answer)
    return answer;
}

다른 사람 풀이

function solution(N, stages) {
    let ans = []

    for (let i = 1; i <= N; ++i) {
        let usersReachedCurrentStage   = stages.reduce((acc, curStage) => acc + ((curStage >= i) ? 1 : 0), 0)
        let usersStagnatedCurrentStage = stages.reduce((acc, curStage) => acc + ((curStage == i) ? 1 : 0), 0)
        if (usersReachedCurrentStage === 0) {
            ans.push({ 'stage': i, 'failRate': 0 })
            continue
        }

        ans.push({ 'stage': i, 'failRate': (usersStagnatedCurrentStage / usersReachedCurrentStage) })
    }

    return ans.sort((a, b) => {
        if (a.failRate > b.failRate) return -1
        if (a.failRate < b.failRate) return 1
        return a.stage - b.stage
    }).map(entry => entry.stage)
}

 

-------------

 

for (let i =0 ; i< arr.length-1;i++){
    let num = arr[i];
    let num1 = 0;
    for (let j =i ; j< arr.length;j++){
        num1 += arr[j]
    }

이렇게 for문을 2번 돌려서 카운팅 해도 좋지만,

 

let usersStagnatedCurrentStage = stages.reduce((acc, curStage) => acc + ((curStage == i) ? 1 : 0), 0)

 

reduce로 구해보자.

너무 for문에 의지하는 경향이 있어서 조금 다른 함수도 쓰는 버릇을 들여보자.

 

그리고 객체로 배열에 넣고 활용하는게 정말 좋았다.