Skip to content

Lv 1 코드 리뷰 #2

@ChanghwanK

Description

@ChanghwanK
  1. Work API를 보면 pw와 같은 정보를 pathvariable로 받는 케이스가 있는데 지양하는 것이 좋아보입니다. 나아가 해당 URL은 Restful 하지도 않은 것 같습니다.
  2. Work Service를 보면 Work work= new Work 로 되어 있는데 = 앞에 공백 추가해주세요 추가로 Work 엔티티의 생성자쪽에선 = 양옆에 공백이 없는데 추가해주세요
  • 코드는 최대한 꼼꼼하게 일관된 형태로 작성해주세요. 같이 일하고픈 개발자의 기본 소양입니다.
  • 코드 컨벤션을 검색해서 보시면 좋을 것 같습니다.
  1. SimpleDateFormat은 Thread Safe 하지 않습니다. 변경해주시고 항상 Thread Safe을 유념해주세요(개념은 찾아보시길 권장합니다.)
  2. 개인적으로 Service에서 ResponseDto를 만드는 것은 역할을 벗어난다고 생각합니다. Mapper란 개념을 찾아보시고 수강생님의 생각을 정리해보세요!! 그렇다고 Service에서 Entity를 리턴하는 것은 지금처럼 지양해주세요.
  3. JPA Repository를 사용하지 않은 이유가 있나요?

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions