[사다리타기] 3단계 - 사다리(게임 실행) #1605
[사다리타기] 3단계 - 사다리(게임 실행) #1605
Conversation
이펙티브 자바 72 표준 예외를 사용하라에
|
|
||
| public class Line { | ||
|
|
||
| private final List<Boolean> connections; | ||
|
|
There was a problem hiding this comment.
#1561 (comment)
설계를 변경하는게 쉽지 않은 선택일 것 같아요 👍
#1561 (comment)
이 코멘트도 고민해 볼 수 있지 않을까요?
| public int getLastVerticalLineIndex(final Participant participant) { | ||
| return this.getLastVerticalLineIndex(this.participants.indexOf(participant)); | ||
| } |
There was a problem hiding this comment.
이 메서드는 view에서 호출 되고 있는데 view가 console이 아니라 web으로 변경된다면 호출 될 수 있을까요?
참여자의 결과를 찾는건 비즈니스 로직이라고 생각해요.
이 부분은 view에서 호출 하는 것이 아닌 controller에서 도메인 모델에게 시키고 결과를 view로 전달해야 하지 않을까요?
view에서는 getter만 사용해서 출력만 담당하는 것이 어떨까요?
일반적으로 getter는 필드 그대로를 꺼내는 것을 의미합니다.
과정중에 getter 사용을 하지 말라는 것은 비즈니스를 풀어내는 도메인 모델끼리 getter를 사용하지 말라는 것이고 view에서는 getter를 사용해야 합니다.
There was a problem hiding this comment.
ladder에 많은 비즈니스 로직이 있는 것 같아요. 어느 정도는 line에게 위임할 수 있지 않을까요?
Line#isConnected 를 제외하면 아무런 역할이 없는 것 같아서요.
실제로 Line 필드에 List<Boolean> connections 가 있어서요. index 1은 해당 line을 통과하면 어떤 index가 되는 지를 line에게 맡겨볼 수 있지 않을까요?
| private LadderFactory() { | ||
| } |
안녕하세요. 박명진입니다.
2단계 피드백 보완 및 3단계 요구사항 구현입니다.
리뷰 부탁드립니다. 🙏