2022 12 28
2022-12-28¶
자동차 경주 피드백¶
- Setter vs Change
- Setter : Null -> A
- Change : A -> B
- Test Fixture
@BeforeEach를 활용하여 각 테스트 케이스에서 중복으로 사용하는 Fixture를 초기
- 코드 리뷰
- 생성 조건에 대한 validation을 더 꼼꼼히
- Null 체크
- empty string
- 네이밍이 check라면 return 값이 존재하리라 생각하기 어려움
- Optional을 활용한다면 isEmpty()를 써야하는데 굳이 싶다
- 확장을 고려한 코드는 확장이 필요한 타이밍에 만드는 것도 좋다
- 불필요한 인스턴스 변수를 굳이 만들지 말자!
- 의미론적으로 유의미하다고 불필요한 함수를 만들지 말자
- 생성 조건에 대한 validation을 더 꼼꼼히