https://github.com/checkuu-0216/order-king
https://www.youtube.com/watch?v=M8DvdKWwODE
1. 식사 시간 (최대한 시간 준수)
- 점심 : 12시 ~ 1시
- 저녁 : 18시 ~ 19시
2. 캠 되도록 키기
- 회의 시간 or 질문할 땐 무조건 키고 대화하기
3. 서로 배려하며 소통
4. 개인 진행도 & 질문 관련 회의
- 회의 시 : 10시 ~ 11시
5. 모르는 거 질문하기! (편하게 주제 상관 없이 😀)
6. 자리 이동 Slack 에 남기기
도전기능 최대한 많이 건들여 보기
회의 시간 : 오전 10시 - 11시
회의 내용 : 과제 진행 상황 및 계획 공유 , 개인 일정
안녕하세요! SA 작성해 주신 것 잘 봤습니다!
아래는 각 요소 별 피드백입니다!
[ERD]
~~1. erd의 연관 관계를 보여주는 선의 경우에는 각각 어디로 향하는지 잘 보이도록 떨어트리면 좋을 것 같습니다.
-> 나중에 구현을 위해 erd를 보거나, 혹은 수정이 필요할 때 선이 겹치면 가독성이 떨어지기 때문입니다! user 경우에는 서로 선이 많이 겹치게 되는데 연관 관계가 잘 보이도록만 선의 위치를 조절해주시면 충분합니다.
2. User 테이블의 User_Enum 칼럼은 case가 중간에 맞지 않습니다. 오타로 보이지만, 이왕이면 case를 맞춰 주시는 것이 개발 시에 유리합니다. (아마 자바 엔티티로는 예쁘게 작성되어있을 것 같습니다!)
-> 자바의 entity에는 camelCase를 쓰고, DB 칼럼으로는 snake_case를 쓰는 경우가 많은데, 이를 손쉽게 구현하기 위하여 springboot의 application 파일에 jpa 네이밍 전략을 지정하고, entity와 table 간의 다른 케이스를 쉽게 매핑할 수 있습니다. 이런 설정을 사용하게 된다면 case의 일관성을 맞추는 것이 중요해지기 때문에 꼭 확인해주시는 것이 좋습니다.
3. open_time과 close_time이 bigint 형으로 되어있습니다. 이 두 칼럼의 값은 시간이므로, 값을 표현하기 힘든 bigint 보다는 time 자료형을 써주시는 것이 좋습니다.
4. review에 있는 point의 경우는 범위가 제한되어 있어서 int 형으로도 충분히 표현이 가능합니다.
5. order의 status 칼럼은 어떤 enum이 들어가는지 남겨주시면 좋습니다.~~
[API]
1. response body가 필요한 경우에 어떤 형식으로 값을 반환하는지 상세히 적어주시는 것이 좋습니다. request body처럼 작성해주시면 됩니다.
~~2. 가게 생성 및 수정의 경우 openTime, closeTime을 int 보다는 최소한 String으로 받아서 시간으로 변환시켜주시거나, 시간 값을 받을 수 있는 타입으로 받아주시는 것이 좋습니다.
-> Java docs 등에서 LocalTime 쪽을 봐주시면 좋을 것 같습니다.~~
~~3. request parameter는 url 상에 있으므로, 이를 명시적으로 api 명세에 표시해주시면 좋을 것 같습니다!~~
~~4. 주문 수락의 경우 상태가 update 되는 것이므로 put 또는 patch가 적절할 것으로 보입니다.~~
~~-> 이를 반영할 시, 주문 취소와 api 명세가 겹칠 수 있습니다. 그런 경우에는 한 개의 api에서 주문 상태를 변경할 수 있도록 처리하시거나, 주문 취소의 http method를 delete 등으로 변경하셔도 될 것 같습니다.~~
~~5. 리뷰 생성, 수정의 경우에 어떤 주문에 대해서 리뷰를 남기고, 별점이 몇 점인지 빠져있습니다. 추가해주시면 좋을 것 같습니다.~~
~~-> 해당 내용이 있어야 어느 가게에 리뷰를 남겼는지 연결이 가능할 것 같습니다!~~
