TDD 3회차

🚀 첫 피드백을 받다.

박재성님의 TDD강의 ch1 메인 실습인 '숫자 야구게임 구현'을 하고, PR을 요청하였었다.

바쁘시지만 시간 될 때 피드백 주시겠다던 친절한 리뷰어님..

첫 피드백을 받았고, 다소 어렵게 짜는 경향이 있다고 하셨다.

고민을 하다가, 숫자 야구게임의 전 단계인 문자열 계산기를 먼저 해보기로 했다.

특정 습관을 줄이려면 상대적으로 쉬운 것에서부터 고쳐나가야 한다고 생각했기 때문이다.

추가적으로 야구게임의 피드백들 중, 적용할 수 있는 것은 적용하려고 생각이 들었다.

• 문자열 테스트에 적용할 수 있는 피드백들 나열

1. EOF

2. 커밋 컨벤션

3. 코딩 컨벤션(IDE활용)

4. toString을 화면 요구사항을 위해 사용하지 말 것
   -> 주로 어디에 사용될까? 로깅?

5. setter 대신 명시적 초기화 / 생성자 / 타 메서드

6. 매직넘버(원시값?)

7. 핵심 로직을 구현하는 코드와 UI를 담당하는 로직을 구분한다.

8. 자료구조의 이름을 그대로 사용x(list), set 으로 바뀌면?

9. 유틸클래스는 다른 서비스에서도 이용가능할까를 고민한다.

10. @DisplayName()적극 사용

11. reflection
    (1) reflection 가독성 ↓, 어쩔 수 없는 경우 있지만, 그런 상황 안 만들기
    (2) 하나의 public메서드가 많은 책임(private's')을 가지고 있을 수 있는 신호
    -> private method는 테스트를 하지 않는 것인가?(public으로 테스트 하고 테스트를 지우거나 주석처리 하는 건가?)

12. 무언가 반복적인 작업을 할때 이용할 만한 것이 없는지 고민(Arrays, 가변인자파라미터 등)

🚀 좀 더 알아볼 요구사항들

1. EOF 설정 - intellij 설정 포스팅에 반영 완료.(이하 참고링크)
   -> https://minz.dev/19
   -> https://stackoverflow.com/questions/729692/why-should-text-files-end-with-a-newline

2. 커밋 컨벤션
   (-) 지금 할 내용이 아닌 듯 하나.. 기본적인 것 포스팅 하였다.

3. 코딩 컨벤션
   (-) 포스팅 및 설정 포스팅 반영 완료

6. 매직 넘버 금지 : 원시값 포장하라
   (-) 가독성을 위하여 원시 값은 포장해야 한다.
   (-) 논쟁거리 중 하나이며, 간혹 매직넘버를 사용하는 게 가독성이 좋아지는 경우가 있다고 한다. 다음에 아래 링크를 보자!
   -> https://stackoverflow.com/questions/47882/what-is-a-magic-number-and-why-is-it-bad

🚀 다음에 할 일

• 위 사항들을 모두 적용하여, 문자열 계산기를 해볼 생각이다.