프로그래머스/JAVA/유한소수 판별하기

문제설명

소수점 아래 숫자가 계속되지 않고 유한개인 소수를 유한소수라고 합니다. 분수를 소수로 고칠 때 유한소수로 나타낼 수 있는 분수인지 판별하려고 합니다. 유한소수가 되기 위한 분수의 조건은 다음과 같습니다.

• 기약분수로 나타내었을 때, 분모의 소인수가 2와 5만 존재해야 합니다.

두 정수 a와 b가 매개변수로 주어질 때, a/b가 유한소수이면 1을, 무한소수라면 2를 return하도록 solution 함수를 완성해주세요.

제한사항

• a, b는 정수
• 0 < a ≤ 1,000
• 0 < b ≤ 1,000

입출력 예

a	b	result
7	20	1
11	22	1
12	21	2

입출력 예 설명

입출력 예 #1

• 분수 7/20은 기약분수 입니다. 분모 20의 소인수가 2, 5 이기 때문에 유한소수입니다. 따라서 1을 return합니다.

입출력 예 #2

• 분수 11/22는 기약분수로 나타내면 1/2 입니다. 분모 2는 소인수가 2 뿐이기 때문에 유한소수 입니다. 따라서 1을 return합니다.

입출력 예 #3

• 분수 12/21는 기약분수로 나타내면 4/7 입니다. 분모 7은 소인수가 7 이므로 무한소수입니다. 따라서 2를 return합니다.

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;
    }
}

참고

• 유한소수 판별하기