[4주차/미키] 워크북 제출합니다.

[migi0401]

✅ 실습 체크리스트

• 이론 학습을 완료하셨나요?
• 미션 요구사항을 모두 이해하셨나요?
• 실습을 수행하기 위한 공부를 완료하셨나요?
• 실습 요구사항을 모두 완료하셨나요?

✅ 컨벤션 체크리스트

• 디렉토리 구조 컨벤션을 잘 지켰나요?
• pr 제목을 컨벤션에 맞게 작성하였나요?
• pr에 해당되는 이슈를 연결하였나요?(중요)
• 적절한 라벨을 설정하였나요?
• 파트장에게 code review를 요청하기 위해 reviewer를 등록하였나요?
• 닉네임/main 브랜치의 최신 상태를 반영하고 있는지 확인했나요?(매우 중요!)

📌 주안점

[YoungJJun]

4주차 피드백

• yml 파일 환경변수 설정하신 점 좋습니다.

• gitignore에 .env파일 추가 좋아요!

• build.gradle 파일 의존성 설정 각자 상황에 맞게 잘 수정해주셨습니다!

• SwaggerConfig 파일에 JWT까지 벌써 고려해주신 점 좋아요.

• mission 패키지에 store 엔티티가 포함되어 있어요.
  store가 mission을 생성하기 때문에 이렇게 설계하신 것 같아요. 분리하는거 추천합니다! 비슷한 설계가 많아서 이유는 아래에서 따로 설명드릴게요.

• member 패키지에 food, policy가 포함되어 있어요.
  이 부분도 member가 food(category)와 정책과 연결되기 때문에 이렇게 설계하신 것 같아요. 여기도 분리 추천합니다!

• 위에서 분리하라는 말은 domain/store 이런식으로 store, food, policy 모두 각각의 도메인으로 볼 수 있도록 패키지를 나누라는 말 입니다.

• 이유에 대해 설명드리자면 우선 정책을 멤버 아래에 두게되면 정책이 멤버에 종속되는 느낌으로 보여지게 됩니다. 하지만 실제로 정책은 멤버 없이도 생성, 존재할 수 있어요.

• 회원가입 시점을 예를들어 사용자에게 동의할 정책을 보여주는 API를 구현한다고 가정하면, 멤버쪽에서 정책에 관한 일을 하는 상황이 발생합니다. 단일책임원칙에 어긋나요.

• 음식(카테고리)를 예를들면 카테고리는 멤버만이 아니라 가게와도 연결됩니다. 중식 가게를 찾으려고 가게 조회를 수행하는데 카테고리가 멤버 아래에 있으니 멤버 패키지를 의존하게 되고 쓸데없는 결합이 생겨요. 순환참조 발생 가능성도 높아집니다.

---

미키 4주차 수고하셨습니다!

원래 피드백에 대한 수정은 필수가 아니지만.. 이후 개발 진행에 있어서 나중에 수정하려고 하면 너무 어려워질 것 같아서 미리 수정하는게 좋을 것 같습니당.

5주차 진행 전에만 머지하면 되니까 시험 끝나고 천천히 해주셔도 됩니다.

다시한번 시험 화이팅입니다~ 🍎