select 쿼리없이 dto 만들지 않은 이유
428
작성한 질문수 19
강의에서는 아래와 같이 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());
}
답변 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





