inflearn logo
강의

강의

N
챌린지

챌린지

멘토링

멘토링

N
클립

클립

로드맵

로드맵

지식공유

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

회원 수정 API

select 쿼리없이 dto 만들지 않은 이유

428

dlwogns3413

작성한 질문수 19

0

강의에서는 아래와 같이 select 쿼리를 이용해 업데이트 된 회원을 조회후 response로 전달해 주었는데

@PutMapping("/api/v2/members/{id}")
public UpdateMemberResponse updateMemberV2(@PathVariable("id") Long id,
                                           @RequestBody @Valid UpdateMemberRequest request) {

    memberService.update(id,request.getName());
    Member findMember = memberService.findOne(id);
    return new UpdateMemberResponse(findMember.getId(), findMember.getName());
}

아래와 같이 하지 않은 이유가 있을까요??

@PutMapping("/api/v2/members/{id}")
public UpdateMemberResponse updateMemberV2(@PathVariable("id") Long id,
                                           @RequestBody @Valid UpdateMemberRequest request) {

    memberService.update(id, request.getName());
    return new UpdateMemberResponse(id, request.getName());
}

java spring spring-boot jpa

답변 1

1

나무늘보

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

결론적으로, findOne(id)를 사용하는 것이 안전합니다.

request 객체는 클라이언트가 백으로 전달한 값이고 findOne(id)를 통해 획득한 Member 는 데이터베이스와 동기화된 데이터인데, 이 둘이 일치하지 않을 수 있습니다. 가령 memberService.update() 내부에서 클라이언트에서 넘어온 name 값을 변경한 후 (띄어쓰기 제거, 특수문자 제거 등의 값 변경 등등) 저장하도록 되어있다면 request.getName() 과 member.getName()이 다를 수 있습니다.

그리고 코드의 늬앙스가 달라지는데요,
'사용자가 전달한 값으로 Member를 갱신 하고, 응답객체를 생성한다' 에서 '사용자가 전달한 값으로 Member를 갱신하고, 갱신한 Member로 응답객체를 생성한다' 로 바뀌어 조금 다른 이야기가 됩니다.

감사합니다.

강의 관련 외 질문입니다.

0

66

2

SpringBoot4 + Hibernate7 모듈 등록 방법 공유

0

90

1

BeanCreationException

0

89

3

Update 후 UpdateMemberResponse 매핑할 때

0

49

1

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

0

99

2

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

0

70

1

Query Dsl Q파일 질문입니다.

0

83

1

루트 쿼리라는것은

0

59

1

메서드를 분리하는 기준

0

63

1

findAllWithMemberDelivery 메서드 질문드립니다.

0

110

3

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

0

85

2

fetch join과 영속화와 OSIV의 관계

0

86

2

Distinct 사용 전 결과에 대한 의문

0

113

2

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

0

58

1

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

0

78

2

dto 필드 속 엔티티 여부

0

60

1

뷰템플릿 사용 시

0

76

2

Result 클래스 관련 질문

0

56

1

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

0

86

1

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

0

137

1

OSIV ON 상태일 때

0

96

1

fetch join VS fetch join 페이징 궁금증

0

180

2

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

0

105

1

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

0

165

2