🚀 3단계 - 기능 우선 패키지 구성하기

[jordy-torvalds]

No description provided.

[hyunssooo]

안녕하세요.
기능 우선 패키지 구성하기 잘 진행해 주셨습니다.
가벼운 코멘트 남겼는데요. 확인 부탁드릴게요.
다음 요청에는 머지하도록 하겠습니다. 🙂

[hyunssooo]

.
├── Application.java
├── common
│   └── infra
│       ├── DefaultPurgomalumClient.java
│       └── PurgomalumClient.java
├── delivery
│   └── infra
│       ├── DefaultKitchenridersClient.java
│       └── KitchenridersClient.java
├── menu
│   ├── api
│   │   └── MenuRestController.java
│   ├── application
│   │   └── MenuService.java
│   ├── domain
│   │   ├── Menu.java
│   │   ├── MenuProduct.java
│   │   └── MenuRepository.java
│   └── infra
│       └── JpaMenuRepository.java
├── menugroup
│   ├── api
│   │   └── MenuGroupRestController.java
│   ├── application
│   │   └── MenuGroupService.java
│   ├── domain
│   │   ├── MenuGroup.java
│   │   └── MenuGroupRepository.java
│   └── infra
│       └── JpaMenuGroupRepository.java
├── order
│   ├── api
│   │   └── OrderRestController.java
│   ├── application
│   │   └── OrderService.java
│   ├── domain
│   │   ├── Order.java
│   │   ├── OrderLineItem.java
│   │   ├── OrderRepository.java
│   │   ├── OrderStatus.java
│   │   └── OrderType.java
│   └── infra
│       └── JpaOrderRepository.java
├── ordertable
│   ├── api
│   │   └── OrderTableRestController.java
│   ├── application
│   │   └── OrderTableService.java
│   ├── domain
│   │   ├── OrderTable.java
│   │   └── OrderTableRepository.java
│   └── infra
│       └── JpaOrderTableRepository.java
└── product
    ├── api
    │   └── ProductRestController.java
    ├── application
    │   └── ProductService.java
    ├── domain
    │   ├── Product.java
    │   └── ProductRepository.java
    └── infra
        └── JpaProductRepository.java

전체적으로 잘 나눠주셨습니다. :)

현재 Order BC가 하나로 합쳐져 있는데요. 나눠보는 것도 고려해 보시면 좋을 것 같습니다.
개인적으로 생각해 봤는데요. 나눴을 때 고려해 볼 부분은 중복되는 부분들은 어떻게 할 것인가? 일 것 같아요.

• OrderType이 필요한가?
  • 클래스명으로 표현 가능하지 않을까? EatinOrder DeliveryOrder TakeoutOrder
• 하나의 OrderStatus로 각 주문의 상태들을 파악할 수 있을까?
• Repository는 Aggregate 하나이니 나눠지는 것이 맞지 않을까?
• OrderService는 각 주문마다 usecase가 다르니 나눠지는 것이 관리 측면에서 좋지 않을까?

정도로 고민해 봤어요.

1. KitchenridersClient 는 delivery로 나눠져 있는데요. Order랑 합쳐볼 수 도 있지 않을까요?

• deliveryOrder 에서만 사용해서요.

3. menu, menugroup이 별도의 BC로 나누신 원식님 생각을 들어보고 싶어요. 🙂
4. order, ordertabler 도 별도의 BC라고 봐주신 걸까요?

원식님도 피드백 한번 주시면 좋을 것 같습니다. 🙂