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

README.md

@@ -56,13 +56,14 @@ docker compose -p kitchenpos up -d
 - 주문 테이블을 등록할 수 있다.
 - 주문 테이블의 이름이 올바르지 않으면 등록할 수 없다.
   - 주문 테이블의 이름은 비워 둘 수 없다.
-- 빈 테이블을 해지할 수 있다.
+- 빈 테이블을 점유할 수 있다.
 - 빈 테이블로 설정할 수 있다.
 - 완료되지 않은 주문이 있는 주문 테이블은 빈 테이블로 설정할 수 없다.
-- 방문한 손님 수를 변경할 수 있다.
-- 방문한 손님 수가 올바르지 않으면 변경할 수 없다.
-  - 방문한 손님 수는 0 이상이어야 한다.
-- 빈 테이블은 방문한 손님 수를 변경할 수 없다.
+
+- 고객 인원를 변경할 수 있다.
+- 고객 인원이 올바르지 않으면 변경할 수 없다.
+  - 고객 인원는 0 이상이어야 한다.
+- 빈 테이블은 방문한 고객 인원을 변경할 수 없다.
 - 주문 테이블의 목록을 조회할 수 있다.
 
 ### 주문
@@ -147,6 +148,7 @@ docker compose -p kitchenpos up -d
 | 주문 종류 | Order Category   | 주문 가능한 유형                           |
 | 주문 상태 | Order Status     | 주문의 생애주기(생성 부터 완료)를 나타내는 상태         |
 | 배달 장소 | Delivery Address | 배달 주문의 경우, 필수적으로 필요한 손님이 전달을 요청한 장소 |
+| 주문 메뉴 | OrderLine Item   | 주문을 구성하는 하위 메뉴 항목들                  |
 
 ### 주문 행위
 | 한글명 | 영문명        | 설명                          |
@@ -249,7 +251,7 @@ docker compose -p kitchenpos up -d
     - DELIVERING: 배달중 (라이더에게 전달 완료)
     - DELIVERED: 배달완료 (라이더가 배달 완료)
     - COMPLETED: 주문완료 (주문완료 설정)
-  - `주문`은 주문한 메뉴들을 가진다.
+  - `주문`은 `주문 메뉴`들을 가진다.
   - `주문`은 `주문 종류`를 가진다.
     - DELIVERY - 배달 (필수값: `배달 장소`)
     - TAKEOUT - 포장
@@ -260,7 +262,7 @@ docker compose -p kitchenpos up -d
     - `주문`을 생성할 때, `대기중(WAITING)` 으로 `주문 상태`를 변경한다.
     - `메뉴` 내의 `주문상품`들이 삭제된 `상품`이면 안된다.
     - `메뉴`가 생성된 상태이고 `공개 상태(VISIBLE)` 되어있어야한다.
-    - `메뉴 가격` 총액과 `주문`에 속한 `상품 가격` 총액이 동일해야한다.
+    - `주문 가격` 총액과 `주문`에 속한 `메뉴 가격` 총액이 동일해야한다.
     - `주문`은 `메뉴` 1개 이상으로 구성되어야 한다.
   2. `주문 종류`: `매장내주문`
     - `주문 테이블`이 없으면 `주문`을 생성 할 수 없다.
@@ -271,14 +273,15 @@ docker compose -p kitchenpos up -d
   - 그림의 3번: `주문`을 수락할 때, `주문 상태`는 `접수(ACCEPTED)`(`ACCEPTED` 상태)
   - 그림의 4번: `주문`을 `손님`에게 전달할 때, `주문 상태`는 `전달(SERVED)`으로 변경한다.(`SERVED` 상태)
   - 그림의 5번: `주문`을 완료할 때, `주문 상태`는 `주문완료(COMPLETED)`, `주문 테이블`은 `미사용중(UNOCCUPIED)`으로 변경한다.(`COMPLETED` 상태)
+
 2. `손님`이 `포장주문`을 하여 주문에 대한 메뉴를 전달 받는다.
 
 - 행동
   1. `주문 종류`: 공통 (해당 조건은 공통으로 지켜져야 한다.)
     - `주문`을 생성할 때, `대기중(WAITING)` 으로 `주문 상태`를 변경한다.
-    - `메뉴` 내의 `주문상품`들이 삭제된 `상품`이면 안된다.
+    - `메뉴` 내의 `주문 메뉴`들이 삭제된 `메뉴`이면 안된다.
     - `메뉴`가 생성된 상태이고 `공개 상태(VISIBLE)` 되어있어야한다.
-    - `메뉴 가격` 총액과 `주문`에 속한 `상품 가격` 총액이 동일해야한다.
+    - `주문 가격` 총액과 `주문`에 속한 `메뉴 가격` 총액이 동일해야한다.
     - `주문`은 `메뉴` 1개 이상으로 구성되어야 한다.
   2. `주문 종류`: `포장주문`
     - 주문을 생성할 때, 주문상품들의 갯수를 감소시킬 수 없다.
@@ -293,12 +296,12 @@ docker compose -p kitchenpos up -d
 - 행동
   1. `주문 종류`: 공통 (해당 조건은 공통으로 지켜져야 한다.)
     - `주문`을 생성할 때, `대기중(WAITING)` 으로 `주문 상태`를 변경한다.
-    - `메뉴` 내의 `주문상품`들이 삭제된 `상품`이면 안된다.
+    - `메뉴` 내의 `주문 메뉴`들이 삭제된 `상품`이면 안된다.
     - `메뉴`가 생성된 상태이고 `공개 상태(VISIBLE)` 되어있어야한다.
-    - `메뉴 가격` 총액과 `주문`에 속한 `상품 가격` 총액이 동일해야한다.
+    - `주문 가격` 총액과 `주문`에 속한 `메뉴 가격` 총액이 동일해야한다.
     - `주문`은 `메뉴` 1개 이상으로 구성되어야 한다.
   2. `주문 종류`: `배달주문`
-    - `주문`이 생성할 때, `주문 상품`들의 `갯수`를 감소 시킬 수 없다.
+    - `주문`이 생성할 때, `주문 메뉴`들의 `갯수`를 감소 시킬 수 없다.
     - `주문`이 생성할 때, `주문 장소`가 필수로 기입되어야 한다.
   3. `손님`이 `배달주문`을 하여 `손님`이 주문에 대한 메뉴를 전달 받는다.
   - 그림의 2번: `주문`을 생성할 때, `주문 상태`는 `대기중(WAITING)`, `주문 종류`는 `배달 주문`으로 변경한다.(`WAITING` 상태)

REQUIREMENTS.md

@@ -0,0 +1,58 @@
+## 과제 요구사항
+
+1. 상품 context 리팩터링
+- 도메인 계층 구현하기
+- 도메인 서비스 
+- 간접 참조로 aggregate 구현하기
+- REPOSITORY 구현
+- Validator 구현
+- 도메인 부분에서 VO 랑 AGGGREGATE 용
+  - 식별을 위한 ID, 추가
+  - VALIDATION을 위한 값들 추가
+  - 테스트 작성
+
+
+
+3. EAT_IN ORDER 리팩터링 구현사항
+A. 연관관계 설정 및 VO 및 도메인 로직 구현
+- ORDER (바운디드 컨텍스트는 넓게 잡기가 선호되서 하나의 패키지로 구현하였습니다.)
+  - EAT_IN
+  - DELIVERY
+    - DELIVERY 주문의 외부 연동 도메인 로직을 DOMAIN SERVICE로 주입
+  - TAKEOUT
+  - 생성 도메인 로직
+  - 상태 변환 도메인 로직
+  - FACTORY로 각 ORDER에 대한 생성로직 따로 클래스로 생성
+
+- ORDER ITEMS
+  - 생성 도메인로직
+    - 메뉴에 대한 의존성을 MenuClient로 가져오는 객체는 OrderMenu로 명시적 표기
+    - N+1 문제를 쿼리 한번으로 변경
+
+- ORDERTABLE
+  - 생성 도메인 로직
+  - 치우기 도메인 로직
+    - OrderTable에서 Order에 대해 접근할때, domain service로 주입
+- ORDER SERVICE
+  - 서비스에 있는 도메인 로직 도메인 객체안으로 넣기
+
+B. @DOMAIN SERVICE 와 @DOMAIN FACTORY에 대한 에노테이션 생성으로 명시적 표기
+
+
+1. 책에 따라 DOMAIN SERVICE를 METHOD 호출시 전달하게끔 변경하였습니다. '외부 시스템 연동이 필요한 도메인 로직일때 혹은 계산로직일때'
+
+
+2. 1-N(주문, 주문항목들) 의 경우 N이 많아질 때가 있는데 해당부분은 어떻게 구현하실지 궁금합니다.
+
+
+3. 주문 타입의 경우가 많아지면, 어떤 타입 -> 팩토리에 대한 맵핑을 지원하는 디자인 패턴을 사용하시나요?
+
+
+4. 지금 상태에서 실무에 돌입하게 되면 ORDER 와 EATIN ORDER의 관계에서 상속이 객체지향적이지 않아 보여요! (EJ 합성 > 상속) 
+   - 웅래님은 만약 실무였다면, 어떻게 구현하셨을지 궁금합니다.
+   - 저라면, 공유커널(COMMON)에 빈 껍데기 ORDER, ORDERITEMS를 두기
+     - 공유커널을 상속받아서 DOMAIN ENTITY들을 각각 EATIN_ORDER, DELIVERY_ORDER, TAKEOUT_ORDER 패키지에서 각각 상속받아서 생성했을것같아요
+
+5. 마지막으로, 공유커널(COMMON)에 있는 이벤트들은 내부 이벤트(내부 서비스) VS 외부 이벤트(외부 서비스) 들이 있을것 같아요
+   - 보통은 어떻게 나누시나요? (인터페이스?, 마커 인터페이스?)
+

http/order-tables.http

@@ -1,24 +1,24 @@
 ###
-POST {{host}}/api/order-tables
+POST {{host}}/api/orderRequest-tables
 Content-Type: application/json
 
 {
   "name": "9번"
 }
 
 ###
-PUT {{host}}/api/order-tables/8d710043-29b6-420e-8452-233f5a035520/sit
+PUT {{host}}/api/orderRequest-tables/8d710043-29b6-420e-8452-233f5a035520/sit
 
 ###
-PUT {{host}}/api/order-tables/8d710043-29b6-420e-8452-233f5a035520/clear
+PUT {{host}}/api/orderRequest-tables/8d710043-29b6-420e-8452-233f5a035520/clear
 
 ###
-PUT {{host}}/api/order-tables/8d710043-29b6-420e-8452-233f5a035520/number-of-guests
+PUT {{host}}/api/orderRequest-tables/8d710043-29b6-420e-8452-233f5a035520/number-of-guests
 Content-Type: application/json
 
 {
   "numberOfGuests": 4
 }
 
 ###
-GET {{host}}/api/order-tables
+GET {{host}}/api/orderRequest-tables

http/orders.http

@@ -5,7 +5,7 @@ Content-Type: application/json
 {
   "type": "EAT_IN",
   "orderTableId": "8d710043-29b6-420e-8452-233f5a035520",
-  "orderLineItems": [
+  "orderLineItemRequests": [
     {
       "menuId": "f59b1e1c-b145-440a-aa6f-6095a0e2d63b",
       "price": 16000,

src/main/java/kitchenpos/Application.java

@@ -2,10 +2,8 @@
 
 import org.springframework.boot.SpringApplication;
 import org.springframework.boot.autoconfigure.SpringBootApplication;
-import org.springframework.scheduling.annotation.EnableAsync;
 
 @SpringBootApplication
-@EnableAsync
 public class Application {
     public static void main(String[] args) {
         SpringApplication.run(Application.class, args);

src/main/java/kitchenpos/common/annotation/DomainService.java

@@ -0,0 +1,13 @@
+package kitchenpos.common.annotation;
+
+import org.springframework.stereotype.Component;
+
+import java.lang.annotation.*;
+
+@Target(ElementType.TYPE)
+@Inherited
+@Documented
+@Retention(RetentionPolicy.RUNTIME)
+@Component
+public @interface DomainService {
+}

src/main/java/kitchenpos/common/annotation/FactoryService.java

@@ -0,0 +1,13 @@
+package kitchenpos.common.annotation;
+
+import org.springframework.stereotype.Component;
+
+import java.lang.annotation.*;
+
+@Target(ElementType.TYPE)
+@Inherited
+@Documented
+@Retention(RetentionPolicy.RUNTIME)
+@Component
+public @interface FactoryService {
+}

src/main/java/kitchenpos/eatinorders/domain/OrderStatus.java → src/main/java/kitchenpos/common/domain/orders/OrderStatus.java

@@ -1,4 +1,4 @@
-package kitchenpos.eatinorders.domain;
+package kitchenpos.common.domain.orders;
 
 public enum OrderStatus {
     WAITING, ACCEPTED, SERVED, DELIVERING, DELIVERED, COMPLETED

src/main/java/kitchenpos/common/domain/orders/OrderTableStatus.java

@@ -0,0 +1,13 @@
+package kitchenpos.common.domain.orders;
+
+public enum OrderTableStatus {
+  UNOCCUPIED, OCCUPIED;
+
+  OrderTableStatus OrderTableStatus(boolean occupied) {
+    if (occupied) {
+      return OrderTableStatus.OCCUPIED;
+    }
+
+    return OrderTableStatus.OCCUPIED;
+  }
+}

src/main/java/kitchenpos/eatinorders/domain/OrderType.java → src/main/java/kitchenpos/common/domain/ordertables/OrderType.java

@@ -1,4 +1,4 @@
-package kitchenpos.eatinorders.domain;
+package kitchenpos.common.domain.ordertables;
 
 public enum OrderType {
     DELIVERY, TAKEOUT, EAT_IN

src/main/java/kitchenpos/common/domain/products/ProductPriceChangeEvent.java

@@ -0,0 +1,21 @@
+package kitchenpos.common.domain.products;
+
+import java.util.UUID;
+
+public class ProductPriceChangeEvent {
+  private final UUID productId;
+  private final Long price;
+
+  public ProductPriceChangeEvent(UUID productId, Long price) {
+    this.productId = productId;
+    this.price = price;
+  }
+
+  public UUID getProductId() {
+    return productId;
+  }
+
+  public Long getPrice() {
+    return price;
+  }
+}