{"componentChunkName":"component---src-templates-post-template-jsx","path":"/works/posts/2021-04-20--001","result":{"data":{"site":{"siteMetadata":{"title":"Blog by Eunyoung","subtitle":"작업 기록 블로그","copyright":"© All rights reserved.","author":{"name":"EunYoung","twitter":"#"},"disqusShortname":"","url":"https://ssongey.github.io"}},"markdownRemark":{"id":"08f36c42-65f1-5ae0-94c1-5bf641deaf94","html":"

요즘 FMS 프로젝트에 대한 리펙터링을 열심히 진행하고 있다.
\n그 중 가장 초점을 맞추고 있는 작업은 Request/Response 객체로 사용되고 있는 Entity를 Dto로 바꾸는 것이다.
\n그래서 Dto to Entity, Entity to Dto 의 편리성을 위해 Mapstruct 를 도입하였고, 큰 문제가 없이 수월한 작업을 예상했는데
\n역시나.. 모든 리펙토링은 많은 고민이 필요하구나를 다시한번 느끼게 된다.

\n

오늘 고민한 내용은 다음과 같다.
\n연관관계(FK)가 많은 Entity를 생성할때 모든 FK 객체는 어떻게 채울 것인가?
\nDTO가 가지고 있는 각 매핑 ID로 모두 조회를 해야하나?

\n
\n

일단, 간단하게 프로젝트를 구성하여 테스트를 했다.

\n\n
@NoArgsConstructor(access = AccessLevel.PRIVATE)\npublic class ReservationDto {\n\n    @Getter\n    @NoArgsConstructor\n    public static class CreateReq {\n        private Long customerId;\n        private PassengerType passengerType;\n        private String usages;\n        private String luggage;\n\n        //case1. customer id로 repository에서 select 한 값을 주입 받아 entity 에 주입\n        public ReservationPassenger toEntity(Customer customer) {\n            ReservationPassenger reservationPassenger = ReservationDtoMapper.INSTANCE.toReservationPassengerEntity(this);\n            reservationPassenger.setCustomer(customer);\n            return reservationPassenger;\n        }\n\n        //Case2. id로 customer id 객체 생성 후 entity 에 주입\n        public ReservationPassenger toEntity() {\n            Customer customer = new Customer(this.customerId);\n\n            ReservationPassenger reservationPassenger = ReservationDtoMapper.INSTANCE.toReservationPassengerEntity(this);\n            reservationPassenger.setCustomer(customer);\n            return reservationPassenger;\n        }\n\n    }\n}
\n

case1. customer id로 repository에서 select 한 값을 주입 받아 entity에 주입

\n
public ReservationPassenger toEntity(Customer customer) {\n    ReservationPassenger reservationPassenger = ReservationDtoMapper.INSTANCE.toReservationPassengerEntity(this);\n    reservationPassenger.setCustomer(customer);\n    return reservationPassenger;\n}\n\n// 호출 메소드 로직\npublic void createReservation(ReservationDto.CreateReq req) {\n    // customer id로 select\n    Customer customer = customerRepository.findById(req.getCustomerId()).orElseThrow(IllegalArgumentException::new);\n    ReservationPassenger reservationPassenger = req.toEntity(customer);\n    ReservationPassenger saved = reservationRepository.save(reservationPassenger);\n}
\n

그래.. 이게 정석이겠지,
\n그런데 Entity의 연관관계가 많으면? 저렇게 객체를 가져와서 매핑해야 하나?
\n실질적으로는 ID만 필요로 할 뿐인데?

\n

case2. customerId 필드만 넣어 Customer 객체 생성을 한 후 entity에 주입

\n
public ReservationPassenger toEntity(Customer customer) {\n    Customer customer = new Customer(this.customerId);\n\n    ReservationPassenger reservationPassenger = ReservationDtoMapper.INSTANCE.toReservationPassengerEntity(this);\n    reservationPassenger.setCustomer(customer);\n    return reservationPassenger;\n}\n\n// 호출 메소드 로직\npublic void createReservation(ReservationDto.CreateReq req) {\n    ReservationPassenger reservationPassenger = req.toEntity();\n    ReservationPassenger saved = reservationRepository.save(reservationPassenger);\n}
\n

헉! DB에는 문제없이 잘 매핑이 된다!
\n그럼 존재하지 않는 ID를 넣으면 어떻게 되지?

\n
\n

오~ 역시
\ncould not execute statement; SQL [n/a]; constraint 오류가 발생한다.
\n똑똑하구로..

\n
org.h2.jdbc.JdbcSQLIntegrityConstraintViolationException: \nReferential integrity constraint violation: "FKT1K88T0M3HKQGVYFI365S5KV8: \nPUBLIC.RESERVATION_PASSENGER FOREIGN KEY(CUSTOMER_ID) REFERENCES PUBLIC.CUSTOMER(ID) (2)"; SQL statement:
\n
\n

물론 매핑 ID의 검증을 위해 첫번째 방법이 당연하다는건 알지만,
\n매핑 ID의 검증이 필요 없는 경우는 두번째 방법을 써도 되지 않을까?

\n

좀 더 나은 방법이 없는지 더 고민 해봐야 겠다.

","fields":{"tagSlugs":["/tags/spring-boot/","/tags/dto/"],"slug":"/works/posts/2021-04-20--001"},"frontmatter":{"title":"[작업로그] Dto to Entity에 대한 고민","tags":["SpringBoot","DTO"],"date":"2021-04-20","description":""}}},"pageContext":{"slug":"/works/posts/2021-04-20--001"}},"staticQueryHashes":[]}