백준 반복문 - 2525번 문제 틀린 이유 + 리팩토링

= 문제 =

문제

KOI 전자에서는 건강에 좋고 맛있는 훈제오리구이 요리를 간편하게 만드는 인공지능 오븐을 개발하려고 한다. 인공지능 오븐을 사용하는 방법은 적당한 양의 오리 훈제 재료를 인공지능 오븐에 넣으면 된다. 그러면 인공지능 오븐은 오븐구이가 끝나는 시간을 분 단위로 자동적으로 계산한다.

또한, KOI 전자의 인공지능 오븐 앞면에는 사용자에게 훈제오리구이 요리가 끝나는 시각을 알려 주는 디지털 시계가 있다.

훈제오리구이를 시작하는 시각과 오븐구이를 하는 데 필요한 시간이 분단위로 주어졌을 때, 오븐구이가 끝나는 시각을 계산하는 프로그램을 작성하시오.

입력

첫째 줄에는 현재 시각이 나온다. 현재 시각은 시 A (0 ≤ A ≤ 23) 와 분 B (0 ≤ B ≤ 59)가 정수로 빈칸을 사이에 두고 순서대로 주어진다. 두 번째 줄에는 요리하는 데 필요한 시간 C (0 ≤ C ≤ 1,000)가 분 단위로 주어진다.

출력

첫째 줄에 종료되는 시각의 시와 분을 공백을 사이에 두고 출력한다. (단, 시는 0부터 23까지의 정수, 분은 0부터 59까지의 정수이다. 디지털 시계는 23시 59분에서 1분이 지나면 0시 0분이 된다.)

예제 입력 1

14 30

20

예제 출력 1

14 50

예제 입력 2

17 40

80

예제 출력 2

19 0

예제 입력 3

23 48

25

예제 출력 3

0 13

= 처음 내가 작성한 코드 = 

 

else if 안에 있는 if문을 보면

시간 값이 24가 넘었을 때, 즉 25부터 h를 -1로 다시 정의하고 현재 시각에 (분 + 타임)/60 한 값을 더해주는 과정이였다.

 

사실 이 땐 내가 왜 이렇게 코드를 짰는지 이해가 안 되는데 다른 사람들의 반례를 보고 그거에 맞게 수정하느라 이렇게 한 것 같다.

 

새로운 반례를 들었는데

#input

21 0

600

 

#output

9 0

 

#answer

7 0

 

21시에서 10시간이 추가되었을 때, 9시가 뜨는게 아니라 당연히 7시가 떠야되는게 맞는데

이 과정에서 이상함을 느꼈다.

 

21 0

240을 기준으로 계속 시험을 했었는데

 

240 = 4시간 따라서 h는 25가 되고 1이 출력이 되어야 한다.

근데 내가 짠 코드를 봤을 때, h > 24일 때  h = -1이 되지만 다시 ( m + time ) / 60값 => ( 0 + 240 ) / 60 = > 4

 

-1 +4를 하면 3이 나오는게 애초에 이상하다... 설계 자체가 틀렸다는 뜻!

 

그래서 코드를 좀 수정했다.

 

 

= 수정한 코드 =

 

import java.util.Scanner;

public class Main {
    public static void main(String[] args) {
        Scanner scanner = new Scanner(System.in);

        int h = scanner.nextInt();
        int m = scanner.nextInt();

        int time = scanner.nextInt();

        if (m + time == 60) {
            h = h + 1;
            m = 0;
        } else if (m + time > 100) {
            h = h + (m + time) / 60;
            if (h > 24) {
                h -= 24;
            }
            m = (m + time) % 60;
        } else if (m + time > 60 && m + time < 100) {
            h = h + 1;
            m = m + time - 60;
        } else if (m + time < 60) {
            m = m + time;
        }
        if (h == 24) {
            h = 0;
        }
        System.out.print(h + " " + m);
    }
}

= 접근 =

 

애초에 h가 24가 넘었을 때, 그 h값에서 24를 빼주면 다 해결될 것 같다고 판단했다.

 

25일 때 24를 빼면 내가 워ㅏㄴ하는 1값이고

 

26일 때 24를 빼면 내가 원하는 2값이 나오고 

 

이렇게 해서 해결할 수 있었다.

 

근데 코드가 너무 복잡하고 가독성도 안 좋고 나중에 수정할 일이 있을 때 너무 불편하기 때문에 리팩토링을 해봤다.

 

= 리팩토링 =

 

import java.util.Scanner;

public class Main {
    public static void main(String[] args) {
        Scanner scanner = new Scanner(System.in);

        int h = scanner.nextInt();
        int m = scanner.nextInt();

        int time = scanner.nextInt();

        int temp = (60 * h) + m + time; 
        int hour = temp / 60;
        if (hour >= 24) {
            hour -= 24;
        }
        int minute = temp % 60;
        System.out.println(hour + " " + minute);
    }
}

이렇게 간결하게 짤 수 있다.

 

내가 입력한 h ( 시간 )을 분으로 바꾸는 작업을 하고 h + m + time ( 시간 + 분 + 타이머 )에 분을 다 합친 값을

 

다시 hour로 바꿔주고 , minute로 바꿔주는 작업이다.

 

24시가 넘었을 때는 그냥 24를 빼주면 되는 아주 간단한 문제였다..

 

브론즈 3문제인데 혼자 해결하지 못했다. 아직 갈 길이 너무 멀었다.

더 열심히 하자..!