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

README.md

@@ -127,18 +127,19 @@ docker compose -p kitchenpos up -d
 
 ### 매장 주문
 
-| 한글명      | 영문명              | 설명                                                 |
-|----------|------------------|----------------------------------------------------|
+| 한글명     | 영문명              | 설명                                                 |
+|---------|------------------|----------------------------------------------------|
 | 방문한 손님 수 | number of guests | 식기가 필요한 사람 수. 필수 사항은 아니며 주문은 0명으로 등록할 수 있다.        |
-| 빈 테이블    | empty table      | 주문을 등록할 수 없는 주문 테이블                                |
-| 서빙       | served           | 조리가 완료되어 음식이 나갈 수 있는 단계                            |
-| 완료       | completed        | 고객이 모든 식사를 마치고 결제를 완료한 단계                          |
-| 접수       | accepted         | 주문을 받고 음식을 조리하는 단계                                 |
-| 접수 대기    | waiting          | 주문이 생성되어 매장으로 전달된 단계                               |
-| 주문       | order            | 매장에서 식사하는 고객 대상. 손님들이 매장에서 먹을 수 있도록 조리된 음식을 가져다준다. |
-| 주문 상태    | order status     | 주문이 생성되면 매장에서 주문을 접수하고 고객이 음식을 받기까지의 단계를 표시한다.     |
-| 주문 테이블   | order table      | 매장에서 주문이 발생하는 영역                                   |
-| 주문 항목    | order line item  | 주문에 속하는 수량이 있는 메뉴                                  |
+| 빈 테이블   | empty table      | 주문을 등록할 수 없는 주문 테이블                                |
+| 서빙완료      | served           | 조리가 완료되어 음식이 나갈 수 있는 단계                            |
+| 매장주문완료      | completed        | 고객이 모든 식사를 마치고 결제를 완료한 단계                          |
+| 접수완료    | accepted         | 주문을 받고 음식을 조리하는 단계                                 |
+| 접수 대기   | waiting          | 주문이 생성되어 매장으로 전달된 단계                               |
+| 주문      | eatInOrder            | 매장에서 식사하는 고객 대상. 손님들이 매장에서 먹을 수 있도록 조리된 음식을 가져다준다. |
+| 주문 상태   | order status     | 주문이 생성되면 매장에서 주문을 접수하고 고객이 음식을 받기까지의 단계를 표시한다.     |
+| 주문 테이블  | order table      | 매장에서 주문이 발생하는 영역                                   |
+| 주문 항목   | order line item  | 주문에 속하는 수량이 있는 메뉴                                  |
+
 
 ### 배달 주문
 
@@ -194,11 +195,12 @@ docker compose -p kitchenpos up -d
 - `OrderTable`은 식별자와 이름, `NumberOfGuests`를 가진다.
 - `OrderTable`의 추가 `Order`는 `OrderTable`에 계속 쌓이며 모든 `Order`가 완료되면 `EmptyTable`이 된다.
 - `EmptyTable`인 경우 `NumberOfGuests`는 0이며 변경할 수 없다.
-- `Order`는 식별자와 `OrderStatus`, 주문 시간, `OrderLineItems`를 가진다.
-- 메뉴가 노출되고 있으며 판매되는 메뉴 가격과 일치하면 `Order`가 생성된다.
-- `Order`는 접수 대기 ➜ 접수 ➜ 서빙 ➜ 계산 완료 순서로 진행된다.
+- `EatInOrder`는 식별자와 `OrderStatus`, 주문 시간, `OrderLineItems`를 가진다.
+- 메뉴가 노출되고 있으며 판매되는 메뉴 가격과 일치하면 `EatInOrder`가 생성된다.
+- `EatInOrder`는 접수 대기 ➜ 접수 ➜ 서빙 ➜ 계산 완료 순서로 진행된다.
 - `OrderLineItem`는 가격과 수량을 가진다.
-- `OrderLineItem`의 수량은 기존 `Order`를 취소하거나 변경해도 수정되지 않기 때문에 0보다 적을 수 있다.
+- `OrderLineItem`의 수량은 기존 `EatInOrder`를 취소하거나 변경해도 수정되지 않기 때문에 0보다 적을 수 있다.
+
 
 ### 배달 주문
 

src/main/java/kitchenpos/eatinorders/adapter/eatinorder/out/JpaEatinOrderRepository.java

@@ -0,0 +1,11 @@
+package kitchenpos.eatinorders.adapter.eatinorder.out;
+
+import java.util.UUID;
+import kitchenpos.eatinorders.application.eatinorder.port.out.EatinOrderRepository;
+import kitchenpos.eatinorders.domain.eatinorder.EatinOrder;
+import org.springframework.data.jpa.repository.JpaRepository;
+
+public interface JpaEatinOrderRepository extends EatinOrderRepository,
+    JpaRepository<EatinOrder, UUID> {
+
+}

src/main/java/kitchenpos/eatinorders/adapter/eatinorder/out/ValidMenuAdapter.java

@@ -0,0 +1,26 @@
+package kitchenpos.eatinorders.adapter.eatinorder.out;
+
+import java.util.List;
+import java.util.UUID;
+import kitchenpos.eatinorders.application.eatinorder.port.out.ValidMenuPort;
+import kitchenpos.menu.application.menu.port.in.MenuFindUseCase;
+
+public class ValidMenuAdapter implements ValidMenuPort {
+
+    private final MenuFindUseCase menuFindUseCase;
+
+    public ValidMenuAdapter(final MenuFindUseCase menuFindUseCase) {
+        this.menuFindUseCase = menuFindUseCase;
+    }
+
+    @Override
+    public void checkValidMenu(final List<UUID> menuIds) {
+        if (menuIds.size() != menuFindUseCase.findAll()
+            .stream()
+            .filter(menu -> menuIds.contains(menu.getId()))
+            .count()) {
+
+            throw new IllegalStateException(String.format("menu is not exist. ids: %s", menuIds));
+        }
+    }
+}

src/main/java/kitchenpos/eatinorders/adapter/ordertable/out/JpaOrderTableRepository.java

@@ -0,0 +1,11 @@
+package kitchenpos.eatinorders.adapter.ordertable.out;
+
+import java.util.UUID;
+import kitchenpos.eatinorders.application.ordertable.port.out.OrderTableNewRepository;
+import kitchenpos.eatinorders.domain.ordertable.OrderTableNew;
+import org.springframework.data.jpa.repository.JpaRepository;
+
+public interface JpaOrderTableRepository extends OrderTableNewRepository,
+    JpaRepository<OrderTableNew, UUID> {
+
+}

src/main/java/kitchenpos/eatinorders/application/eatinorder/DefaultEatinOrderService.java

@@ -0,0 +1,68 @@
+package kitchenpos.eatinorders.application.eatinorder;
+
+import java.util.UUID;
+import java.util.stream.Collectors;
+import kitchenpos.eatinorders.application.eatinorder.port.in.EatinOrderDTO;
+import kitchenpos.eatinorders.application.eatinorder.port.in.EatinOrderInitCommand;
+import kitchenpos.eatinorders.application.eatinorder.port.in.EatinOrderUseCase;
+import kitchenpos.eatinorders.application.eatinorder.port.out.EatinOrderRepository;
+import kitchenpos.eatinorders.application.eatinorder.port.out.ValidMenuPort;
+import kitchenpos.eatinorders.application.exception.NotExistEatinOrderException;
+import kitchenpos.eatinorders.application.ordertable.port.out.OrderTableNewRepository;
+import kitchenpos.eatinorders.domain.eatinorder.EatinOrder;
+import org.apache.commons.lang3.tuple.Pair;
+import org.springframework.transaction.annotation.Transactional;
+
+public class DefaultEatinOrderService implements EatinOrderUseCase {
+
+    private final EatinOrderRepository eatinOrderRepository;
+    private final OrderTableNewRepository orderTableRepository;
+    private final ValidMenuPort menuFindPort;
+
+    public DefaultEatinOrderService(final EatinOrderRepository eatinOrderRepository,
+        final OrderTableNewRepository orderTableRepository, final ValidMenuPort menuFindPort) {
+
+        this.eatinOrderRepository = eatinOrderRepository;
+        this.orderTableRepository = orderTableRepository;
+        this.menuFindPort = menuFindPort;
+    }
+
+    @Override
+    public EatinOrderDTO init(final EatinOrderInitCommand command) {
+        final EatinOrder eatinOrder = eatinOrderRepository.save(
+            EatinOrder.create(command.getOrderTableId(),
+                command.getOrderLineItemMenuIds(),
+                command.getOrderLineItemCommands()
+                    .stream()
+                    .map(item -> Pair.of(item.getMenuId(), item.getQuantity()))
+                    .collect(Collectors.toUnmodifiableList()),
+                orderTableRepository, menuFindPort));
+
+        return new EatinOrderDTO(eatinOrder);
+    }
+
+    @Override
+    public void accept(final UUID id) {
+        final EatinOrder order = eatinOrderRepository.findById(id)
+            .orElseThrow(() -> new NotExistEatinOrderException(id));
+
+        order.acceptOrder();
+    }
+
+    @Override
+    public void serve(final UUID id) {
+        final EatinOrder order = eatinOrderRepository.findById(id)
+            .orElseThrow(() -> new NotExistEatinOrderException(id));
+
+        order.serveOrder();
+    }
+
+    @Transactional
+    @Override
+    public void complete(final UUID id) {
+        final EatinOrder order = eatinOrderRepository.findById(id)
+            .orElseThrow(() -> new NotExistEatinOrderException(id));
+
+        order.completeOrder(orderTableRepository);
+    }
+}

src/main/java/kitchenpos/eatinorders/application/eatinorder/port/in/EatinOrderDTO.java

@@ -0,0 +1,22 @@
+package kitchenpos.eatinorders.application.eatinorder.port.in;
+
+import java.util.List;
+import java.util.UUID;
+import java.util.stream.Collectors;
+import kitchenpos.eatinorders.domain.eatinorder.EatinOrder;
+
+public final class EatinOrderDTO {
+
+    private final UUID id;
+    private final List<OrderLineItemDTO> orderLineItems;
+    private final UUID orderTableId;
+
+    public EatinOrderDTO(final EatinOrder order) {
+        this.id = order.getId();
+        this.orderLineItems = order.getOrderLineItems()
+            .stream()
+            .map(OrderLineItemDTO::new)
+            .collect(Collectors.toUnmodifiableList());
+        this.orderTableId = order.getOrderTableId();
+    }
+}

src/main/java/kitchenpos/eatinorders/application/eatinorder/port/in/EatinOrderInitCommand.java

@@ -0,0 +1,51 @@
+package kitchenpos.eatinorders.application.eatinorder.port.in;
+
+import java.util.List;
+import java.util.UUID;
+import java.util.stream.Collectors;
+
+public final class EatinOrderInitCommand {
+
+    private final UUID orderTableId;
+    private final List<OrderLineItemCommand> orderLineItemCommands;
+
+    public EatinOrderInitCommand(final UUID orderTableId,
+        final List<OrderLineItemCommand> orderLineItemCommands) {
+
+        this.orderTableId = orderTableId;
+        this.orderLineItemCommands = orderLineItemCommands;
+    }
+
+    public UUID getOrderTableId() {
+        return orderTableId;
+    }
+
+    public List<OrderLineItemCommand> getOrderLineItemCommands() {
+        return orderLineItemCommands;
+    }
+
+    public List<UUID> getOrderLineItemMenuIds() {
+        return orderLineItemCommands.stream()
+            .map(OrderLineItemCommand::getMenuId)
+            .collect(Collectors.toUnmodifiableList());
+    }
+
+    public static final class OrderLineItemCommand {
+
+        private final UUID menuId;
+        private final int quantity;
+
+        public OrderLineItemCommand(final UUID menuId, final int quantity) {
+            this.menuId = menuId;
+            this.quantity = quantity;
+        }
+
+        public UUID getMenuId() {
+            return menuId;
+        }
+
+        public int getQuantity() {
+            return quantity;
+        }
+    }
+}

src/main/java/kitchenpos/eatinorders/application/eatinorder/port/in/EatinOrderUseCase.java

@@ -0,0 +1,34 @@
+package kitchenpos.eatinorders.application.eatinorder.port.in;
+
+import java.util.UUID;
+import kitchenpos.eatinorders.application.exception.InvalidAcceptStatusException;
+import kitchenpos.eatinorders.application.exception.InvalidCompleteStatusException;
+import kitchenpos.eatinorders.application.exception.InvalidServeStatusException;
+import kitchenpos.eatinorders.application.exception.NotExistOrderTableException;
+
+public interface EatinOrderUseCase {
+
+    /**
+     * @throws NotExistOrderTableException orderTableId에 해당하는 table이 없을 때
+     * @throws IllegalStateException       orderLineItem에 있는 menu가 없는 menu일 때
+     */
+    EatinOrderDTO init(final EatinOrderInitCommand command);
+
+    /**
+     * @throws NotExistOrderTableException  orderTableId에 해당하는 table이 없을 때
+     * @throws InvalidAcceptStatusException accept를 할 수 없는 상태일 때
+     */
+    void accept(final UUID id);
+
+    /**
+     * @throws NotExistOrderTableException orderTableId에 해당하는 table이 없을 때
+     * @throws InvalidServeStatusException serve를 할 수 없는 상태일 때
+     */
+    void serve(final UUID id);
+
+    /**
+     * @throws NotExistOrderTableException    orderTableId에 해당하는 table이 없을 때
+     * @throws InvalidCompleteStatusException complete를 할 수 없는 상태일 때
+     */
+    void complete(final UUID id);
+}

src/main/java/kitchenpos/eatinorders/application/eatinorder/port/in/OrderLineItemDTO.java

@@ -0,0 +1,17 @@
+package kitchenpos.eatinorders.application.eatinorder.port.in;
+
+import java.util.UUID;
+import kitchenpos.eatinorders.domain.eatinorder.OrderLineItemNew;
+
+public final class OrderLineItemDTO {
+
+    private final UUID id;
+    private final UUID menuId;
+    private final int quantity;
+
+    public OrderLineItemDTO(final OrderLineItemNew orderLineItem) {
+        this.id = orderLineItem.getId();
+        this.menuId = orderLineItem.getMenuId();
+        this.quantity = orderLineItem.getQuantity();
+    }
+}

src/main/java/kitchenpos/eatinorders/application/eatinorder/port/out/EatinOrderRepository.java

@@ -0,0 +1,12 @@
+package kitchenpos.eatinorders.application.eatinorder.port.out;
+
+import java.util.Optional;
+import java.util.UUID;
+import kitchenpos.eatinorders.domain.eatinorder.EatinOrder;
+
+public interface EatinOrderRepository {
+
+    EatinOrder save(final EatinOrder eatinOrder);
+
+    Optional<EatinOrder> findById(final UUID id);
+}

src/main/java/kitchenpos/eatinorders/application/eatinorder/port/out/ValidMenuPort.java

@@ -0,0 +1,12 @@
+package kitchenpos.eatinorders.application.eatinorder.port.out;
+
+import java.util.List;
+import java.util.UUID;
+
+public interface ValidMenuPort {
+
+    /**
+     * @throws IllegalStateException orderLineItem에 있는 menu가 없는 menu일 때
+     */
+    void checkValidMenu(final List<UUID> menuIds);
+}

src/main/java/kitchenpos/eatinorders/application/exception/InvalidAcceptStatusException.java

@@ -0,0 +1,11 @@
+package kitchenpos.eatinorders.application.exception;
+
+import java.util.UUID;
+import kitchenpos.eatinorders.domain.eatinorder.Status;
+
+public class InvalidAcceptStatusException extends InvalidEatinOrderStatusException {
+
+    public InvalidAcceptStatusException(final UUID id, final Status currentStatus) {
+        super(id, currentStatus);
+    }
+}

src/main/java/kitchenpos/eatinorders/application/exception/InvalidCompleteStatusException.java

@@ -0,0 +1,11 @@
+package kitchenpos.eatinorders.application.exception;
+
+import java.util.UUID;
+import kitchenpos.eatinorders.domain.eatinorder.Status;
+
+public class InvalidCompleteStatusException extends InvalidEatinOrderStatusException {
+
+    public InvalidCompleteStatusException(final UUID id, final Status currentStatus) {
+        super(id, currentStatus);
+    }
+}

src/main/java/kitchenpos/eatinorders/application/exception/InvalidEatinOrderStatusException.java

@@ -0,0 +1,12 @@
+package kitchenpos.eatinorders.application.exception;
+
+import java.util.UUID;
+import kitchenpos.eatinorders.domain.eatinorder.Status;
+
+public class InvalidEatinOrderStatusException extends RuntimeException {
+
+    public InvalidEatinOrderStatusException(final UUID id, final Status currentStatus) {
+        super(String.format("invalid eatin order status. id: %s, currentStatus: %s", id,
+            currentStatus));
+    }
+}

src/main/java/kitchenpos/eatinorders/application/exception/InvalidServeStatusException.java

@@ -0,0 +1,11 @@
+package kitchenpos.eatinorders.application.exception;
+
+import java.util.UUID;
+import kitchenpos.eatinorders.domain.eatinorder.Status;
+
+public class InvalidServeStatusException extends InvalidEatinOrderStatusException {
+
+    public InvalidServeStatusException(final UUID id, final Status currentStatus) {
+        super(id, currentStatus);
+    }
+}

src/main/java/kitchenpos/eatinorders/application/exception/NotExistEatinOrderException.java

@@ -0,0 +1,22 @@
+package kitchenpos.eatinorders.application.exception;
+
+import com.google.common.base.MoreObjects;
+import java.util.UUID;
+
+public final class NotExistEatinOrderException extends RuntimeException {
+
+    private final UUID id;
+
+    public NotExistEatinOrderException(final UUID id) {
+        super(String.format("not exist order. id: %s", id));
+
+        this.id = id;
+    }
+
+    @Override
+    public String toString() {
+        return MoreObjects.toStringHelper(this)
+            .add("id", id)
+            .toString();
+    }
+}