Dion의 PR 리뷰
- List를 직접 return하는 형태는 API디자인시 지양해야할 부분
breaking change가 발생할 수 있다. Wrapping DTO를 만들어서 한번 감싸준다.
-
RequestScoreDTO와 같은 네이밍 X → ScoreRequest 와 같은 형태의 네이밍 전략을 사용한다. DTO와 같이 불필요한 내용은 제거하는 것이 좋음
-
생성자 인젝션인 경우 final 키워드 붙여주기
-
로직을 생각하며 네이밍 신경쓰기
예) 아래의 convertToGameScoreDTO(Long gameId) 는 gameId를 가지고 GameScoreDTO로 바로 변환하는게 아니다.
public GameScoreDTO convertToGameScoreDTO(Long gameId) {
Game game = gameRepository.findById(gameId).orElseThrow(GameNotFoundException::new);
Team homeTeam = teamRepository.findById(game.getHomeTeamId()).orElseThrow(TeamNotFoundException::new);
Team awayTeam = teamRepository.findById(game.getAwayTeamId()).orElseThrow(TeamNotFoundException::new);
return new GameScoreDTO(game, homeTeam, awayTeam);
}- Domain 객체에 getter는 필요한 시점에 추가하기! (객체지향 프로그래밍을 위해)
- Set에 객체를 담을때는 해당 객체에 equals와 hashCode 오버라이딩하기
오늘 한일
- 호눅스의 인프라 수업 아는 내용이 많이 나와서 그런가 재밌었다. 매일 수업이 이러면 좋으련만..ㅠㅠ
- 코테 스터디 문제 다 못풀었다.. 코테도 공격적으로 준비해야할거 같은데 지금은 팀프로젝트만으로도 벅차니깐 코쿼 끝나고 진짜 열심히 준비해야지
- PR 보낸거 피드백 받았는데 구현을 너무 급하게 했나... 라는 생각이 든다. 네이밍도 항상 로직을 생각하면서 더 고민하고 구현해야겠다.
- 내일 코쿼가야해서 피드백 받은거 다 수정하지 못했다. 얼른 마무리해야지
