3단계 - 리팩터링(매장 식사 주문)

README.md

@@ -51,47 +51,47 @@ docker compose -p kitchenpos up -d
 
 ### 주문 테이블
 
-- 주문 테이블을 등록할 수 있다.
-- 주문 테이블의 이름이 올바르지 않으면 등록할 수 없다.
-  - 주문 테이블의 이름은 비워 둘 수 없다.
-- 빈 테이블을 해지할 수 있다.
-- 빈 테이블로 설정할 수 있다.
-- 완료되지 않은 주문이 있는 주문 테이블은 빈 테이블로 설정할 수 없다.
-- 방문한 손님 수를 변경할 수 있다.
-- 방문한 손님 수가 올바르지 않으면 변경할 수 없다.
-  - 방문한 손님 수는 0 이상이어야 한다.
-- 빈 테이블은 방문한 손님 수를 변경할 수 없다.
-- 주문 테이블의 목록을 조회할 수 있다.
+- [X] 주문 테이블을 등록할 수 있다.
+- [X] 주문 테이블의 이름이 올바르지 않으면 등록할 수 없다.
+  - [X] 주문 테이블의 이름은 비워 둘 수 없다.
+- [X] 빈 테이블을 해지할 수 있다.
+- [X] 빈 테이블로 설정할 수 있다.
+- [] 완료되지 않은 주문이 있는 주문 테이블은 빈 테이블로 설정할 수 없다.
+- [X] 방문한 손님 수를 변경할 수 있다.
+- [X] 방문한 손님 수가 올바르지 않으면 변경할 수 없다.
+  - [X] 방문한 손님 수는 0 이상이어야 한다.
+- [X] 빈 테이블은 방문한 손님 수를 변경할 수 없다.
+- [] 주문 테이블의 목록을 조회할 수 있다.
 
 ### 주문
 
 - 1개 이상의 등록된 메뉴로 배달 주문을 등록할 수 있다.
 - 1개 이상의 등록된 메뉴로 포장 주문을 등록할 수 있다.
-- 1개 이상의 등록된 메뉴로 매장 주문을 등록할 수 있다.
-- 주문 유형이 올바르지 않으면 등록할 수 없다.
-- 메뉴가 없으면 등록할 수 없다.
-- 매장 주문은 주문 항목의 수량이 0 미만일 수 있다.
-- 매장 주문을 제외한 주문의 경우 주문 항목의 수량은 0 이상이어야 한다.
+- [] 1개 이상의 등록된 메뉴로 매장 주문을 등록할 수 있다.
+-  [] 주문 유형이 올바르지 않으면 등록할 수 없다.
+- [X] 메뉴가 없으면 등록할 수 없다.
+- [X] 매장 주문은 주문 항목의 수량이 0 미만일 수 있다.
+-  매장 주문을 제외한 주문의 경우 주문 항목의 수량은 0 이상이어야 한다.
 - 배달 주소가 올바르지 않으면 배달 주문을 등록할 수 없다.
   - 배달 주소는 비워 둘 수 없다.
-- 빈 테이블에는 매장 주문을 등록할 수 없다.
-- 숨겨진 메뉴는 주문할 수 없다.
-- 주문한 메뉴의 가격은 실제 메뉴 가격과 일치해야 한다.
-- 주문을 접수한다.
-- 접수 대기 중인 주문만 접수할 수 있다.
+- [X] 빈 테이블에는 매장 주문을 등록할 수 없다.
+- [X] 숨겨진 메뉴는 주문할 수 없다.
+- [] 주문한 메뉴의 가격은 실제 메뉴 가격과 일치해야 한다.
+- [X] 주문을 접수한다.
+- [X] 접수 대기 중인 주문만 접수할 수 있다.
 - 배달 주문을 접수되면 배달 대행사를 호출한다.
-- 주문을 서빙한다.
-- 접수된 주문만 서빙할 수 있다.
+- [X] 주문을 서빙한다.
+- [X] 접수된 주문만 서빙할 수 있다.
 - 주문을 배달한다.
 - 배달 주문만 배달할 수 있다.
 - 서빙된 주문만 배달할 수 있다.
 - 주문을 배달 완료한다.
 - 배달 중인 주문만 배달 완료할 수 있다.
-- 주문을 완료한다.
+- [X] 주문을 완료한다.
 - 배달 주문의 경우 배달 완료된 주문만 완료할 수 있다.
-- 포장 및 매장 주문의 경우 서빙된 주문만 완료할 수 있다.
-- 주문 테이블의 모든 매장 주문이 완료되면 빈 테이블로 설정한다.
-- 완료되지 않은 매장 주문이 있는 주문 테이블은 빈 테이블로 설정하지 않는다.
+- [X] 포장 및 매장 주문의 경우 서빙된 주문만 완료할 수 있다.
+- [X] 주문 테이블의 모든 매장 주문이 완료되면 빈 테이블로 설정한다.
+- [X] 완료되지 않은 매장 주문이 있는 주문 테이블은 빈 테이블로 설정하지 않는다.
 - 주문 목록을 조회할 수 있다.
 
 ## 용어 사전
@@ -176,14 +176,14 @@ docker compose -p kitchenpos up -d
 
 ### 매장 주문
 
-- `OrderTable`은 식별자와 이름, `NumberOfGuests`를 가진다.
-- `OrderTable`의 추가 `Order`는 `OrderTable`에 계속 쌓이며 모든 `Order`가 완료되면 `EmptyTable`이 된다.
-- `EmptyTable`인 경우 `NumberOfGuests`는 0이며 변경할 수 없다.
-- `Order`는 식별자와 `OrderStatus`, 주문 시간, `OrderLineItems`를 가진다.
-- 메뉴가 노출되고 있으며 판매되는 메뉴 가격과 일치하면 `Order`가 생성된다.
-- `Order`는 접수 대기 ➜ 접수 ➜ 서빙 ➜ 계산 완료 순서로 진행된다.
-- `OrderLineItem`는 가격과 수량을 가진다.
-- `OrderLineItem`의 수량은 기존 `Order`를 취소하거나 변경해도 수정되지 않기 때문에 0보다 적을 수 있다.
+- [X] `OrderTable`은 식별자와 이름, `NumberOfGuests`를 가진다.
+- [] `OrderTable`의 추가 `Order`는 `OrderTable`에 계속 쌓이며 모든 `Order`가 완료되면 `EmptyTable`이 된다.
+- [X] `EmptyTable`인 경우 `NumberOfGuests`는 0이며 변경할 수 없다.
+- [X] `Order`는 식별자와 `OrderStatus`, 주문 시간, `OrderLineItems`를 가진다.
+- [] 메뉴가 노출되고 있으며 판매되는 메뉴 가격과 일치하면 `Order`가 생성된다.
+- [X] `Order`는 접수 대기 ➜ 접수 ➜ 서빙 ➜ 계산 완료 순서로 진행된다.
+- [] `OrderLineItem`는 가격과 수량을 가진다.
+- [] `OrderLineItem`의 수량은 기존 `Order`를 취소하거나 변경해도 수정되지 않기 때문에 0보다 적을 수 있다.
 
 ### 배달 주문
 

src/main/java/kitchenpos/eatinorders/tobe/application/acl/EatInOrderServiceAdapter.java

@@ -0,0 +1,56 @@
+package kitchenpos.eatinorders.tobe.application.acl;
+
+import kitchenpos.eatinorders.tobe.domain.entity.EatInOrder;
+import kitchenpos.eatinorders.tobe.domain.entity.OrderTable;
+import kitchenpos.eatinorders.tobe.domain.repository.EatInOrderRepository;
+import kitchenpos.eatinorders.tobe.domain.repository.OrderTableRepository;
+import kitchenpos.menus.tobe.domain.entity.Menu;
+import kitchenpos.menus.tobe.domain.repository.MenuRepository;
+
+import java.util.NoSuchElementException;
+import java.util.UUID;
+
+public class EatInOrderServiceAdapter {
+    private final EatInOrderRepository orderRepository;
+    private final OrderTableRepository orderTableRepository;
+    private final MenuRepository menuRepository;
+
+    public EatInOrderServiceAdapter(EatInOrderRepository orderRepository,
+                                    OrderTableRepository orderTableRepository,
+                                    MenuRepository menuRepository
+    ) {
+        this.orderRepository = orderRepository;
+        this.orderTableRepository = orderTableRepository;
+        this.menuRepository = menuRepository;
+    }
+
+    public boolean existHideMenu(UUID menuId) {
+        Menu menu = findMenuById(menuId);
+        if (menu.isNotDisplayed()) {
+            return true;
+        }
+        return false;
+    }
+
+    private Menu findMenuById(UUID menuId) {
+        return menuRepository.findById(menuId)
+                .orElseThrow(() -> new NoSuchElementException());
+    }
+
+    public boolean existIsNotOccupiedOrderTable(EatInOrder order) {
+        OrderTable orderTable = findOrderTableBy(order.getOrderTableId());
+        return orderTable.isNotOccupied();
+    }
+
+    public void clearOrderTable(EatInOrder order) {
+        OrderTable orderTable = findOrderTableBy(order.getOrderTableId());
+        if (orderRepository.isAllCompleteByOrderTable(orderTable)) {
+            orderTable.clear();
+        }
+    }
+
+    private OrderTable findOrderTableBy(UUID id) {
+        return orderTableRepository.findBy(id)
+                .orElseThrow(() -> new NoSuchElementException());
+    }
+}

src/main/java/kitchenpos/eatinorders/tobe/domain/constant/EatInOrderStatus.java

@@ -0,0 +1,5 @@
+package kitchenpos.eatinorders.tobe.domain.constant;
+
+public enum EatInOrderStatus {
+    WAITING, ACCEPTED, SERVED, COMPLETED
+}

src/main/java/kitchenpos/eatinorders/tobe/domain/constant/EatInOrderType.java

@@ -0,0 +1,5 @@
+package kitchenpos.eatinorders.tobe.domain.constant;
+
+public enum EatInOrderType {
+    EAT_IN
+}

src/main/java/kitchenpos/eatinorders/tobe/domain/constant/OrderTableStatus.java

@@ -0,0 +1,6 @@
+package kitchenpos.eatinorders.tobe.domain.constant;
+
+public enum OrderTableStatus {
+    EMPTY_TABLE,
+    SIT_TABLE
+}

src/main/java/kitchenpos/eatinorders/tobe/domain/entity/EatInOrder.java

@@ -0,0 +1,102 @@
+package kitchenpos.eatinorders.tobe.domain.entity;
+
+import jakarta.persistence.*;
+import kitchenpos.eatinorders.tobe.domain.constant.EatInOrderStatus;
+import kitchenpos.eatinorders.tobe.domain.constant.EatInOrderType;
+import kitchenpos.eatinorders.tobe.application.acl.EatInOrderServiceAdapter;
+
+import java.time.LocalDateTime;
+import java.util.UUID;
+
+@Table(name = "orders")
+@Entity
+public class EatInOrder {
+    @Column(name = "id", columnDefinition = "binary(16)")
+    @Id
+    private UUID id;
+
+    @Column(name = "type", nullable = false, columnDefinition = "varchar(255)")
+    @Enumerated(EnumType.STRING)
+    private EatInOrderType type;
+
+    @Column(name = "status", nullable = false, columnDefinition = "varchar(255)")
+    @Enumerated(EnumType.STRING)
+    private EatInOrderStatus status;
+
+    @Column(name = "order_date_time", nullable = false)
+    private LocalDateTime orderDateTime;
+
+    @Embedded
+    private OrderLineItems orderLineItems;
+
+    @Transient
+    private UUID orderTableId;
+
+    public EatInOrder(UUID id, EatInOrderType type, EatInOrderStatus status, LocalDateTime orderDateTime,
+                      OrderLineItems orderLineItems, UUID orderTableId,
+                      EatInOrderServiceAdapter orderDomainService) {
+        this(id, type, status, orderDateTime, orderLineItems, orderTableId);
+        checkOrderTableOccupied(orderDomainService);
+        checkAllMenuIsDisplayed(orderDomainService);
+    }
+
+    public EatInOrder(UUID id, EatInOrderType type, EatInOrderStatus status, LocalDateTime orderDateTime,
+                      OrderLineItems orderLineItems, UUID orderTableId) {
+        this.id = id;
+        this.type = type;
+        this.status = status;
+        this.orderDateTime = orderDateTime;
+        this.orderLineItems = orderLineItems;
+        this.orderTableId = orderTableId;
+    }
+
+    private void checkAllMenuIsDisplayed(EatInOrderServiceAdapter orderDomainService) {
+        if (orderLineItems.hasDisplayedMenu(orderDomainService)) {
+            throw new IllegalStateException();
+        }
+    }
+
+    private void checkOrderTableOccupied(EatInOrderServiceAdapter orderDomainService) {
+        if (orderDomainService.existIsNotOccupiedOrderTable(this)) {
+            throw new IllegalStateException();
+        }
+    }
+
+    public void accept() {
+        if (status != EatInOrderStatus.WAITING) {
+            throw new IllegalStateException();
+        }
+        status = EatInOrderStatus.ACCEPTED;
+    }
+
+    public void serve() {
+        if (status != EatInOrderStatus.ACCEPTED) {
+            throw new IllegalStateException();
+        }
+        status = EatInOrderStatus.SERVED;
+    }
+
+    public void complete(EatInOrderServiceAdapter eatInOrderServiceAdapter) {
+        if (status != EatInOrderStatus.SERVED) {
+            throw new IllegalStateException();
+        }
+        status = EatInOrderStatus.COMPLETED;
+        eatInOrderServiceAdapter.clearOrderTable(this);
+    }
+
+    public UUID getId() {
+        return id;
+    }
+
+    public EatInOrderType getType() {
+        return type;
+    }
+
+    public EatInOrderStatus getStatus() {
+        return status;
+    }
+
+    public UUID getOrderTableId() {
+        return orderTableId;
+    }
+}

src/main/java/kitchenpos/eatinorders/tobe/domain/entity/OrderLineItem.java

@@ -0,0 +1,53 @@
+package kitchenpos.eatinorders.tobe.domain.entity;
+
+import jakarta.persistence.*;
+
+import java.math.BigDecimal;
+import java.util.UUID;
+
+@Table(name = "order_line_item")
+@Entity
+public class OrderLineItem {
+    @Column(name = "seq")
+    @GeneratedValue(strategy = GenerationType.IDENTITY)
+    @Id
+    private Long seq;
+
+    @Column(name = "quantity", nullable = false)
+    private long quantity;
+
+    @Transient
+    private BigDecimal price;
+
+    @Transient
+    private UUID menuId;
+
+    protected OrderLineItem() {}
+
+    public OrderLineItem(Long seq, long quantity, BigDecimal price, UUID menuId) {
+        this(seq, quantity, price);
+        this.menuId = menuId;
+    }
+
+    public OrderLineItem(Long seq, long quantity, BigDecimal price) {
+        this.seq = seq;
+        this.quantity = quantity;
+        this.price = price;
+    }
+
+    public Long getSeq() {
+        return seq;
+    }
+
+    public long getQuantity() {
+        return quantity;
+    }
+
+    public BigDecimal getPrice() {
+        return price;
+    }
+
+    public UUID getMenuId() {
+        return menuId;
+    }
+}

src/main/java/kitchenpos/eatinorders/tobe/domain/entity/OrderLineItems.java

@@ -0,0 +1,30 @@
+package kitchenpos.eatinorders.tobe.domain.entity;
+
+import jakarta.persistence.*;
+import kitchenpos.eatinorders.tobe.application.acl.EatInOrderServiceAdapter;
+
+import java.util.List;
+
+@Embeddable
+public class OrderLineItems {
+    @OneToMany(cascade = {CascadeType.PERSIST, CascadeType.MERGE})
+    @JoinColumn(
+            name = "order_id",
+            nullable = false,
+            columnDefinition = "binary(16)",
+            foreignKey = @ForeignKey(name = "fk_order_line_item_to_orders")
+    )
+    private List<OrderLineItem> orderLineItems;
+
+    protected OrderLineItems() {}
+
+    public boolean hasDisplayedMenu(EatInOrderServiceAdapter orderDomainService) {
+        return orderLineItems.stream()
+                .map(orderLineItem -> orderLineItem.getMenuId())
+                .anyMatch(menuId -> orderDomainService.existHideMenu(menuId));
+    }
+
+    public OrderLineItems(List<OrderLineItem> orderLineItems) {
+        this.orderLineItems = orderLineItems;
+    }
+}