콘텐츠로 이동

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()를 써야하는데 굳이 싶다
    • 확장을 고려한 코드는 확장이 필요한 타이밍에 만드는 것도 좋다
    • 불필요한 인스턴스 변수를 굳이 만들지 말자!
    • 의미론적으로 유의미하다고 불필요한 함수를 만들지 말자