일단 머지하고 수정할 부분 다음 PR에 적용하는게 나을거같아요

멘토님 PR 피드백

• 한줄로 적으면 가독성이 떨어질 수 있다.

  

• Service 인터페이스 분리

  

• naming

  

• 이름 길이 규칙, 평균점수 소수점 → 내림

  • validation 정규표현식
  • @NotBlank
  • @Min(0)
  • @Email
  • https://hyeran-story.tistory.com/81 - Spring Validation Annotation 총정리

• softDelete setter말고 각각 엔티티 내부에서 변경

  • baseEntity에서 함수 이름을 수정하면 되지않을까요?

• SoftDelete하면 조회 안되게

  • 테스트에서 size(0)이 되게
  • 이렇게 되어있는줄 알았는데 아닌가봐요..?

• 인증 인가가 필요한 로직인 경우 어떻게 테스트 할 수 있는지

• Color Code validation

  • http://jmlim.github.io/javascript/2018/12/11/javascript-regex-colorcode/

컨벤션 정할 부분

• Exception
• Test
  • 멘토님 : 서비스 테스트를 SpringBootTest로 하신 이유?
  • mock 객체를 활용한 테스트보다는 직접 SpringBootTest로 확인해보고 싶었다~
  • 통합 테스트 필수! + 필요에 따라 유닛 테스트는 자유로
• Controller
  • /api/v1/users 공통?
• Service
  • Impl 클래스 사용 X → 인터페이스 사용
• Entity
  • @GeneratedValue(strategy = GenerationType.AUTO)? Identity?
• 생성자 주입 할때 @RequiredArgsConstructor or 생성자
  • 다 롬복 사용