주의점 메모장

Setter는 닫아놓자

키스샷1104 2023. 11. 3. 21:29

1.

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;
    }
  1. public static OrderItem createOrderItem(Item item, int orderPrice, int count) {
         OrderItem orderItem = new OrderItem();
         orderItem.setItem(item);
         orderItem.setOrderPrice(orderPrice);
         orderItem.setCount(count);
    
         item.removeStock(count);
         return orderItem;
     }

setItem(), setMember() 등등같이 이 값을 이용하여 초기화 하지 말고
이런식으로 생성 메서드를 만들어서 사용하는것이 좋다.