update 메서드의 매개변수 작성 스타일?
// 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를 만들고, 이를 사용하는 것을 추천합니다 🙂
감사합니다.
0
안녕하세요, 인프런 AI 인턴입니다.
매개변수의 개수가 많을 때, DTO(Data Transfer Object)를 사용하는 방식(b 방식)과 각 매개변수를 직접 전달하는 방식(a 방식) 사이에서 선택해야 하는 상황은 개발을 하다 보면 자주 마주치게 됩니다. 여러분이 제시한 두 가지 사례에 관해서는, 정석적이라고 할 수 있는 ‘하나의 올바른 답’이 존재하지는 않습니다. 대신, 각 방식의 장단점과 상황에 맞는 사용법을 이해하는 것이 중요합니다.
a 방식은 간단한 경우에 적합하며, 메서드 시그니처를 통해 수정되는 필드가 무엇인지 명확하게 알 수 있는 장점이 있습니다. 하지만 매개변수가 많아지면 메서드 호출 시 인자의 순서를 혼동하기 쉽고, 메서드 시그니처도 복잡해집니다.
b 방식은 매개변수의 개수가 많거나, 파라미터의 값 중 일부만 종종 변경되는 경우 유용합니다. DTO를 사용하면 필드의 추가나 제거가 발생하여도 메서드 시그니처는 변경되지 않으므로, 코드의 유지 보수성이 향상됩니다. 또한, 해당 DTO 내에서 데이터의 검증 로직 등을 추가할 수 있어 유효성 검사를 간결하게 관리할 수 있는 이점이 있습니다. 하지만 엔터티 클래스와 DTO 간에 추가적인 매핑 과정이 필요하다는 단점이 있습니다.
실제 질문에서 언급한 ‘정석적인 방식이 있다면’에 대한 답변으로는, 상황에 따라 가장 적합한 방법을 선택하는 것입니다. 수정해야 할 필드가 많고, 특정 상황에서만 일부 필드가 업데이트 되는 복잡한 경우라면 DTO를 사용하는 방식을, 간단하게 몇 개의 필드만 변경되는 경우라면 직접 매개변수를 넘기는 방식을 선택할 수 있겠습니다.
질문하신 내용과 유사한 문제를 다룬 다음 질문 드립니다.에서는 매개변수를 확인하고 업데이트하는 로직을 엔터티 내부에 두는 것에 대해 설명하고 있습니다. 해당 답변에는 업데이트 로직이 엔티티 내부에 두는 것이 적절할지, 서비스 레이어에 구현하는 것이 낫겠는지에 대한 토론이 포함되어 있습니다. 여기서는 서비스 레이어보다는 엔티티 내부에 업데이트 로직을 두는 것을 권장하고 있으며, 필요에 따른 유효성 검사나 데이터 검증 로직도 함께 고려할 수 있다는 점에서 관련지어 생각해볼 만합니다.
저는 질문자님의 이해를 돕고자 지식공유자님의 기존 답변을 참고하여 유사한 답변 링크를 추천해드리고 있습니다.
현재 베타 기능으로 답변이 만족스럽지 않을 수 있는 점 양해 부탁드립니다. 🙏
추가적으로 궁금한 점이 있으시면, 이어서 질문해 주세요. 곧 지식공유자께서 답변해 주실 것입니다.
OrderServiceTest 상문주문 테스트 시 update 쿼리 문의
0
27
1
sdk 설정 오류
0
68
2
오탈자 - @Transactional
0
63
1
src/test/resources 테스트 경로 문제
0
60
1
상품 등록후 H2 db 출력 순서 바꿀 수 있나요?
0
70
1
MemberRepositoryTest 실행오류
0
90
1
boot 4.x >>> trasasction rolled back log & p6spy(영한님, 수업 자료 업데이트 해주시면 감사하겠습니다!!)
1
196
2
강의 마지막 QueryDSL 사용 부분 질문있습니다
1
154
2
클라이언트에서 isbn과 author 수정 요청을 한 경우에 대해 질문드립니다.
0
56
1
도메인 모델 패턴 vs 트랜잭션 스크립트 패턴
0
80
1
기본 생성자
0
67
1
h2 DB 연결시 jdbc url 변경 이유가 궁금합니다.
0
107
1
멤버서비스테스트 부분에서 막힙니다.
0
174
4
실무에서도 EntityManager를 이용해서 많이 작업하는 편일까요?
0
124
1
초반에 h2 다운로드 과정 꼭 필요한가요?
0
129
2
자신 필드에도 get으로 접근하는 이유가 있을까요?
0
122
1
24분 27초 연관관계 편의 메서드 위치
0
118
1
단건 주문만 가능하게 한건 의도한 부분이신가요?
0
114
2
빌드 툴, Gradle
0
65
1
h2연결은 된 것 같은데 엔티티 테이블까지 작성 후 확인해보아도 테이블이 안보입니다
0
82
2
Repository에서 EntityManager 주입 방식 차이
0
97
1
롬복과 사용자 정의 setter 메서드
0
78
1
주문 목록 조회 fetch join 질문드립니다
0
91
1
dirty checking 질문드립니다.
0
87
1