Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

🚀 1단계 - 용어 사전 만들기 #276

Open
wants to merge 2 commits into
base: shinseongsu
Choose a base branch
from

Conversation

shinseongsu
Copy link

안녕하세요

nextstep ddd 수강하게 된 신성수입니다
업무에서는 코드만 많이짜봐서 기획하는건 아직 어색하네요

제가 기초관련해서 잘못된 점이 있다면 서슴없이 말씀해주시면 감사하겠습니다 😄

Copy link

@wkdchdaud123 wkdchdaud123 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

안녕하세요. 😄
용어 사전 정리 잘 해주셨어요!
몇 가지 코멘트 남겼습니다.
확인 부탁드려요.

@@ -96,8 +96,69 @@ docker compose -p kitchenpos up -d

## 용어 사전

| 한글명 | 영문명 | 설명 |
Copy link

@wkdchdaud123 wkdchdaud123 Sep 3, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이쪽은 ###이 빠진 것 같아요.
###을 바운디드 컨텍스트 별로 생각하고 용어사전을 나누면 추후 보기도 편할 것 같고 정리하시기도 편할 것 같아요.
이쪽은 어떤 용어 사전일까요?
용어 사전은 왜 필요하다고 생각하시나요? 😄

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

상품 도메인영역인데 제가 빼먹고 쓰지 못했네요~

README.md Outdated
Comment on lines 101 to 103
| 상품 | Product | 키친포스에서 판매하는 음식 |
| 가격 | Price | 상품의 가격 |
| 상품 이름 | name | 상품의 이름 |

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

곳곳에 소문자, 대문자로 시작하는 것이 다다른데요 혹시 이유가 있나요? 😄

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

다른 에디터에서 정리하고 붙인거라 첫 단어는 대문자가 나와서 그렇게 했는데 통일성 유지하도록 하겠습니다.

README.md Outdated
Comment on lines 139 to 140
| 주문 항목 | Orderline Item | 손님이 주문한 메뉴 |
| 주문 가격 | order line price | 주문한 메뉴의 총 가격 |

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이쪽도 Orderline, order line 둘 중 하나로 통일되어야 독자에게 혼란을 주지 않을 것 같아요 😄

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

넵 알겠습니다.

README.md Outdated
Comment on lines 141 to 143
| 주문 상태 | Order Status | 주문의 상태 |
| 배달 주소 | delivery address | 배달 주문의 목적지 |
| 주문 상태 | order status | 손님의 주문 진행 상태 |

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

주문 상태 동일한 것이 2개인데 차이가 있나요? 😄

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

소스보면서 하느라 두개가 같은게 나왔네요~

@@ -96,8 +96,69 @@ docker compose -p kitchenpos up -d

## 용어 사전

| 한글명 | 영문명 | 설명 |

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

혹시 페어분과 함께 진행하셨나요~? 🙇

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@JungTag 님께서 팀 정해지기 전에 먼저 통과했다고 해서 모델링 부터 같이 하였습니다
용어사전은 저혼자 짜보는 연습하였습니다.

Comment on lines +105 to +109
### 메뉴 그룹

| 한글명 | 영문명 | 설명 |
| --- | --- | --- |
| | | |
|-----|-----|----|
| 메뉴 그룹 | menu group | 관련 메뉴 모음 |

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

메뉴에는 이름이 있는데 메뉴 그룹에 있는 이름 같은 것들도 정리해서 통일성을 제공하면 어떨까요?
전체적으로 다시 고민해보실 수 있을까요? 😄

용어 사전은 이해관계자들 모두와 커뮤니케이션 할 때 동일한 용어에 동일한 생각을 할 수 있는 수단의 일부로 보는 것이라고 생각해요 그래서 조금이라도 모호해서 동일한 용어에 다른 생각(그림)을 할 수 있는 용어들은 정리하는 것이 좋다고 생각해요.
더불어서 용어 사전에 정리된 것들이 코드에서도 동일하게 사용되니 통일성을 주는 것이 좋다고 생각합니다.

혹시 어떠신가요? 😄

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

역시 옆에서 누가 봐줘야하는것들이 있군요 좋은 의견입니다👏

README.md Outdated
| 메뉴 이름 | name | 메뉴의 이름 |
| 메뉴 가격 | price | 메뉴의 갸격 |
| 메뉴 상품 | Menu Product | 메뉴에 등록되는 상품 |
| 메뉴 주문 숨김여부 | displayed | 메뉴에서 손님이 볼 수 있는지 여부 |

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

요구사항에서 숨김과 노출이 별도로 있었던 것 같은데 최대한 동일한 용어를 사용하면 모든 이해관계자가 쉽게 이해할 수 있도록 하면 어떨까요? 😄
다른 의도가 있나요?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

displayed 를 flag로 생각하고 숨김/노출 이라기보다는 한단어로 사용했네요 수정하겠습니다

README.md Outdated
Comment on lines 128 to 131
| 사용 테이블 | occupied order table | 사용 테이블 |
| 미사용 테이블 | unoccupied order table | 미사용 테이블 |
| 사용 | sit | 테이블의 상태를 사용으로 변경 |
| 미사용 | clear | 테이블의 상태를 미사용으로 변경 |

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이쪽사용 테이블사용이쪽사용 용어가 동일해서 헷갈릴 수 있지 않을까요? 😄

추후 모든 이해관계자들과 대화 시 사용이라는 용어에 무엇을 생각해야 하는지 혼돈이 생기지는 않을까요? (미사용도 동일)

착석, 정리 와 같은 것으로 해보면 어떤가요? 😄

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

착석, 정리 확실히 알아듣기 쉬운 용어네요

README.md Outdated

| 한글명 | 영문명 | 설명 |
|-------|-------------|----------------------------|
| 주문 유형 | Order Type | 손님이 주문하는 유형 |

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

주문의 유형들이 어떤 것들이 있는지 용어는 어떻게 칭할 것인지 정리해서 보편언어로 사용될 수 있지 않을까요? 😄

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

유비쿼터스 언어로 사용하도록 하겠습니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants