학습하는 분들께 도움이 되고, 더 좋은 답변을 드릴 수 있도록 질문전에 다음을 꼭 확인해주세요.

1. 강의 내용과 관련된 질문을 남겨주세요.
2. 인프런의 질문 게시판과 자주 하는 질문(링크)을 먼저 확인해주세요.
(자주 하는 질문 링크: https://bit.ly/3fX6ygx)
3. 질문 잘하기 메뉴얼(링크)을 먼저 읽어주세요.
(질문 잘하기 메뉴얼 링크: https://bit.ly/2UfeqCG)

질문 시에는 위 내용은 삭제하고 다음 내용을 남겨주세요.
=========================================
[질문 템플릿]
1. 강의 내용과 관련된 질문인가요? (예/아니오)
2. 인프런의 질문 게시판과 자주 하는 질문에 없는 내용인가요? (예/아니오)
3. 질문 잘하기 메뉴얼을 읽어보셨나요? (예/아니오)

[질문 내용]
여기에 질문 내용을 남겨주세요.

 

안녕하세요. 영항님! 좋은 강의 잘 듣고 있습니다.

강의를 듣고 토이 프로젝트를 진행하는 와중에 궁금한 점이 생겨 질문글을 올리게 되었습니다.

저는 Board라는 Entity를 다음과 같이 선언하였습니다.

@Id
@GeneratedValue(strategy = GenerationType.IDENTITY)
@Column(name = "board_id")
private Long id;
private String title;
private String topic;
private String content;
@Enumerated(EnumType.STRING)
private StudyState studyState;
@Enumerated(EnumType.STRING)
private RecruitState recruitState;
private Long headCount;
@ManyToOne(fetch = FetchType.LAZY)
@JoinColumn(name = "member_id")
private Member member;

그리고 저는 프론트에서부터 전달받은 Dto로 실제 Board Entity를 저장하려고 했습니다.

그래서 저는 Dto로 부터 Entity를 만들 수 있는 메소드 toEntity를 다음과 같이 구성하였습니다.

@Data
@NoArgsConstructor
@AllArgsConstructor
public class BoardCreateRequestDto {
    private Long memberId;
    private String title;
    private String topic;
    private String content;
    private Long headCount;
    private StudyState studyState;
    private RecruitState recruitState;

    public Board toEntity() {
        return Board.builder()
                .title(this.title)
                .topic(this.topic)
                .content(this.content)
                .studyState(this.studyState)
                .recruitState(this.recruitState)
                .headCount(this.headCount)
                .member(new Member(this.memberId))
                .build();
    }
}

Board board = boardRepository.save(requestDto.toEntity());

실제로 저장을 할 시에 Board Entity에는 Member를 넣어 저장하나, 실제 DB에서는 Board 객체에서의 Id만을 외래키로 저장하기에 다음과 같이 저장해도 무방하다는 것을 알게 되었습니다.

저는 이 방법이 MemberId를 통해 Member 객체를 찾지 않고 저장할 수 있기에 쿼리 1번을 덜 날릴 수 있어 비용적인 측면에서 조금의 이득이 생긴다고 생각하여 다음과 같이 구성해보았습니다.

이러한 방법이 문제가 있는지 궁금합니다.

@Query("select b from Board b join fetch b.member")
List<Board> findAllByFetchJoin();

@Test
public void fetchJoin() throws Exception {
    //given
    Member member = Member.builder()
            .email("email")
            .nickname("nick")
            .build();
    Member saveMember = memberRepository.save(member);

    for (int i = 0; i < 10; i++) {
        Board board = Board.builder()
                .title("Test")
                .member(new Member(saveMember.getId()))
                .build();
        boardRepository.save(board);
    }

    em.flush();
    em.clear();

    //when
    List<Board> boards = boardRepository.findAllByFetchJoin();

    //then
    for (Board board : boards) {
        System.out.println("board = " + board.getMember().getEmail());
    }
}