출처: https://school.programmers.co.kr/learn/courses/30/lessons/42889
문제 설명
실패율
failture_rate1.png
슈퍼 게임 개발자 오렐리는 큰 고민에 빠졌다. 그녀가 만든 프랜즈 오천성이 대성공을 거뒀지만, 요즘 신규 사용자의 수가 급감한 것이다. 원인은 신규 사용자와 기존 사용자 사이에 스테이지 차이가 너무 큰 것이 문제였다.
이 문제를 어떻게 할까 고민 한 그녀는 동적으로 게임 시간을 늘려서 난이도를 조절하기로 했다. 역시 슈퍼 개발자라 대부분의 로직은 쉽게 구현했지만, 실패율을 구하는 부분에서 위기에 빠지고 말았다. 오렐리를 위해 실패율을 구하는 코드를 완성하라.
실패율은 다음과 같이 정의한다.
스테이지에 도달했으나 아직 클리어하지 못한 플레이어의 수 / 스테이지에 도달한 플레이어 수
전체 스테이지의 개수 N, 게임을 이용하는 사용자가 현재 멈춰있는 스테이지의 번호가 담긴 배열 stages가 매개변수로 주어질 때, 실패율이 높은 스테이지부터 내림차순으로 스테이지의 번호가 담겨있는 배열을 return 하도록 solution 함수를 완성하라.
제한사항
스테이지의 개수 N은 1 이상 500 이하의 자연수이다.
stages의 길이는 1 이상 200,000 이하이다.
stages에는 1 이상 N + 1 이하의 자연수가 담겨있다.
각 자연수는 사용자가 현재 도전 중인 스테이지의 번호를 나타낸다.
단, N + 1 은 마지막 스테이지(N 번째 스테이지) 까지 클리어 한 사용자를 나타낸다.
만약 실패율이 같은 스테이지가 있다면 작은 번호의 스테이지가 먼저 오도록 하면 된다.
스테이지에 도달한 유저가 없는 경우 해당 스테이지의 실패율은 0 으로 정의한다.
입출력 예
N stages result
5 [2, 1, 2, 6, 2, 4, 3, 3][3,4,2,1,5]
4 [4,4,4,4,4][4,1,2,3]
입출력 예 설명
입출력 예 #1
1번 스테이지에는 총 8명의 사용자가 도전했으며, 이 중 1명의 사용자가 아직 클리어하지 못했다. 따라서 1번 스테이지의 실패율은 다음과 같다.
1 번 스테이지 실패율 : 1/8
2번 스테이지에는 총 7명의 사용자가 도전했으며, 이 중 3명의 사용자가 아직 클리어하지 못했다. 따라서 2번 스테이지의 실패율은 다음과 같다.
2 번 스테이지 실패율 : 3/7
마찬가지로 나머지 스테이지의 실패율은 다음과 같다.
3 번 스테이지 실패율 : 2/4
4번 스테이지 실패율 : 1/2
5번 스테이지 실패율 : 0/1
각 스테이지의 번호를 실패율의 내림차순으로 정렬하면 다음과 같다.
[3,4,2,1,5]
입출력 예 #2
모든 사용자가 마지막 스테이지에 있으므로 4번 스테이지의 실패율은 1이며 나머지 스테이지의 실패율은 0이다.
[4,1,2,3]
내가 30분동안 작성한 코드문
import java.util.*;
class Solution {
public int[] solution(int N, int[] stages) {
int[] answer = new int[N];
// 실패율 = 스테이지에 도달했으나 아직 클리어하지 못한 플레이어의 수 / 스테이지에 도달한 플레이어 수
// 전체 스테이지의 개수 N
// stages = 게임을 이용하는 사용자가 현재 멈춰있는 스테이지의 번호
// 실패율이 높은 스테이지부터 내림차순으로 스테이지의 번호가 담겨있는 배열을 return
double failRate = 0.0; // 실패율 변수
double arr[] = new int[N];
for(int i = 0; i < stages.length; i++){
int count = 0;// i번 스테이지 총 도전자 수
for(int j = 1; j <= stages.length; j++){
// 일단 배열 인덱스 하나하나 다 돈다.
if(stages[i] > j){// 그다음에 i번 스테이지 사용자 수를 센다.
count++;
}
failRate = j / count;
arr[i] = failRate;
}
// arr 배열에 저장한 후 해당 인덱스 순위 별로 저장??
}
// map에 값과 인덱스 저장을 해본다면? 인덱스와 실패율 이렇게
HashMap<Integer, Integer> map = new HashMap<Integer, Integer>();
for(int i = 0; i < arr.length; i++){
map.put(i, arr[i]);
}
return answer;
}}
🔍 전체 코드에서의 문제점 요약
1. 실패율을 구하는 방식이 틀림
❌ 문제점:
stages[i] > j 조건은 stages[i]가 현재 어떤 플레이어의 스테이지인지를 기준으로, j가 1부터 전체 스테이지까지 비교되는데 논리적으로 잘못되었다.
failRate = j / count 는 int / int로 나눠서 항상 0 또는 1이 되며, j와 count의 관계도 잘못됐다.
✅ 수정 방향:
1부터 N까지 각 스테이지에 대해 실패율을 계산해야 한다.
해당 스테이지에 도달한 사람 수, 그 중 아직 클리어하지 못한 사람 수를 기준으로 해야 한다.
- 배열 인덱스와 실패율을 연결하는 로직이 없음
double arr[] = new int[N]; // 실수형인데 int로 선언함 (⚠️ 컴파일 에러)double[] arr = new double[N]; 으로 수정해야 함
스테이지 번호와 실패율을 따로 매핑해야 하기 때문에, 단순 배열보다 객체(List)를 쓰는 게 좋다.
- HashMap에 잘못된 타입 사용
HashMap<Integer, Integer> map = new HashMap<>();
for (int i = 0; i < arr.length; i++) {
map.put(i, arr[i]); // ⚠️ double을 int에 넣음 → 값 손실
}
❌ 문제점:
arr[i]는 double인데 int로 변환되면서 실패율의 소수점이 사라짐
또한 HashMap은 순서가 유지되지 않기 때문에 정렬 목적에 부적합
✅ 수정 방향:
Map<Integer, Double> 또는 List 같은 커스텀 클래스 사용 추천
-
결국 실패율을 기준으로 정렬하지 않음
arr이나 map을 정렬해서 answer 배열에 넣는 로직이 없음요구사항을 반영한 코드
import java.util.*;
class Solution {
public int[] solution(int N, int[] stages) {
int[] answer = new int[N];
int[] challengers = new int[N + 2]; // 각 스테이지에 도달한 사람 수 (1~N+1)
for (int stage : stages) {
challengers[stage]++;
}
List<Stage> stageList = new ArrayList<>();
int total = stages.length;
for (int i = 1; i <= N; i++) {
int fail = challengers[i]; // 스테이지 i에 머물러 있는 사람 수
double failRate = 0.0;
if (total != 0) {
failRate = (double) fail / total; // 실패율 계산
}
stageList.add(new Stage(i, failRate)); // 스테이지 번호와 실패율을 객체로 저장
total -= fail; // 다음 스테이지 도전자 수 업데이트
}
// 실패율 기준 내림차순, 만약 실패율이 같으면, 스테이지 번호 오름차순
stageList.sort((a, b) -> {
if (b.failRate == a.failRate) {
return a.stage - b.stage;
}
return Double.compare(b.failRate, a.failRate);
});
for (int i = 0; i < N; i++) {
answer[i] = stageList.get(i).stage;
}
return answer;
}
// 스테이지와 실패율을 함께 저장할 클래스
class Stage {
int stage;
double failRate;
Stage(int stage, double failRate) {
this.stage = stage;
this.failRate = failRate;
}
}
}
다른사람 풀이
import java.util.ArrayList;
import java.util.Collections;
import java.util.List;
class Solution {
public int[] solution(int N, int[] lastStages) {
int nPlayers = lastStages.length;
int[] nStagePlayers = new int[N + 2];
for (int stage : lastStages) {
nStagePlayers[stage] += 1;
}
int remainingPlayers = nPlayers;
List<Stage> stages = new ArrayList<>();
for (int id = 1 ; id <= N; id++) {
double failure = (double) nStagePlayers[id] / remainingPlayers;
remainingPlayers -= nStagePlayers[id];
Stage s = new Stage(id, failure);
stages.add(s);
}
Collections.sort(stages, Collections.reverseOrder());
int[] answer = new int[N];
for (int i = 0; i < N; i++) {
answer[i] = stages.get(i).id;
}
return answer;
}
class Stage implements Comparable<Stage> {
public int id;
public double failure;
public Stage(int id_, double failure_) {
id = id_;
failure = failure_;
}
@Override
public int compareTo(Stage o) {
if (failure < o.failure ) {
return -1;
}
if (failure > o.failure ) {
return 1;
}
return 0;
}
}
} 