2019.08.26 회의록 (코드리뷰) - woowa-turkey/miniprojects-2019 GitHub Wiki

전체 코드리뷰

  • GoodController를 분리하는 것이 좋아보임 - chulsea

  • api url를 안쓰더라도 필요하다 싶은 것은 만들자. (O)

    → 201에서 location은 권장사항을 뿐 필수는 아니므로 null도 상관없다.

  • good 생성/삭제 (코치찬스)

    → POST HTTP Method

  • Post 도메인 Validator 사용

  • BaseControllerTest에 addPost, addComment - easy

  • Good에서 isGood 메서드 이름 변경 - chulsea

  • userId로 비교 - chulsea

  • PostResponse init 팩토리 메서드 - chulsea

  • 여러 서비스를 쓰는 순간 컨트롤러에서 로직이 생기니깐...

  • findBy* @Transaction(readOnly = true)

  • 추후 log 파일로 출력 문제 고민