인프런 커뮤니티 질문&답변
Setter 대신 생성자를 사용하는 방법에 대해 궁금합니다
작성
·
452
0
안녕하세요 영한님.
프로젝트 내에서 OrderService 에서 Order를 생성하려고 합니다.
강의 내용에서는 @Setter를 사용하셔서 하셨는데
중간 Order.java 내에서 연관관계 메서드를 만들어 놓은 부분 중
setter가 아닌 생성자로 넣을 수 있는 방법이 어떤게 있을지 고민해보았는데
도통 생각이 나지 않아 문의드려봅니다!
OrderService.java
@Transactional
public Long order(Long memberId, Long itemId, int count) {
//엔터티 조회
Member member = memberRepository.findOne(memberId);
Item item = itemRepository.findOne(itemId);
//배송정보 생성
Delivery delivery = new Delivery();
delivery.setAddress(member.getAddress());
//주문상품 생성
OrderItem orderItem = OrderItem.createOrderItem(item, item.getPrice(), count);
//주문 생성
Order order = Order.createOrder(member, delivery, orderItem);
//주문 저장
orderRepository.save(order);
return order.getId();
}
Order.java
public static Order createOrder(Member member, Delivery delivery, OrderItem... orderItems) {
Order order = new Order();
order.setMember(member);
order.setDelivery(delivery);
for (OrderItem orderItem : orderItems) {
order.addOrderItem(orderItem);
}
order.setStatus(OrderStatus.ORDER);
order.setOrderDate(LocalDateTime.now());
return order;
}
같으 클래스 내에
public void addOrderItem(OrderItem orderItem) {
orderItems.add(orderItem);
orderItem.setOrder((this));
/* OrderItem orderItem2 = new OrderItem(this)*/
}
저 주석 부분 처럼 orderItem 객체를 새로 생성해서 사용하면 다른것으로 판별이 되어 DB에 Order가 들어가지 않는것 같은데...
어떤 방법이 제일 좋을까요?
아니면 그냥 OrderItem 이 엔터티에는 그냥 Setter를 사용하는게 합리적일까요??
감사합니다!
답변 1
0
김영한
지식공유자
안녕하세요. KwangMin Lee님
createOrder를 호출할 때 OrderItem을 생성하기 위한 파라미터를 다 전달하면 될 것 같아요.
item, item.getPrice(), count
그리고 이 값으로 OrderItem을 내부에서 생성하고, 생성자를 호출하면서 this를 넘기면 될 듯요.
감사합니다.





