[프로그래머스][코딩테스트 연습] 콜라츠 추측

문제 설명

1937년 Collatz란 사람에 의해 제기된 이 추측은, 주어진 수가 1이 될 때까지 다음 작업을 반복하면, 모든 수를 1로 만들 수 있다는 추측입니다. 작업은 다음과 같습니다.

1-1. 입력된 수가 짝수라면 2로 나눕니다. 
1-2. 입력된 수가 홀수라면 3을 곱하고 1을 더합니다. 
2. 결과로 나온 수에 같은 작업을 1이 될 때까지 반복합니다. 

예를 들어, 주어진 수가 6이라면 6 → 3 → 10 → 5 → 16 → 8 → 4 → 2 → 1 이 되어 총 8번 만에 1이 됩니다. 위 작업을 몇 번이나 반복해야 하는지 반환하는 함수, solution을 완성해 주세요. 단, 주어진 수가 1인 경우에는 0을, 작업을 500번 반복할 때까지 1이 되지 않는다면 –1을 반환해 주세요.

제한 사항

• 입력된 수, num은 1 이상 8,000,000 미만인 정수입니다.

입출력 예nresult

6	8
16	4
626331	-1

입출력 예 설명

입출력 예 #1
문제의 설명과 같습니다.

입출력 예 #2
16 → 8 → 4 → 2 → 1 이 되어 총 4번 만에 1이 됩니다.

입출력 예 #3
626331은 500번을 시도해도 1이 되지 못하므로 -1을 리턴해야 합니다.

 

 

 

내 잘못된 풀이

public class Main {
	public static int solution(int num) {
        int answer = 0;
        int cnt = 0;
        
        if(num == 1) {
        	return 0;
        }
        while(num != 1) {
        	if(num % 2 == 0) {
        		num = num / 2;
        		cnt++;
        		break;
        	}else {
        		num = num * 3 + 1;
        		cnt++;
        		break;
        	}
        }
        answer = cnt;
        
        if(cnt >= 500) {
        	return -1;
        }
        return answer;
    }
	
    public static void main(String[] args) {
    	System.out.println(solution(16));
    }
}
// 1

- cnt를 if 문 안에 굳이 넣어줄 필요가 없다...while 한번 반복할 때마다 넣어주면 된다..

- if 문마다 break를 할 필요도 없다.... break를 왜 쓰는지 ,, 다시 공부하자..후우 ㅠㅠ

- if 문은 while문 안에 다 때려박자..

 

 

정답 풀이

public class Main {
	public static int solution(int n) {
	long num = n; // 오버플로우를 방지한 형변환
        int cnt = 0;
        
        while(num != 1) {
        	if(num % 2 == 0){
        		num /= 2;
        	}else {
        		num = num * 3 + 1;
        	}
        	cnt++;
        	
        	if(cnt > 500) {
        		return -1;
        	}
        }
        return cnt;
    }
	
    public static void main(String[] args) {
    	System.out.println(solution(626331));
    }
}

매개변수로 입력 받은 int형 n값을 long으로 바꿔주지 않으면, 

예제 3번에서 488이라는 결과가 나온다.

입력받은 n값을 int형 그대로 사용하면 100번째 근처에서 오버플로우가 나와서 ,

음수로 한번 바뀌기 때문에 결과 값이 꼬인다고 한다....!!!