[작년 문제 풀어보기] 5기 1주차 문제 - 1~4번

별의개발자커비·2023년 8월 30일

우테코 도전기

목록 보기

3/37

개요

우테코 6기를 준비하며 5기 프리코스 문제를 하루에 하나씩 풀어보려고 한다.
기능구현 계획, 클린코드, 객체 지향을 생각하며 코드를 짜보자!

생각할 점

자바 코드 컨벤션 지키기
- https://velog.io/@dlguswl936/우테코-6기-도전기-클린코드알아보기
클린 코드 체크 리스트 읽기
- 한 메서드에 오직 한 단계의 들여쓰기(indent)만 허용했는가?
- else 예약어를 쓰지 않는다
의식적인 연습으로 TDD, 리팩토링 연습하기
기능 단위로 메소드를 분리하기
기능 단위로 커밋을 보내기
정규표현식 사용해보기
문제를 해결할 때 필요한 중요 상수를 private static final로 저장하기

[사진 출처]

1번 문제

요구사항

과정

클린 코드나 메소드 명칭을 신경쓰다보니 다른 예시글을 거의 따라친 것 같다..

내가 생각치 못한 예외!
- List에 들어있는 원소의 개수는 2개인가?
- 왼쪽 페이지는 홀수인가?
- 오른쪽 페이지는 짝수인가?

1차 코드

package week1;

public class Problem1 {

    public static int find_max_between_plus_and_multi(int num) {
        int units = 0;
        int plus_sum = 0;
        int multi_sum = 1;
        while (num != 0) {
            units = num % 10;
            plus_sum += units;
            multi_sum *= units;
            num = num / 10;
        }
        return Math.max(plus_sum, multi_sum);
    }

    public static int find_max_between_left_and_right(int[] pages) {
        int left_sum = find_max_between_plus_and_multi(pages[0]);
        int right_sum = find_max_between_plus_and_multi(pages[1]);
        return Math.max(left_sum, right_sum);
    }

    public static boolean is_invalid_page(int[] pages) {
        int left_page = pages[0];
        int right_page = pages[1];
        if (pages.length != 2) {
            return true;
        }
        if (left_page % 2 != 1) {
            return true;
        }
        if (right_page > 400) {
            return true;
        }
        return false;
    }


    public static int find_who_win(int pobi_num, int crong_num) {
        if (pobi_num < crong_num) {
            return 1;
        }
        if (pobi_num > crong_num) {
            return 2;
        }
        return 0;
    }

    public static int solution(int[] pobi, int[] crong) {
        int result = 0;
        if (is_invalid_page(pobi) || is_invalid_page(crong)) {
            return -1;
        }
        try {
            int pobi_max = find_max_between_left_and_right(pobi);
            int crong_max = find_max_between_left_and_right(crong);
            result = find_who_win(pobi_max, crong_max);
        } catch (Exception e) {
            return -1;
        }
        return result;
    }

    public static void main(String[] args) {
        System.out.println(solution(new int[]{11, 12}, new int[]{13, 14}));
    }
}

보완할 점

내가 혼자 쳐보기!
예외 다양하게 생각해보기
% 10 에 있는 10같은 숫자도 상수화를 하면 좋을 것 같은데 어떻게 하면 될까? 아래 사진 참고하기!
검증 로직은 모든 문제에 있을테니 아예 검증 폴더를 만들어서 클래스를 만들어 넣을 수도 있겠구나
참고코드 1
테스트 폴더를 빼서 테스트 코드를 만들어놓는 방법 해보기!
참고코드 2
이런식의 solution 코드와 변수명도 재밌다!
상수화를 enum으로 하는 것, 클래스를 아예 분리하는 것 따라해보기
십진수를 digit number 라고 하는구나!
초기화를 이렇게
? 궁금한점

여기서 resultStatus = referee.playGame(pobi, crong) 하지 않고
regeree 객체 생성 때 넣어주는게 더 나은 건가?
애초에 palyGame도 ()와 (num, num) 이렇게 두개를 만들어서 ()가 뒤를 호출하게 되어있는 것도 신기
exception을 여러개 두는 valid 클래스인데 그 안에서도

이렇게 boolean을 반환하는 is~ 메소드와 그 결과인 pass와 exception을 던지는 void가 따로있네
test 코드 작성해보기
- assertThat은 아는데
- assertThatThrownBy는, is Instance Of, hasMessage 이건 뭐지 해보기

테스트 할 때 List = new arrayList 하지 않고 어차피 데이터를 수정할 거 아니니까 List.of를 쓰면 되네!
https://kim-jong-hyun.tistory.com/31
이런 어노테이션을 테스트에 쓰면 한번만 입력할 수 있다

https://velog.io/@ak2j38/Junit-BeforeEach-BeforeAll-AfterEach-AfterAll를-알아보자
아 근데 public , private 이 헷갈린다. 메인 로직이 아니라 안에서 호출하는 거면 웬만하면 private을 써야겠지? 생성자는 public이어야 할거고