인프런 커뮤니티 질문&답변

ycllyo님의 프로필 이미지

작성한 질문수

실전! 스프링 부트와 JPA 활용1 - 웹 애플리케이션 개발

update 메서드의 매개변수 작성 스타일?

해결된 질문

작성

·

117

0

// a.
public void update(
            String password,
            String nickname) {
        this.password = password;
        this.nickname = nickname;
    }

// b.
public void update(UpdateContent content) {
        this.password = content.getPassword;
        this.nickname = content.getNickname;
    }

안녕하세요 update 방식에 대해 궁금한점이 생겼는데, Member 엔터티 클래스 내부에 위와 같이 update 메서드를 둔다고 했을 때, 매개변수의 개수가 많다고 가정하면, 별도의 클래스를 만들어서 b.처럼 전송해주는 방식을 사용해도 괜찮을까요? 엔터티 클래스와 별도의 클래스가 결합?되는 느낌이라 찝찝한데, a.와 b.중 좀 더 정석적인 방식이 있는지 궁금합니다!

답변 2

1

안녕하세요. ycllyo님, 공식 서포터즈 y2gcoder입니다.

AI 답변에 더하여 답변드리고 싶습니다!

파라미터가 많아지면 업데이트용 DTO를 사용하는 것은 지극히 자연스러운 것입니다!

다만 의존성에 주의하셨으면 합니다 🙂

영한님께서도 양방향으로 의존하지 않도록 코딩하기를 권장하십니다.

예를 들어 보통 서비스 는 도메인에 의존합니다. 이때 UpdateContent를 서비스에서 만들고 도메인 엔티티에서 의존하도록 하면 서비스와 도메인 간의 양방향 의존성이 생기게 되어 유지보수에 좋지 않습니다!

도메인 내에 UpdateContent를 만들고, 이를 사용하는 것을 추천합니다 🙂

 

감사합니다.

ycllyo님의 프로필 이미지
ycllyo
질문자

답변 감사드립니다! 그런데 혹시 도메인 내에 만든다는게 디렉토리 관점에서 말씀하시는걸까요? 아니면 엔터티 클래스에 내부 클래스 생성을 말씀하시는걸까요?

패키지 관점에서 말씀드렸습니다 🙂

0

안녕하세요, 인프런 AI 인턴입니다.

매개변수의 개수가 많을 때, DTO(Data Transfer Object)를 사용하는 방식(b 방식)과 각 매개변수를 직접 전달하는 방식(a 방식) 사이에서 선택해야 하는 상황은 개발을 하다 보면 자주 마주치게 됩니다. 여러분이 제시한 두 가지 사례에 관해서는, 정석적이라고 할 수 있는 ‘하나의 올바른 답’이 존재하지는 않습니다. 대신, 각 방식의 장단점과 상황에 맞는 사용법을 이해하는 것이 중요합니다.

a 방식은 간단한 경우에 적합하며, 메서드 시그니처를 통해 수정되는 필드가 무엇인지 명확하게 알 수 있는 장점이 있습니다. 하지만 매개변수가 많아지면 메서드 호출 시 인자의 순서를 혼동하기 쉽고, 메서드 시그니처도 복잡해집니다.

b 방식은 매개변수의 개수가 많거나, 파라미터의 값 중 일부만 종종 변경되는 경우 유용합니다. DTO를 사용하면 필드의 추가나 제거가 발생하여도 메서드 시그니처는 변경되지 않으므로, 코드의 유지 보수성이 향상됩니다. 또한, 해당 DTO 내에서 데이터의 검증 로직 등을 추가할 수 있어 유효성 검사를 간결하게 관리할 수 있는 이점이 있습니다. 하지만 엔터티 클래스와 DTO 간에 추가적인 매핑 과정이 필요하다는 단점이 있습니다.

실제 질문에서 언급한 ‘정석적인 방식이 있다면’에 대한 답변으로는, 상황에 따라 가장 적합한 방법을 선택하는 것입니다. 수정해야 할 필드가 많고, 특정 상황에서만 일부 필드가 업데이트 되는 복잡한 경우라면 DTO를 사용하는 방식을, 간단하게 몇 개의 필드만 변경되는 경우라면 직접 매개변수를 넘기는 방식을 선택할 수 있겠습니다.

질문하신 내용과 유사한 문제를 다룬 다음 질문 드립니다.에서는 매개변수를 확인하고 업데이트하는 로직을 엔터티 내부에 두는 것에 대해 설명하고 있습니다. 해당 답변에는 업데이트 로직이 엔티티 내부에 두는 것이 적절할지, 서비스 레이어에 구현하는 것이 낫겠는지에 대한 토론이 포함되어 있습니다. 여기서는 서비스 레이어보다는 엔티티 내부에 업데이트 로직을 두는 것을 권장하고 있으며, 필요에 따른 유효성 검사나 데이터 검증 로직도 함께 고려할 수 있다는 점에서 관련지어 생각해볼 만합니다.

저는 질문자님의 이해를 돕고자 지식공유자님의 기존 답변을 참고하여 유사한 답변 링크를 추천해드리고 있습니다.
현재 베타 기능으로 답변이 만족스럽지 않을 수 있는 점 양해 부탁드립니다. 🙏
추가적으로 궁금한 점이 있으시면, 이어서 질문해 주세요. 곧 지식공유자께서 답변해 주실 것입니다.

ycllyo님의 프로필 이미지

작성한 질문수

질문하기