Step5 PR요청드립니다. #6227
Open
Step5 PR요청드립니다. #6227
Conversation
This file contains hidden or 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
javajigi
requested changes
Nov 18, 2025
Contributor
javajigi
left a comment
javajigi
left a comment
There was a problem hiding this comment.
전체 코드를 삭제하고 다시 구현하는 것이 쉽지 않았을텐데요.
다시 구현하면서 새로운 인사이트를 느낀 점이 인상적이네요. 💯
확실히 기존에 비해 전체적인 설계가 개선된 것을 느낄 수 있네요.
추가로 "규칙 8: 일급 콜렉션을 쓴다." 원칙까지 적용하면서 마무리하면 좋을 것 같아 피드백 남겨 봅니다.
이제 자동차 경주 미션은 충분히 도전한 만큼 5단계는 시간 나는 짬짬히 도전해 보시고요.
로또 미션을 병행해 보면 어떨까요?
| this.value = value; | ||
| } | ||
|
|
||
| private void vaildation(String value) { |
Contributor
There was a problem hiding this comment.
Suggested change
| private void vaildation(String value) { | |
| private void vaildate(String value) { |
명사보다는 동사로
| return "Position{" + | ||
| "value=" + value + | ||
| '}'; | ||
| public Position increase() { |
| import java.util.List; | ||
|
|
||
| public class RacingGame { | ||
| private List<RacingCar> racingCars; |
Contributor
There was a problem hiding this comment.
한 단계 더 나아가 "규칙 8: 일급 콜렉션을 쓴다." 적용해 구현해 보면 어떨까?
Comment on lines
+8
to
+9
| private List<RacingCar> racingCars; | ||
| private TryCount tryCount; |
Contributor
There was a problem hiding this comment.
Suggested change
| private List<RacingCar> racingCars; | |
| private TryCount tryCount; | |
| private final List<RacingCar> racingCars; | |
| private final TryCount tryCount; |
Comment on lines
+34
to
+36
| if (racingCar.isGreaterThanPosition(maxPosition)) { | ||
| maxPosition = racingCar.getPosition(); | ||
| } |
Contributor
There was a problem hiding this comment.
Suggested change
| if (racingCar.isGreaterThanPosition(maxPosition)) { | |
| maxPosition = racingCar.getPosition(); | |
| } | |
| maxPosition = racingCar.max(maxPosition); |
객체의 값을 꺼내지 말고 메시지를 보낸다.
| public class TryCount { | ||
| private final int value; | ||
|
|
||
| public TryCount(int value) { |
Comment on lines
+17
to
+19
| for (int i = 0; i < position.value(); i++) { | ||
| System.out.print("-"); | ||
| } |
Contributor
There was a problem hiding this comment.
Suggested change
| for (int i = 0; i < position.value(); i++) { | |
| System.out.print("-"); | |
| } | |
| "-".repeat(position.value()); |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
step5 피드백 요청드립니다.
전부 지우고 다시 시작해보니, 접근이 조금 달라진거 같습니다.
결국은 비슷한 결과물이 나왔던거 같은데, 조금 더 하나의 객체에 역할을 부여하는 방향으로 접근해봤습니다.
다만, 마지막 RacingResult는 'MVC 패턴구조로 구현해본 샘플코드'를 보고 접근해봤습니다.
저는 RacingGame으로 끝냈던 상황이었는데, RacingResult를 보니 Game은 play의 역할에만, Result는 결과물에 대한 역할을 부여할 수 있을거 같아서 나누어 보았습니다.
아직 모든 원시값과 문자열을 포장하는데에 익숙하진 않은거 같은데, 아마 너무 절차지향적인 방법으로 많이 개발해와서 그런거 같습니다.
조금씩 적응하며 적용해보려고 합니다.
피드백 주시면 다시 한번 확인해보겠습니다.
감사합니다!