[4주차/빈] 워크북 제출합니다

[2u6in]

✅ 실습 체크리스트

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

✅ 컨벤션 체크리스트

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

📌 주안점

[migi0401]

문의사진까지 하시다니 꼼꼼하게 잘 하신 거 같아요!

[migi0401]

소셜 로그인을 고려하여 enums 패키지를 잘 구성해주신 거 같습니다.

[migi0401]

미션 지역도 잘 구성한 것 같아요.

[migi0401]

꼼꼼하게 도메인 기반 구조를 구성하신 거 같습니다! 고생하셨습니다!

[YoungJJun]

4주차 피드백

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

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

• Inquiry.java 파일이 member 내부의 entity 패키지에 있어요.
  무조건 빼야한다는 아니지만 일반적으로는 하나의 도메인으로 빼는편입니다. Inquiry가 멤버 내부에 있게되면 member의 책임소재가 너무 커진다고 생각해요.

• Term, Food(선호 카테고리같아요)도 위와같은 이유로 별도 도메인으로 빼는걸 추천드려요.
  카테고리를 예를들어 설명드리자면 멤버와도 관련있지만 나중에 가게를 분류하거나 검색 필터로 쓸 때 사용될 수 있는데 이 경우 해당 상황에서 member 패키지를 의존해야해서 순환 참조나 결합이 너무 강해지는 문제가 발생할 수 있어요.

• mission 패키지에 region과 store도 위와 동일한 이유로 분리 추천드려요.

• 분리 이유에 대해서는 미키 4주차 리뷰에 약간 더 자세히 작성한 것 같아서 이해 안되시면 보고 오시면 좋을 것 같아용.

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

---

빈 4주차 수고하셨습니다!

원래 피드백에 대한 수정은 필수가 아니지만.. 이후 개발 진행에 있어서 나중에 수정하려고 하면 너무 어려워질 것 같아서 미리 수정하는게 좋을 것 같습니당.
5주차 진행 전에만 머지하면 되니까 시험 끝나고 천천히 해주셔도 됩니다.

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