Conversation
| view.winner(cars); | ||
| } | ||
|
|
||
| public static void nameException(String carName) { |
There was a problem hiding this comment.
메소드는 보통 동사로 작성하는 것이 관례입니다
해당 메소드는 자동차 이름이 유효한지 검증하는 메소드니까,
validCarName 이런 식으로 이름짓는 것이 적절해 보입니다
| @@ -0,0 +1,28 @@ | |||
| 자동차 경주 | |||
There was a problem hiding this comment.
감사합니다! 다음에는 클래스들의 역할과 구조를 추가해서 적어보겠습니다.
| 우승자 안내 문구 출력 기능 | ||
| - 우승자 안내, 쉼표로 우승자 나열 | ||
|
|
||
| `예외처리` |
There was a problem hiding this comment.
예외처리도 프로그램 설계할 때 정말 중요한 요소입니다
우선 작성해주신 코드를 봤을 때는 깔끔하게 예외처리 잘 해주신 것 같습니다!
추가로 다른 예외 상황(시도횟수에 소숫점을 입력했을 때, 시도횟수에 너무 큰 수를 입력했을 때 등)들도 있을지 더 고민해보시면 좋을 것 같습니다
There was a problem hiding this comment.
예외처리를 test파일에 있는 것만 생각했었는데 더 완벽한 프로그램을 위해서는 추가해야할 것 같습니다. 조언 감사합니다.
| int counts = tryCountNumCheckException(racingCount); // 예외처리 | ||
| tryCountException(counts); // 예외처리 | ||
|
|
||
| System.out.println(); |
There was a problem hiding this comment.
개행과 관련해서 팁을 드리자면,
println 보단 print('\n'); 으로 하는 것이 성능 상 이득을 볼 수 있습니다
그 이유도 함께 찾아보시면 좋을 것 같아요
src/main/java/racingcar/Racing.java
Outdated
|
|
||
| public void goOrStrop(Car car) { | ||
| int randomNum = Randoms.pickNumberInRange(0, 9); | ||
| if (randomNum >= 4) { |
There was a problem hiding this comment.
4와 같이 무언가 의미가 있는 숫자들은 상수로 처리하는 것을 권장합니다!
private static final int CAR_MOVE_STANDARD = 4;
public void goOrStop(Car car) {
int randomNum = randoms.picknumberInRange(0, 9(;
if (randomNum >= CAR_MOVE_STANDARD) ...
There was a problem hiding this comment.
조언 감사합니다. final로 초기화 이 후 재할당 할 수 없는 상수를 선언할 수 있는 것을 알게 되었습니다. 관례로는 언더바와 스네이크 표기법을 사용하는 것 또한 확인하였습니다. 변경하겠습니다!
|
레이싱 카의 코드를 객체지향적으로 리팩토링하였습니다. |
2 participants
test 코드가 정상 작동하지 않으나 일단 해결한 만큼이라도 제출합니다.
생각보다 예외처리나 신경써야할 것이 많았고, 커밋과 함수를 더 잘게 쪼개지 못한 것 같아 아쉽습니다.
객체지향적으로 문제를 해결하지 못한 게 아쉬운 거 같아 더 공부하고 고쳐야할 것 같습니다.