[1단계 - 자동차 경주 구현] 레넌(조형래) 미션 제출합니다.#294
Merged
48 commits merged intowoowacourse:broraefrom Feb 14, 2022
Merged
Conversation
ghost
suggested changes
Feb 12, 2022
ghost
left a comment
There was a problem hiding this comment.
안녕하세요 레넌! 리뷰어 던입니다.
너무 수고 많으셨습니다! 잘 구현해주셨습니다! 💯
몇 가지 코멘트 남겼으니 확인 부탁드릴게요!
질문하신것들 답변 드리겠습니다! 제 말이 반드시 맞다는건 아니고 저는 이렇게 생각한다는 정도로 참고하시면 좋겠습니다.
- Controller는 View와 Domain사이에서 값을 전달해주는 역할로도 충분하다고 생각합니다. 말씀하신 메서드들은 값을 전달하는 로직이 아니라 게임을 진행하는 로직이기 때문에 Controller가 아니라 게임을 진행하는 Domain이 수행하는게 어떨까합니다.
- FuntionalInterface를 이용해서 람다를 주입해주는 방식으로 구현해주셨는데요! 저는 괜찮다고 생각합니다. 다만, 이것 때문에 새로운 생성자를 만드는 방법은 지양하는게 좋다고 생각합니다.
- 테스트 코드를 작성하기 위한 용도로 새로운 메서드를 만드는건 지양하는게 좋다고 생각합니다. 테스트 코드를 작성하는 목적을 한번 생각해보시면 도움될 것 같습니다!
ghost
approved these changes
Feb 14, 2022
ghost
left a comment
There was a problem hiding this comment.
안녕하세요 레넌! 질문에 대한 답변과 간단한 코멘트 남겼으니 확인 부탁드립니다.
다음 단계에서 뵙겠습니다. 1단계 구현하시느라 수고많으셨습니다! 👍
This pull request was closed.
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
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.
안녕하세요.
많이 부족하지만, 잘 부탁드립니다!
질문사항
MVC 패턴으로 코드를 작성하여 보았는데, Cars 클래스에 있는 getWinner() 메소드나 moveRound() 같은 메소드는 controller에서 처리하는게 맞을까요?? Controller와 model 에서 각각 어느 수준까지 처리를 해줘야할지 헷갈립니다.
RandomUtil 인터페이스를 만들어 랜덤 테스트를 진행하였는데, 이런식으로 랜덤 테스트를 진행해도 괜찮을까요??
실제 프로그램에서는 사용되지 않지만, 테스트를 위한 용도만으로 메소드를 작성하여 테스트 검증을 해도 되나요??
+++
피드백 해주신 내용 수정하면서 질문사항 추가적으로 남겼습니다. 감사합니다.