inflearn logo
강의

강의

N
챌린지

챌린지

멘토링

멘토링

N
클립

클립

로드맵

로드맵

지식공유

실전! 스프링 부트와 JPA 활용2 - API 개발과 성능 최적화

DTO의 사용 범위와 Controller에서 엔티티 생성 가능 여부

423

양성준

작성한 질문수 4

0

안녕하세요.

해당 강의를 수강하다 궁금증이 생겨 질문 드립니다.

강의에서는

MemberService 같은 경우에

@Transactional
public Long join(Member member) {
    validateDuplicateMember(member); // 중복 회원 검증
    memberRepository.save(member);
    return member.getId();
}

이런식으로 엔터티를 파라미터로 받고,

MemberApiController 같은 경우에는

@PostMapping("/api/v2/members")
public CreateMemberResponse saveMemberV2(@RequestBody @Valid CreateMemberRequest request) {
    Member member = new Member();
    member.setName(request.getName());

    Long id = memberService.join(member);
    return new CreateMemberResponse(id);

}

별도의 DTO를 파라미터로 받아 member 엔티티 객체를 생성하여 set 해주는 식으로 진행하였는데,
(set 말고 생성메소드를 이용해야하는건 알아요!)

 

1. 엔티티를 외부에 노출하면 안되고, API 스펙이 바뀌어도 문제가 생기지 않기 위해 Controller단에 파라미터 및 반환값에 DTO를 사용하는 것은 당연하다. 그렇다면 서비스나 리포지토리에서는 파라미터로 엔티티를 받아야 하나요? DTO를 받아야 하나요??

2. 만약 DTO를 Controller에서만 받고, Service단에서 파라미터로 엔티티를 받는다면, Controller 단에서 엔티티 객체를 생성하여 Dto의 정보를 넣어준 뒤 Service단으로 보내줘야하는데, Controller단에서 엔티티를 생성하는 것이 괜찮은 방법인가요?

 

어떤게 더 좋은 방법이고, 실무에서 더 많이 쓰는 방법인지, 어떤 장단이 있는지도 궁금합니다.

 

java spring spring-boot jpa

답변 1

0

나무늘보

안녕하세요. 양성준님, 공식 서포터즈 코즈위버입니다.

강의에서는 컨트롤러 내에서 DTO를 엔티티로 반환하고 있습니다. 실무에서는 컨트롤러에서 엔티티를 직접 제어하지 않는것이 일반적인 것 같습니다.

컨트롤러는 사용자의 요청과 파라미터를 적절한 서비스에 전달하고, 결과값을 다시 반환하는 역할만 수행하는 것이 좋습니다. 그래서 컨트롤러가 파라미터로 받은 DTO는 그대로 서비스로 전달합니다. 강의와 다르게 컨트롤러에서 서비스를 호출할 때 엔티티를 사용하지 않고 DTO를 사용하지요. 그리고 DTO를 엔티티로 변환하는 것은 서비스 에서 처리하는 것이 일반적입니다.

 

감사합니

0

양성준

답변 감사드립니다.

https://www.inflearn.com/questions/29890/ui-%EA%B3%84%EC%B8%B5%EC%97%90%EC%84%9C-entity-object-%EC%83%9D%EC%84%B1-%EC%A7%88%EB%AC%B8

위 글에서는 컨트롤러에서도 도메인 계층을 의존하기 때문에 엔티티를 생성해서 서비스로 넘겨도 된다고 하셨는데,

혹시 DTO를 서비스로 넘기고 엔티티로 변환하는 것과

Controller에서 엔티티를 생성하는 것 모두 트레이드오프가 있으며,

둘 다 사용해도 괜찮은 것으로 이해해도 되나요?

 

또, DTO를 서비스로 넘겨 엔티티로 변환해서 반환하고,

이를 다시 Controller에서 받아서 DTO로 내보내는 과정을 코드로 짜면

*MemberSerivce

@Transactional
public Member join(MemberDto.CreateMemberRequest request) { // 초기 메소드에만 DTO를 받아줘서 엔티티로 변환해서 반환
    Member member = new Member();
    CreateMemberRequestToEntity(request, member);
    validateDuplicateMember(member); //중복 회원 검증
    memberRepository.save(member);
    return member;
}

private void CreateMemberRequestToEntity(MemberDto.CreateMemberRequest request, Member member) {
    member.createMember(request.getAccountId(), request.getPassword(), request.getNickname(), request.getName(), request.getBirth(),
            request.getGender(), request.getAddress(), request.getAddressDetail(), request.getZipcode(), request.getEmail(), request.getPhonenumber());
}

*MemberApiController

// 멤버 가입
@PostMapping("ypjs/member/join")
public MemberDto.CreateMemberResponse join(@RequestBody @Valid MemberDto.CreateMemberRequest request) {
    Member member = memberService.join(request);
    return new MemberDto.CreateMemberResponse(member.getMemberId());
}

이런식으로 되는게 맞는지 궁금합니다.

그리고 validateDuplicateMember()와 같은 join 하위 메소드는
join에서 이미 DTO를 엔티티로 변환하기 때문에 파라미터로 DTO를 받아 엔티티로 또 변환시키기 보단

엔티티를 받아주는게 더 좋은건지도 궁금합니다!

강의 관련 외 질문입니다.

0

80

2

SpringBoot4 + Hibernate7 모듈 등록 방법 공유

1

97

1

BeanCreationException

0

96

3

Update 후 UpdateMemberResponse 매핑할 때

0

56

1

트랜잭션을 사용 안 할 때 커넥션은 언제 가져오나요?

0

102

2

페이징 + 검색조건 관련해서 질문드립니다.

0

74

1

Query Dsl Q파일 질문입니다.

0

86

1

루트 쿼리라는것은

0

62

1

메서드를 분리하는 기준

0

70

1

findAllWithMemberDelivery 메서드 질문드립니다.

0

116

3

연관관계 매핑을 안 쓸 경우, 사용해야 하는 전략

0

91

2

fetch join과 영속화와 OSIV의 관계

0

90

2

Distinct 사용 전 결과에 대한 의문

0

117

2

레포지토리 계층에서의 트랜잭션에 대한 의문

0

61

1

영속성 컨텍스트 생명주기의 신기한 부분이 있습니다.

0

79

2

dto 필드 속 엔티티 여부

0

63

1

뷰템플릿 사용 시

0

82

2

Result 클래스 관련 질문

0

56

1

@PostConstruct 프록시 관련 질문드립니다

0

88

1

DTO 대신 Form 사용은 안되나요?

0

140

1

OSIV ON 상태일 때

0

99

1

fetch join VS fetch join 페이징 궁금증

0

189

2

양방향 연관관계 알아보는 법?

0

109

1

16강 17강 간단 정리 이게 맞을까요 ?

0

168

2