[프로그래머스] Level1 - 콜라츠 추측 (Python)

문제 설명

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을 완성해 주세요. 단, 작업을 500번을 반복해도 1이 되지 않는다면 –1을 반환해 주세요.

제한 사항

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

입출력	예
n	result
6	8
16	4
626331	-1

입출력 예 설명

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

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

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

내 문제 풀이

• answer 값(1이 될 때 까지 작업 수행 횟수)가 500일때 while 빠져나가는 조건으로 함
  (사실상 잘못 짠게 500번을 반복해도 1이 되지 않았을 경우 인데 내가 짠 코드는 500이 됐을 때 바로 빠져나가게 만듬. 즉 1이 되지 않았을 경우를 종료 조건으로 봐야하는것.)
• 홀수랑 짝수인 경우에 각각 작업 수행 후 1인지 check해서 1이면 반복문 빠져나가게
• answer 가 500인 경우엔 -1 return하도록 값 바꿔줌

def solution(num):
    answer = 0

    while (answer != 500):
        if num == 1:
            break
        else :
            if num % 2 == 0 :
                num /= 2
                answer += 1
                if num == 1:
                    break
            else :
                num = 3*num + 1
                answer += 1
                if num == 1:
                    break
    if answer == 500:
        answer = -1

    return answer

아쉬운 점

• while의 조건문 자체를 잘못 짠 것 (num != 1로 했어야 함)
• 반복되는 부분이 너무 많다. 이걸 줄일 수 있었을텐데