[회고록] 우아한테크 프리코스 회고록1-4

https://github.com/woowacourse-precourse/java-baseball-6/pull/1299

[숫자 야구 게임] 이성호 미션 제출합니다. by dltjdgh0428 · Pull Request #1299 · woowacourse-precourse/java-base

---

첫 PR후 많은 코드리뷰를 받았던 하루였습니다. 제가 코드리뷰를 먼저 많이 해드리니 다른 분들도 그에 보답해서 잘해주셔서 너무 감사드립니다. 그에 따라 1주 차를 마무리할 겸 피드백을 정리해보려 합니다.

 

1. 기능목록의 구체화

이거는 혼자서도 많이 느끼고 있었지만 지적해주셔서 너무 감사했습니다. 10/26일 어제부터 2주 차 미션이 시작됐는데 이 날 하루는 "온전히 기능문서 작성하는데" 쓰자라고 생각하고, 만일 기능문서를 다 작성했다 하더라도 하루가 끝날 때까지 생각하면서 어떤 점을 더 보완할 수 있을까 생각하게 되는 계기가 됐습니다.

2. Helper라는 Model의 모호함.

MVC 모델을 도입함으로써 생겨난 class인데 하는일이 너무 모호했습니다. 뭔가 객체답지 못하고 그냥 짬뽕으로 처리한 느낌이 강해서 다음부터는 Model이름을 봤을 때 직관적으로 기능을 파악할 수 있도록 작성해야겠다고 생각했습니다.

3. 입력과 출력의 분리

String input = Valid.startValid(gameView.printInputMessage());

저두 리팩토링하는 과정에서 발견하지 못했는데 코드를 보니 한 번에 처리하려는 느낌이 강해 보여서 다음엔 더 보완해야겠다. 생각했습니다.

4. 의존관계

public BaseBallGame(Helper helper){
 ...
 }

전체적인 코드를 봤을 때 BaseBallGame모델이 Helper를 파라미터로 받을 이유가 딱히 없는데요. 2주 차 미션에는 최대한 의존관계를 해소하며 코딩하자고 생각했습니다. 피드백 감사합니다.

5. 네이밍

이후에는 네이밍 피드백이 많았는데. 역시 메서드명 클래스명이 가장 어려운 것 같습니다ㅋㅋ..

 

전체적으로 느낀 점은 기능목록 설정이 가장 중요하다...
코딩은 설7구3이라~
설계 70 구현 30인 것 같네요.

 

다들 파이팅~!