[feat] contents 공통 코드 작성 #5
Conversation
ssunnykku
requested review from
rhaehf,
K-0joo,
pie2457,
jw427 and
jeongeungyeong
| @Controller | ||
| @RequestMapping("/contents") | ||
| public class ContentController { | ||
| } |
| import org.springframework.stereotype.Controller; | ||
| import org.springframework.web.bind.annotation.RequestMapping; | ||
|
|
||
| @Controller |
There was a problem hiding this comment.
@RestController 대신 @Controller를 사용하신 이유가 있을까요??
There was a problem hiding this comment.
@RestController로 수정하겠습니다.
|
|
||
| @Entity | ||
| @Getter | ||
| @Setter |
| @NoArgsConstructor | ||
| @AllArgsConstructor | ||
| @Builder | ||
| @ToString |
There was a problem hiding this comment.
@ToString도 당장은 사용할 필요가 없어보여요 !
| @EntityListeners(AuditingEntityListener.class) | ||
| public class Content { | ||
| @Id | ||
| @GeneratedValue(strategy = GenerationType.SEQUENCE) |
There was a problem hiding this comment.
GenerationType을 SEQUENCE로 하신 이유가 있을까요?
There was a problem hiding this comment.
찾아보니 오라클 등은 sequence를 주로 사용하고 mysql의 경우에는 IDENTITY를 사용한다고 하네요! 수정하겠습니다.
| @CreatedDate | ||
| private LocalDateTime createdAt; | ||
|
|
||
| @ManyToOne |
There was a problem hiding this comment.
@ManyToOne을 사용하실 때 즉시로딩과 지연로딩에 대해서 들어보신 적 있을까요!?
| private Long id; | ||
| private Long likeCount; | ||
| @Size(max = 50) | ||
| private String type; |
There was a problem hiding this comment.
type은 String이 아니라 Enum이네요 !
| @Column(name = "userId", nullable = false) | ||
| private UUID userId; | ||
|
|
||
| @OneToMany(mappedBy = "user") |
There was a problem hiding this comment.
이거는 단방향 연관관계와 양방향 연관관계를 알아보시고, 어떻게 사용할지 저희끼리 회의를 해야할 것 같습니다!
| private UUID userId; | ||
|
|
||
| @OneToMany(mappedBy = "user") | ||
| @Builder.Default |
There was a problem hiding this comment.
@Builder.Default는 처음보네요 ! 왜 사용하셨는지 궁금합니당💭
There was a problem hiding this comment.
현재는 필요없는 설정이라 삭제하겠습니다.
| @Column(name = "userId", nullable = false) | ||
| private UUID userId; | ||
|
|
||
| @OneToMany(mappedBy = "user") |
There was a problem hiding this comment.
저도 양방향 연관관계에 대한 논의가 필요하다고 생각합니다!
| @Getter | ||
| @Setter | ||
| @Table(name="contents") | ||
| @NoArgsConstructor |
There was a problem hiding this comment.
@NoArgsConstructor를 @NoArgsConstructor (access = AccessLevel.PROTECTED) 로 변경하면 어떨지 제안합니다.
There was a problem hiding this comment.
기본 access 설정은 AccessLevel.PUBLIC이고, 별도로 access 속성을 설정하지 않으면 public 접근 수준으로 기본 생성자가 만들어진다고 합니다.
다른 패키지에서 엔티티에 대한 접근이 너무 자유로우면 안될것 같다고 생각해서 protected로 설정을 추가하자고 제안했던거였습니다.
| public class User { | ||
| @Id | ||
| @GeneratedValue(strategy = GenerationType.UUID) | ||
| @Column(name = "userId", nullable = false) |
There was a problem hiding this comment.
혹시 user_id로 통일해도 될까요?
|
다 확인했습니다! 감사합니다! |
Issue
PR 타입(하나 이상의 PR 타입을 선택해주세요)
반영 브랜치
feat/contents -> dev
변경 사항
공통 코드 작성