Conversation
| ### Secure ### | ||
| .env | ||
|
|
||
|
|
| package umc.domain.point.controller; | ||
|
|
||
| public class PointController { | ||
| } |
There was a problem hiding this comment.
제 ERD에는 관련된 내용이 없지만, 와이어 프레임에서 볼 때 '미션 10개 달성 시 1000p'라는 내용이 있었어요. 포인트가 미션 완료 뿐만 아니라 다른 상황에서도 발생한다고 생각했습니다.
그래서 미션 도메인에 종속시키는 것 보다 별도로 분리하는 방향을 선택했습니다.
| package umc.domain.notification.controller; | ||
|
|
||
| public class NotificationController { | ||
| } |
| package umc.domain.category.controller; | ||
|
|
||
| public class CategoryController { | ||
| } |
There was a problem hiding this comment.
카테고리 도메인도 분리해주셨네요! 어떤 카테고리를, 어떤 상황을 염두에 두고 분리하신건지 궁금합니다
There was a problem hiding this comment.
제 ERD를 기준으로 카테고리는 사용자, 가게와 연결되어 있습니다. 한 쪽 도메인에 종속시키면 다른 도메인에서 카테고리를 사용할 때 결합도가 올라간다고 판단했습니다. 또, 사용자 선호 카테고리에 따라 해당 카테고리의 가게를 추천하는 등 별도의 기능으로 사용될 것 같다고 생각해서 따로 도메인을 분리했습니다.
YoungJJun
left a comment
YoungJJun
left a comment
There was a problem hiding this comment.
4주차 피드백
-
yml 파일 환경변수 설정하신 점 좋습니다.
-
build.gradle 파일 의존성 설정 각자 상황에 맞게 잘 수정해주셨습니다.
-
build.gradle 9th → 10th 찾아주셨군요!!
-
본인 DB 설계에 맞춰서 domain 잘 설계해주신 것 같아요.
여니가 먼저 리뷰해주시고 해당 리뷰에 설계 의도까지 작성해주셔서 보면서 바로바로 이해됐습니다.- 카테고리가 사용자, 가게와 모두 연결되어 있다는 이유로 분리
- 알람구현
- 포인트 분리
- 정책 분리
→ 모두 많이 고민하시고 설계하신 것 같아요!
-
하나 피드백 드릴 사항으로는 Member 아니고 User라고 만들어 주셨습니다.
USER는 대부분의 RDBMS에서 예약어로 사용되고 이로인해 충돌가능성이 있습니다. 또한 Spring Security 사용하시게 되면 UserDetails, User등이 있어서 Member로 많이 사용하는 것 같습니다!
윤샘 4주차 수고하셨습니다! 시험 잘보세요~ 🍎
3 participants
✅ 실습 체크리스트
✅ 컨벤션 체크리스트
📌 주안점