DTO to JPA Entity

Question

안녕하세요 스프링부트에서 Controller에서 받은 DTO를JPA Entity로 변환하려고 할때여러가지 방법이 있을텐데예를들면setter, 생성자, builder, mapstruct, objectmapper등등만약 업데이트의 API를 만들때다른분들은 어떻게 작업을 하시나요?예를들면 저는 UserEntity에 private Long id;private String name;private String zipCode;private String addr1;public void modifyAddress(AddressDto dto){ this.zipCode &#61; dto.getZipCode(); this.addr1 &#61; dto.getAddr1(); }이런식으로 변경메소드를 활용해서 업데이트를 하는데이런방법은 잘못되었다는 의견이 있어서여러 사람들의 의견을 듣고싶습니다.

Clickin · Answer

AddressDto라는 명칭으로 미루어봐서 주소 관련 update 컨트롤러에서 넘어올거로 보이는데요만약 개인정보 변경을 한번에 처리하는 컨트롤러를 만들어야한다고 가정하면 컨트롤러에서 DTO를 쪼개던가 아니면 modifyAddress(UserInfoDto dto) 이런 메소드 오버라이딩을 또 만들어야겠죠.어떤 방식으로 setter를 쓰면 좋을지는 다음 글을 참고해보세요 https://okky.kr/questions/1214302

신입개발자어따쓸까요 · Answer

변경이 아닌 새로운 인스턴스를 반환하는 메소드로 만드시는게 좋을듯하네요. 참고로 이런 Dto ↔︎Entity 변환시에 사용하기 좋은 MapStruct라는게 있습니다.

wharup · Answer

Controller를 통해 외부로 노출되는 DTO를 내부에서만 관리하는 Entity로 변환할 때는 의존 관계가 중요합니다. 이 경우 내부의 Entity가 외부로 노출하는 DTO를 참조하는건 좋지 않고 DTO, Service 등이 DTO를 Entity로 변환하는게 좋습니다.그런데 보여주신 예제는 좀 다른 케이스입니다. AddressDto를 Address Entity로 1:1 변환하는게 아니고, 더 큰 범위인 User Entity의 일부를 반영하는거니까요. 따라서 오브젝트를 1:1로 변환하는 mapstruct, objectmapper 등이 관련이 없다고 봐야겠네요. 오히려 Service가 Entity의 우편번호와 상세 주소를 하나씩 업데이트할지, Entity가 주소를 업데이트하는 메소드를 제공하는게 맞을지에 대한 이야기겠네요.JPA Entity를 비즈니스 로직을 갖는 객체로 보고 주소 변경이라는 로직을 캠슐화하겠다면 modifyAddress 메소드가 우편번호와 상세주소를 받아서 내부에서 변경하는게 맞습니다. 반면에 JPA Entity를 단순하게 값을 저장하는 걸로 보면 Service각 필드를 업데이트하는게 어울리지요. 요건 헷갈릴만한 일은 아니고 여기에 적용하는 프로그램 스타일이라고 봐야겠지요.참고로 JPA Entity가 로직을 갖는 객체로 modifyAddress 메소드를 제공한다면 Controller로 노출하는 AddressDto가 파라메터가 되면안됩니다. modifyAddress 메소드는 AddressUpdateCommand 갖은 자신이 제공하는 DTO를 같은 레이어에 두고 있고, service 등이 Controller의 AddredsDto를 AddressUpdateCommand로 변환해서 modifyAddress 메소드를 호출해야 합니다.

saIIy · Answer

entity에 컬럼이 많아도 결국 update가 필요한 컬럼은 그렇게 많지 않은게 일반적이니 entity 내에 update 메서드를 만드는게 좋다고 봅니다. 더 FM적인 이론이 있겠지만 사실상 과히다고 생각하고 코드에 유연성을 생각하면 entity 내에 선언하고 파라미터는 update할 대상의 값 하나만 받는게 좋다고 생각합니다.

장독깨기 · Answer

그게 도대체 왜 뭐가 잘 못 됐다는 걸까요. ㅎ주소, 우편번호를 입력받아 변경하려는거죠.그걸 각각 변경하나 한번에 변경하나 또 다른 방법을 쓰나 도대체 뭔 차이가 있나요.세터를 함부로 쓰지 마라는 것은, 다른 필드 또는 특정 행위(메서드)에 영향을 줄 때 얘깁니다.겨우, 주소, 우편번호를 그냥 업데이트 하는 겁니다. 아무런 영향이 없어요.별 쓰잘데기 없는 것에 힘 쓰면서 정답 찾으려고 시간 허비 하지 마세요. 정답 없구요, 중요한 것에 집중하세요. :)

집킨 · Answer

dto가 presentation layer에 위치한다면 잘못된게 맞습니다1차원적으로 DDD에 부합하게 하려면 modifyAddress 메소드의 파라미터에 모든 주소 정보를 각각 넘겨주는게 맞지만, 파라미터의 갯수가 많거나 많아질것으로 예상해서 클래스로 만든다면 domain layer에 위치시켜야 합니다만약 presentation layer에 dto가 있다면 그 dto를 domain layer의 dto로 변환하는 과정도 application 이나 presentation layer에 작성해야하구요물론 이건 엄격한 방법론이긴 해요

DTO to JPA Entity

6개의 답변

Q&A 목록