sallyjellyy / codesquad_cocoa Goto Github PK

안녕하세요 새리, 소니입니다.
3주차 미션 전반적으로 잘 구현하셨네요! 몇가지 아쉬운 점이 있어 리뷰 남기니 참고해주세요~

1. 프로그램을 실행하고 종료하는 로직이 없네요?! 입력값을 받아 종료하는 로직도 추가하면 좋을 것 같아요

2. 다음과 같이 checker.checker로 필드에 직접 접근하는건 좋지 않습니다. 언제든 필드 값이 변경될 수 있기 때문입니다. 클래스 내 필드는 private으로 설정해주세요~

if (checker.checker[i]) { // print in red if the letter's checked according to the time
                System.out.print(ANSI_RED + hangul[i] + " " + ANSI_RESET);
                continue;
            }

3. Checker 클래스에 switch 문이 길어져 메서드가 매우 길어지는데요, Enum에 대해 공부해보시고 코드의 양을 줄이고 로직을 개선할 수 있는지 고민해보시면 좋을 것 같아요~

안녕하세요 새리, 소니입니다.
2주차 미션에 대해 간단한 리뷰 남기니 참고해주세요~

1. 여러개의 클래스를 하나의 파일에 작성하지 말고, 파일로 분리해서 import하는 방식으로 사용해보세요
2. 접근제어자에 공부해보세요. 현재 클래스 내 필드가 모두 default 접근제어자로 되어있는데 어떻게 변경하면 좋을까요?
3. Scanner 객체는 Round 객체가 생성될 때마다 생성되는 구조인데, 한 번만 메모리에 할당에서 사용하는 방법도 찾아보면 좋을 것 같아요!
4. 출력 로직을 클래스로 분리해보는 건 어떨까요?
5. 메서드명에 get,set은 getter,setter에만 사용하는 걸 권장해요. 그리고 setter 메서드는 사용하지 않는 걸 권장합니다. 그 이유는 무엇인지 찾아보세요 👀
6. 메서드명은 메서드 이름만 봐도 어떤 역할을 하는지 알 수 있도록 작성하는게 중요해요. 메서드 이름이 모호하면 협업 할 때 동료가 코드를 이해하는데 어려움을 겪을 수 있거든요. 예를 들어, setBetting()이라는 메서드는 이름만 봐서는 betting 금액을 입력받아 설정한다고 유추하기 힘들어요. 그리고 인자로 아무값도 받지 않고 값을 set 한다는 것도 적절치 않네요. 자바의 메서드 네이밍에 대해 한 번 공부해보시면 좋을 것 같아요
7. 사용하지 않는 코드는 주석처리하지 말고 삭제해주세요~