Spring JPA | Entity와 Dto 분리

 

Spring JPA를 활용한 프로젝트를 진행하면서 새롭게 알게된 점이 있어 정리하고자 한다.

Entity를 정의해서 사용하면서 이전 프로젝트에서는 서비스단에서 Entity와 Dto를 빌더 패턴을 사용해서 데이터 값을 넣어주곤 했었는데, 이번 프로젝트에서는 Entity에 toResponseDto() 메서드와 생성자를 만들어서 데이터 값을 넣어주는 코드를 작성했었다.

이에 대해 Entity를 보호하라! 라는 피드백을 받았고, 고치는 방법도 전달받았기에 기록해보려고 한다.

 

 

🐧 Entity를 보호해야 하는 이유

Entity는 중요한 데이터 객체이므로 최대한 순수한 데이터 객체로 유지하고, DTO와의 의존성을 없애는 것이 좋다.
Entity는 데이터베이스와 1:1 매핑되는 객체인데, DTO와의 변환 로직이 들어가면 역할이 애매해질 수 있다.

따라서, 변환이 필요하다면 DTO에 변환 로직을 구현하는 것이 옳다!

 

이전에 작성한 Entity 코드

@Entity
@Table(name = "p_restaurant_menu")
@Getter
@NoArgsConstructor(access = AccessLevel.PROTECTED)
@AllArgsConstructor(access = AccessLevel.PROTECTED)
@Builder
public class Menu extends BaseEntity {

    @Id
    @GeneratedValue(strategy = GenerationType.UUID)
    private UUID rmId;
    @Column(length = 100, nullable = false)
    private String rmName;
    @ColumnDefault("true")
    private Boolean isPublic;

    public Menu(CreateMenuRequestDto requestDto) {
        this.rmName = requestDto.getRmName();
        this.isPublic = requestDto.getIsPublic();
    }

    public MenuResponseDto toResponseDto() {
        return MenuResponseDto
                .builder()
                .rmId(this.rmId)
                .rmName(this.rmName)
                .isPublic(this.isPublic)
                .build();
    }

생성자와 toResponseDto 메서드를 각각 RequestDto와 ResponseDto로 옮겨서 작성해주었다.

 

RequestDto 

@Builder
@Getter
@AllArgsConstructor
public class CreateMenuRequestDto {

    @NotNull
    private UUID resId;
    @Length(max = 100)
    @NotNull
    private String rmName;
    private Boolean isPublic;
    @NotNull
    private UUID resId;
    private Boolean isPublic;

    public Menu toEntity(CreateMenuRequestDto requestDto) {
        return Menu
            .builder()
            .rmName(requestDto.getRmName())
            .isPublic(requestDto.getIsPublic())
            .build();
    }
}

 

ResponseDto

@Getter
@Builder
@AllArgsConstructor
public class MenuResponseDto {
    private UUID rmId;
    private String rmName;
    private Boolean isPublic;
}
    private UUID rmId;
    private String rmName;
    private Boolean isPublic;

    public static MenuResponseDto fromEntity(Menu menu) {
        return MenuResponseDto
            .builder()
            .rmId(menu.getRmId())
            .rmName(menu.getRmName())
            .isPublic(menu.getIsPublic())
            .build();
    }
}