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

안녕하세요. asdkfur님 좋은 질문입니다.

1번 먼저 답을 해드리면

a,b,c 모두 정답입니다. 저는 개인적으로 model mapper를 선호하지 않지만, 주변에서 사용하는 것을 말리지는 않습니다^^

답을 드리면 생성자를 사용하는게 좋은가? 빌더를 사용하는게 좋은가? 별도의 생성자용 static 메서드를 사용하는게 좋은가? 아니면 별도의 생성 클래스를 만들어서 여기에서 생성해 주는게 좋은가? 등등 여러가지 상황이 있습니다.

결국 모두가 정답이기에 제가 선호하는 기준을 말씀드리면, 가장 단순한 방법부터 생각합니다. 만약 생성 로직이 정말 단순한 객체라면 그냥 생성자를 쓰는게 편합니다. 그런데 생성할 때 뭔가 비즈니스 로직이나 의미있는 생성자 메서드 명이 필요하다면 생성자용 static 메서드를 사용합니다. 만약 파라미터가 너무 많거나 복잡하면 빌더를 사용합니다. 이런식으로 상황에 맞는 방법을 잘 선택하는게 중요합니다. 결국 상황에 따라서 복잡도를 높이지 않는 선에서 단순한 방법을 선택하는 것이 좋은 방법이라 생각합니다.

2번도 같은 방법으로 선택하시면 됩니다. 파라미터로 넘기면 아무래도 재사용성이 좋아집니다. DTO를 넘기면 편하지만, 해당 엔티티가 DTO에 의존하게 됩니다. 그래도 파라미터가 너무 많으면 update용 dto를 만드는 것을 고려하는 것이 좋습니다.

3번은 null 체크를 하면서 아닌 것에 대한 업데이트를 해야합니다.

4번은 이번 강의에서 보여드린 모든 방법을 사용합니다. 1번의 답과 마찬가지로 model mapper를 사용하기도 하고, 직접 한땀한땀 필요한 값을 꺼내서 DTO로 변환하기도 합니다. 또는 JPQL에서 DTO로 바로 변환하기도 하구요. 딱 한가지만 조심하면 되는데, 절대 엔티티가 View용 DTO를 의존하면 안됩니다. 반대로 View용 DTO가 엔티티를 의존하는 것은 괜찮습니다.

다 장단점이 있어서 고민이 쉽지 않을 때가 있는데요. 저는 나름 다음과 같은 방법을 사용합니다.

1. 가장 쉬운 방법을 선택한다. (머리속으로 생각이 도는 것 보다 일단 가장 쉬운 방법으로 진행하는 것을 추천합니다.)

2. 직접 코드를 작성해본다. (머리속으로 생각이 도는 것 보다 일단 짜보면 쉽게 풀릴 때가 많습니다. 그리고 그 상황에 따라서 더 나은 코드가 있습니다.)

3. 의존관계를 생각한다. (의존관계가 복잡해지면 잘못된 코드입니다.)

도움이 되셨길 바라며^^ 감사합니다.

항상 진짜 친절한 답변 감사합니다. 정말 많이 배우고 있습니다.

김영한님 안녕하세요~ 항상 좋은 강의 감사드립니다.

Entity update 시 좋은 방법을 찾아보다가 여기까지 오게 되었습니다.

혹시 update 할 때 modelMapper를 쓰는 것은 어떻게 생각하시나요?

modelMapper.map(entityUpdateRequest, entity)

이런 식입니다.

한번도 이렇게 써본 적은 없는데 테스트를 했을 때 update 쿼리가 잘 날라가긴 했습니다.

혹시 다른 문제는 없을지 궁금합니다~

감사합니다!