반응형

미르 3

[220218] 페어 협업미션 리팩토링 피드백 (레벨1 - 자동차 경주 미션)

우아한테크코스에선, 같은 미션이어도 1단계-구현, 2단계-리팩토링으로 나누어진다. 지난 포스팅에서 1단계-구현 피드백 포스팅을 다뤄보았으니, 이번엔 2단계-리팩토링 피드백 포스팅을 다뤄보려한다. 1단계-구현 피드백은 여기서 볼 수 있다. https://kth990303.tistory.com/262 [220211][JAVA] 페어 협업미션 리뷰어의 피드백_문자열 덧셈 계산기 저번에 알렉스와 함께 페어 프로그래밍을 진행한 '문자열 덧셈 계산기', '자동차 경주 미션'이 벌써 피드백이 올라왔다! 리뷰어님(이하 미르)께서 새벽 2시에 피드백해주셨던데, 미르의 성실함에 kth990303.tistory.com 2단계 - 자동차 경주 리팩터링 PR은 https://github.com/woowacourse/java-r..

[220213][JAVA] 페어 협업미션 리뷰어의 피드백_자동차 경주 미션

이번에는 자동차 경주 미션 관련 피드백이다. 문자열 덧셈 계산기에 비해 로직이 많다보니, 리뷰 내용도 상당히 많았는데, 한 번 살펴보도록 하자. 전체 피드백은 https://github.com/woowacourse/java-racingcar/pull/256 여기서 볼 수 있으며, 포스팅에는 인상깊었던 피드백 위주로 작성할 예정이다. Stream, Lambda로 불필요한 메소드와 변수를 줄이자. 이번에도 역시 stream 피드백이 존재한다. 아직 stream에 익숙하지 않아서 서툰 면이 많이 존재하는데, filter를 이용하면 List에 maxPosition에 해당하는 값들만 넣어줄 수 있기 때문에 stream을 사용하면 26~29라인에 해당하는 줄들을 바로 return문에 넣을 수 있다. return c..

[220211][JAVA] 페어 협업미션 리뷰어의 피드백_문자열 덧셈 계산기

저번에 알렉스와 함께 페어 프로그래밍을 진행한 '문자열 덧셈 계산기', '자동차 경주 미션'이 벌써 피드백이 올라왔다! 리뷰어님(이하 미르)께서 새벽 2시에 피드백해주셨던데, 미르의 성실함에 감탄하지 않을 수 없었다... 전체 피드백 내용은 https://github.com/woowacourse/java-racingcar/pull/256 에서 볼 수 있다. [1단계 - 자동차 경주 구현] 케이(김태현) 미션 제출합니다. by kth990303 · Pull Request #256 · woowacourse/j 선배님 안녕하세요! 우테코 4기 케이입니다. 열심히 배우겠습니다. 잘 부탁드립니다! 미션을 진행하면서 궁금했던 점이 있습니다. CarNameValidator 클래스에서 조건문 내부에 name.isBla..

반응형