4단계 - 자동차 경주(우승자) #6057
Open
4단계 - 자동차 경주(우승자) #6057
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
pci2676
requested changes
Mar 20, 2025
| System.out.print(getCurrentPositionDash()); | ||
| System.out.print("\n"); | ||
| } | ||
|
|
||
| public String getCurrentPositionDash() { |
There was a problem hiding this comment.
이 부분도 view와 강하게 관계가 있는 코드로 보이네요!
여러가지 view에서 서로다른 표현방식을 지원해야할때 이러한 메서드가 유용할까요? 🤔
Comment on lines
+12
to
+16
| if (name.length() > 5) { | ||
| throw new RuntimeException("name은 5자를 초과할 수 없습니다. name :" + name); | ||
| } | ||
|
|
||
| this.name = name; |
There was a problem hiding this comment.
Comment on lines
+52
to
+66
| public List<Car> getWinner() { | ||
| List<Car> result = new ArrayList<>(); | ||
|
|
||
| int max = 0; | ||
| for (Car car : carList) { | ||
| car.print(); | ||
| if (max < car.getCurrentPosition()) { | ||
| max = car.getCurrentPosition(); | ||
| result.clear(); | ||
| result.add(car); | ||
| } else if (max == car.getCurrentPosition()) { | ||
| result.add(car); | ||
| } | ||
| } | ||
| System.out.print("\n"); | ||
|
|
||
| return result; |
There was a problem hiding this comment.
이러한 역할을 담당해줄 별도의 객체를 설계해보면 어떨까요?
일급컬렉션에 대해 알아보시면 좋을것 같습니다. :)
Comment on lines
+48
to
+57
| while(racing.isRemainTry()){ | ||
| racing.move(); | ||
| } | ||
|
|
||
| List<Car> winnerList = racing.getWinner(); | ||
| int winnerPosition = winnerList.get(0).getCurrentPosition(); | ||
|
|
||
| for(Car car : racing.getCarList()){ | ||
| assertThat(car.getCurrentPosition()).isLessThanOrEqualTo(winnerPosition); | ||
| } |
There was a problem hiding this comment.
테스트 코드에서 while과 for-loop는 제거해주세요~!
테스트 코드는 한번에 읽기 좋아야 합니다!
만약 이러한 로직이 없이 테스트를 수행해야 한다면 객체의 설계가 잘못되었거나 새로운 객체가 필요하지 않은지 고민해보시면 좋을것 같습니다. :)
Comment on lines
+55
to
+63
| int max = 0; | ||
| for (Car car : carList) { | ||
| car.print(); | ||
| if (max < car.getCurrentPosition()) { | ||
| max = car.getCurrentPosition(); | ||
| result.clear(); | ||
| result.add(car); | ||
| } else if (max == car.getCurrentPosition()) { | ||
| result.add(car); | ||
| } |
There was a problem hiding this comment.
else 예약어를 쓰지 않는다.
힌트: if 조건절에서 값을 return하는 방식으로 구현하면 else를 사용하지 않아도 된다.
else를 쓰지 말라고 하니 switch/case로 구현하는 경우가 있는데 switch/case도 허용하지 않는다.
else는 사용하지 않고 구현해보시지요~!
indent(인덴트, 들여쓰기) depth를 2를 넘지 않도록 구현한다. 1까지만 허용한다.
예를 들어 while문 안에 if문이 있으면 들여쓰기는 2이다.
그리고 depth 조건도 위배된 것 같습니다. :)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
4단계 구현하였습니다.
구현하면서, 이전 단계에서 피드백주셨던 내용들 반영하였습니다.