문제설명
소수점 아래 숫자가 계속되지 않고 유한개인 소수를 유한소수라고 합니다. 분수를 소수로 고칠 때 유한소수로 나타낼 수 있는 분수인지 판별하려고 합니다. 유한소수가 되기 위한 분수의 조건은 다음과 같습니다.
두 정수 a와 b가 매개변수로 주어질 때, a/b가 유한소수이면 1을, 무한소수라면 2를 return하도록 solution 함수를 완성해주세요.
제한사항
입출력 예
a | b | result |
---|---|---|
7 | 20 | 1 |
11 | 22 | 1 |
12 | 21 | 2 |
입출력 예 설명
입출력 예 #1
입출력 예 #2
입출력 예 #3
Hint
풀이
풀이에 가장 많은 시간이 들어간 문제다.
초기 코드에서 테스트 케이스에서 계속 실패가 뜨고, 런타임 오류도 많았다.
분자가 분모보다 큰 경우 고려하지 않아 앞부분 코드를 다시 짰다.
최대 공약수를 구해 분자와 분모를 나누어 주고 a를 b로 나눈 나머지가 0일 경우
1을 리턴해주고, 해당부분을 처리했다.
그 다음은 분모가 더 큰 기약순수의 처리부분이다.
기약분수의 분모의 소인수를 구해서 list에 추가 해준뒤, 다시 list에서 2와 5를 제거해 주었다.
그 때 list의 size가 0이라면, 소인수는 2나 5, 또는 2와 5였다는 뜻 이므로,
해당 분수는 유한소수라고 판단할 수 있다(2 리턴).
처음에는 list의 합을 2,5,7일 경우로 조건 식을 잡았는데,
list요소가 [2,3]이거나 [7]일 경우는 무한 소수임에도 1일 리턴 하기 때문에 테스트 케이스에서
실패가 뜬다.
통과는 했으나 코드가 뭔가 답답해 보인다...코드를 손볼 필요가 있어보인다.
class Solution {
public int solution(int a, int b) {
List<Integer>list = new ArrayList<>();
//최대 공약수
int max = Math.max(a, b);
int GCD = 1;
for (int i = max; i >= 2; i--) {
if(a%i==0 && b%i==0) {
GCD=i;
break;
}
}
a = a/GCD;
b = b/GCD;
if(a%b==0) {
return 1;
}
//기약분수 처리, 소인수를 구해 list에 추가
for (int i = 2; i <= b; i++) {
if(b%i==0) {
int count=0;
for (int j = 2; j <= i; j++) {
if(i%j==0) {
count++;
}
}
if(count==1) {
list.add(i);
}
}
}
list.remove(Integer.valueOf(2));
list.remove(Integer.valueOf(5));
if(list.size()==0) {
return 1;
}
return 2;
}
}
참고