Skip to content

feat: week4 task#6

Open
taehyeon1998 wants to merge 1 commit intogonfrom
week4-gon
Open

feat: week4 task#6
taehyeon1998 wants to merge 1 commit intogonfrom
week4-gon

Conversation

@taehyeon1998
Copy link
Copy Markdown
Contributor

📂 관련 이슈

  • closes #[이슈 번호]

🛠️ 작업 사항

  • 4주차 아키텍처 구조 만들기

📸 관련 이미지 (스크린샷 또는 동영상)


image

💬 기타 설명

💡 추가적으로 공유할 내용이나 리뷰어에게 전달할 사항이 있다면 작성해 주세요.

@YONGJAEEE
Copy link
Copy Markdown
Contributor

YONGJAEEE commented Apr 7, 2026

전체적인 파일 명을 API 구현하며 실제 사용할 것들로 결정하여 잘 만들어둔 것 같아!

정말 사소하게 피드백주고 싶은 내용은 두가지인데,

  1. .gitkeep 활용하면 좋을 듯 !
    전체적으로 이름을 잘 지어둬서 크게 적용되는 내용은 아닐 것 같지만, 일부는 그냥 "도메인+모듈명" 형식으로 커밋되어있네
    크게 영향가는건 아니지만, 나중에 이름을 바꾸거나 삭제될 경우 리뷰어 입장에서 "왜 지웠지?" 하는 의문이 생길 수 있을거 같아

.gitkeep 파일을 대신 사용하면, 지금처럼 패키지들은 잘 커밋 되면서 리뷰어에게도 의문점 없이 진행이 가능해서 나중에 검토해보면 좋을듯 ~~

  1. build.gradle 파일 정리
    아직은 의존성이 많지 않아 괜찮지만, 프로젝트 규모가 커질수록 build.gradle 파일에 의존성도 많아져서 정렬되지 않으면 관리하기 힘들 것 같다는 생각이 들어
    지금은 별도 정렬 기준 없이 작성한 것 같은데

    1. implementation / testImplementation / testRuntimeOnly 등 구성에 맞추어 분리
    2. 공통된 의존성들끼리 (e.g. Swagger 의존성 (Swagger-ui, Swagger-api) 분리) 줄바꿈 혹은 주석

이런식으로 정리해보면 좋을 것 같아

@gyeonseo
Copy link
Copy Markdown

곤 전체적으로 다 잘 설계해줬다! 미션 도메인 안에 가게랑 지역 엔티티까지 다 넣어줬네??
확장성을 고려해보면 가게랑 지역은 따로 도메인 빼도 괜찮아보인다! 나머진 도메인형 아키텍처에 맞게 잘 설계했다 굿

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.

3 participants