🔙뒤로가기

MemberApiController에 PUT 매핑 추

@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());

}

@Data
@AllArgsConstructor
static class UpdateMemberResponse {
    private Long id;
    private String name;
}

@Data
static class UpdateMemberRequest {
    private String name;
}

회원 수정도 DTO를 요청 파라미터에 제한시켰다.

MemberService에 update 메소드 추가

@Transactional
public void update(Long id, String name) {
    Member member = memberRepository.findOne(id);
    member.setName(name);
}

findOne()메소드를 통해 member객체를 영속화한 채로 불러온 후 setter를 사용한다. 영속성 컨텍스트의 변경 감지 기능을 통해 데이터베이스가 수정된다.

트러블슈팅

@Data
@AllArgsConstructor
private class UpdateMemberResponse {
// ^ 접근제한자를 private으로 함
    private Long id;
    private String name;
}

@Data
private class UpdateMemberRequest {
// ^ 접근제한자를 private으로 함
    private String name;
}

내부 클래스는 정적(static)이어야 한다. @PutMapping("/api/v2/members/{id}") 엔드포인트를 처리하는 updateMemberV2 메서드에서 **UpdateMemberRequest**와 **UpdateMemberResponse**를 사용하고 있다. 따라서, 해당 내부 클래스들에 private 접근 제한자를 설정하면 컨트롤러에서 해당 클래스들을 사용할 수 없어 에러가 발생하게 된다.

static이어야 하는 이유

1. 성능에 유리 데아터 전달을 위한 간단한 컨테이너 역할을 하므로, 인스턴스화하여 상태를 공유할 필요도 딱히 없다. 정적 클래스로 선언하면 컨트롤러 인스턴스와 독립적으로 사용할 수 있으며, 리소스 사용량이 줄어들고 메모리 할당이 효율적으로 이루어진다.
2. 코드 가독성 향상 필요할 때마다 클래스를 인스턴화 없이 바로 사용할 수 있어 코드가 간결해진다.
3. 구조적 명확성 내부 클래스는 외부 클래스와 별개로 동작하므로 외부/내부의 연결이 약해진다. 내부 클래스가 외부 클래스에 종속되지 않고 독립적일 수 있다.

<aside> ⚠️ 본 예제에서는 부분 업데이트를 하며 PUT방식을 사용했는데, PUT은 전체 업데이트를 할 때 사용하는 것이 맞다. 부분 업데이트에는 PATCH나 POST를 사용하는 것이 RESTful한 방식이다.

</aside>