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

생성 메서드(정적팩토리)를 만드는 도중 

.setMember없이 (.changeMember같은 것 포함) 

기본 생성자 외에 다른 생성자 없이 

다음과 같이 접근이 가능하길래 이렇게 써도 되나 싶어 질문드립니다.

public static createOrder(Member member, Delivery delivery, .. ) {

Order order = new Order();

order.member = member;

..

]

처럼 세터(order.setMember)나 생성자(Order order = new Order(member , ...)) 없이 

order.member로 그대로 값에 접근해도 test시 사용할 때 문제가 없는 것 같아 질문 드립니다.

이렇게 써도 되는 건가요 ?

 

오더 클래스 Order.class : 

@Entity
@Table(name = "orders")
@Getter
@NoArgsConstructor(access = AccessLevel.PROTECTED)
public class Order  {
    @Id @GeneratedValue
    @Column(name = "order_id")
    private Long id;

    @ManyToOne(fetch = FetchType.LAZY)
    @JoinColumn(name = "member_id")
    private Member member;

    @OneToMany(mappedBy = "order", cascade = CascadeType.ALL)
    private List<OrderItem> orderItems = new ArrayList<>();

    @OneToOne(fetch = FetchType.LAZY, cascade = CascadeType.ALL)
    @JoinColumn(name = "delivery_id")
    private Delivery delivery;

    @Enumerated(EnumType.STRING)
    private OrderStatus orderStatus;

    //연관관계 편의 메서드
    public void addOrderItem(OrderItem orderItem) {
        this.orderItems.add(orderItem);
        orderItem.changeOrder(this);
    }

    //생성 메서드
    public static Order createOrder(Member member, Delivery delivery, OrderItem... orderItem) {
        Order order = new Order();
        order.member = member;
        order.delivery = delivery;

        for (OrderItem item : orderItem) {
            order.addOrderItem(item);
        }

        order.orderStatus = OrderStatus.결제완료;
        return order;
    }

}

성공 테스트 :

@Test
@Commit
public void test2(){
    Member memberA = new Member("memberG", 1234);
    em.persist(memberA);
    Item item = new Item("itemA", 1000, "Cats TowerAA", "");
    em.persist(item);

    Address address = new Address("seoul", "12345");
    Delivery delivery = new Delivery(address);
    OrderItem orderItem = OrderItem.createOrderItem(item, 900, 1);
    Order order = Order.createOrder(memberA, delivery, orderItem);
    em.persist(order);
    System.out.println("order = " + order);

}