출처: https://school.programmers.co.kr/learn/courses/30/lessons/136798
숫자나라 기사단의 각 기사에게는 1번부터 number까지 번호가 지정되어 있습니다. 기사들은 무기점에서 무기를 구매하려고 합니다.
각 기사는 자신의 기사 번호의 약수 개수에 해당하는 공격력을 가진 무기를 구매하려 합니다. 단, 이웃나라와의 협약에 의해 공격력의 제한수치를 정하고, 제한수치보다 큰 공격력을 가진 무기를 구매해야 하는 기사는 협약기관에서 정한 공격력을 가지는 무기를 구매해야 합니다.
예를 들어, 15번으로 지정된 기사단원은 15의 약수가 1, 3, 5, 15로 4개 이므로, 공격력이 4인 무기를 구매합니다. 만약, 이웃나라와의 협약으로 정해진 공격력의 제한수치가 3이고 제한수치를 초과한 기사가 사용할 무기의 공격력이 2라면, 15번으로 지정된 기사단원은 무기점에서 공격력이 2인 무기를 구매합니다. 무기를 만들 때, 무기의 공격력 1당 1kg의 철이 필요합니다. 그래서 무기점에서 무기를 모두 만들기 위해 필요한 철의 무게를 미리 계산하려 합니다.
기사단원의 수를 나타내는 정수 number와 이웃나라와 협약으로 정해진 공격력의 제한수치를 나타내는 정수 limit와 제한수치를 초과한 기사가 사용할 무기의 공격력을 나타내는 정수 power가 주어졌을 때, 무기점의 주인이 무기를 모두 만들기 위해 필요한 철의 무게를 return 하는 solution 함수를 완성하시오.
제한사항
1 ≤ number ≤ 100,000
2 ≤ limit ≤ 100
1 ≤ power ≤ limit
입출력 예
number limit power result
5 3 2 10
10 3 2 21
입출력 예 설명
입출력 예 #1
1부터 5까지의 약수의 개수는 순서대로 [1, 2, 2, 3, 2]개입니다. 모두 공격력 제한 수치인 3을 넘지 않기 때문에 필요한 철의 무게는 해당 수들의 합인 10이 됩니다. 따라서 10을 return 합니다.
입출력 예 #2
1부터 10까지의 약수의 개수는 순서대로 [1, 2, 2, 3, 2, 4, 2, 4, 3, 4]개입니다. 공격력의 제한수치가 3이기 때문에, 6, 8, 10번 기사는 공격력이 2인 무기를 구매합니다. 따라서 해당 수들의 합인 21을 return 합니다.
class Solution {
public int solution(int number, int limit, int power) {
int answer = 0;
// 1번부터 number까지
// 15번으로 지정된 기사단원은 15의 약수가 1, 3, 5, 15로 4개 이므로, 공격력이 4인 무기를 구매합니다.
// 제한수치가 3이고 제한수치를 초과한 기사가 사용할 무기의 공격력이 2라면, 15번으로 지정된 기사단원은 무기점에서 공격력이 2인 무기를 구매합니다.
// 무기를 만들 때, 무기의 공격력 1당 1kg의 철이 필요합니다.
for(int i = 1; i <= number; i++){// number 까지 반복문
int count = 0;
// 각 원소의 약수 갯수구하기
// number 까지 반복문
for (int j = 1; j * j <= number; j++) {
if (j * j == number) count++;
else if (number % j == 0) count += 2;
// limit 넘지 않는 철의 무게 합하기
if(limit >= count) {
answer += count;// 안 넘을시 더한다.
} else { // 넘을 경우 제한수치 보다 작은 무기 구매 한다.
answer += limit-1;
}
}
}
return answer;
}}
내가 30분 정도 고민해서 작성한 코드문이다.
에러가 뜨지는 않지만,
테스트 1
입력값 〉 5, 3, 2
기댓값 〉 10
실행 결과 〉 실행한 결괏값 20이 기댓값 10과 다릅니다.
테스트 2
입력값 〉 10, 3, 2
기댓값 〉 21
실행 결과 〉 실행한 결괏값 60이 기댓값 21과 다릅니다.
이렇게 결과가 나온다.
도대체 뭐가 문제일까?
이 부분에서 j * j <= number와 number % j == 0 을 하고 있는데, number는 반복 대상 전체 기사단원의 수인데, 이 반복문은 각 i마다 돌아야 한다.
즉, number가 아니라 i를 기준으로 약수를 구해야 하는데, 지금은 모든 i에 대해 똑같이 number의 약수를 구하고 있다.
안쪽 반복문을 수정했다.
for (int j = 1; j j <= i; j++) {
if (j * j == i) count++;
else if (i % j == 0) count += 2;
// limit 넘지 않는 철의 무게 합하기
if(limit >= count) {
answer += count;// 안 넘을시 더한다.
} else { // 넘을 경우 제한수치 보다 작은 무기 구매 한다.
answer += limit-1;
}
}그리고 반복문 내 조건문도 요구사항을 만족하지 않는다.
count가 limit을 넘을시에 limit보다 작은 숫자를 더해라가 아닌
power 변수를 합해야 한다.
그리고 조건문 위치가 약수 계산 중간에 들어가 있음.
이렇게 하면 약수의 개수가 완전히 계산되기도 전에 answer += ... 이 실행돼서 중복 누적이 되는 것이다.
최종적으로 수정한 코드문..
class Solution {
public int solution(int number, int limit, int power) {
int answer = 0;
// 1번부터 number까지
// 15번으로 지정된 기사단원은 15의 약수가 1, 3, 5, 15로 4개 이므로, 공격력이 4인 무기를 구매합니다.
// 제한수치가 3이고 제한수치를 초과한 기사가 사용할 무기의 공격력이 2라면, 15번으로 지정된 기사단원은 무기점에서 공격력이 2인 무기를 구매합니다.
// 무기를 만들 때, 무기의 공격력 1당 1kg의 철이 필요합니다.
for(int i = 1; i <= number; i++){// number 까지 반복문
int count = 0;
// 각 원소의 약수 갯수구하기
// number 까지 반복문
for (int j = 1; j * j <= i; j++) {
if (j * j == i) count++;
else if (i % j == 0) count += 2;
}
// limit 넘지 않는 철의 무게 합하기
if(limit >= count) {
answer += count;// 안 넘을시 더한다.
} else { // 넘을 경우 제한수치 보다 작은 무기 구매 한다.
answer += power;
}
}
return answer;
}}
