New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[김대겸] Step1 PR #756
[김대겸] Step1 PR #756
Conversation
- Sections 일급 컬렉션 추가
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
안녕하세요 이번미션 함께하게된 리뷰어 강태화입니다 😄
1단계 꼼꼼하게 잘 진행해주셨습니다.
몇가지 코멘트 남겨드렸는데 확인 부탁드려요~
merge
진행하겠습니다.
다음단계 진행해주세요~
- [X] Stations 조회 기능 | ||
- [X] Domain의 단위 테스트를 작성하기 | ||
- [X] 새로운 로직 만들기 | ||
- [X] 정상 동작 확인 후 기존 로직 제거 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
꼼꼼하게 잘 정리해주셨네요 👍
void addSection() { | ||
Station 시청역 = new Station("시청역"); | ||
|
||
line.addSection(new Section(종각역, 시청역, 5)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
도메인 테스트 잘 진행해주셨네요 👍
|
||
@BeforeEach | ||
void setUp() { | ||
종각역 = new Station("종각역"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
보시면 도움될듯하여 공유합니다!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
테스트 픽스처를 올바르게 사용하도록 습관들여봐야겠습니다. 좋은 정보 공유 감사드립니다!
.findFirst(); | ||
} | ||
|
||
private Optional<Section> findUpStation(final Station station) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Optional
사용시 잘 정리된 블로그 내용있어서 공유해요~
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Optional 바르게 쓰는 방법에 대해 잘 정리가 되어있네요 👍
유용한 정보 감사합니다.
this.distance = new Distance(distance); | ||
} | ||
|
||
public Section(Station upStation, Station downStation, int distance) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
참고해주세요~
안녕하세요 리뷰어님 😄
Step1 인수 테스트 기반 리팩터링 PR 리뷰 요청 드립니다.
잘 부탁드립니다 :)