Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
| export const getConsumables = async () => { | ||
| const { data } = await instance.get('/shop/consumables'); | ||
| return data; | ||
| }; |
There was a problem hiding this comment.
@electrohyun 이런 각각의 단일 api 로직들을 각각의 파일로 관리하는게 이유가 있을까요? 🤔 단일 api 로직들을 모두 개별 파일로 관리하면 파일 수가 과도하게 많아지지 않을까요? 어떤 의도가 있는지 궁금합니다 ㅎㅎ
There was a problem hiding this comment.
리뷰 감사합니다! 프로젝트 초기에 고민했던 부분입니다.
유틸 함수쪽에서는(사진은 Drawing입니다.) 제가 느끼기에 같은 목적을 위해 작성된 코드는 같은 파일에 작성해두려 했던것 같습니다.
각각 '색상 관련' / 'DOM 관련' / '페인트통 도구 관련'으로 생각할 수 있었던 반면,
API는 'Consumable 불러오기' / 'Cosmetic 불러오기' / 'Inventory 불러오기' / '구매하기' 이렇게 다른 목적을 위해 작성되었기 때문에 다른 파일 로 나누자라고 생각해왔습니다.
물론 그렇다고 해서, 파일을 이렇게 계속 분리한다면, 서비스 규모가 더 커질 때 파일이 정말정말 많아질 것 같고, 결과적으로 안좋겠다. 하는 생각도 들기 때문에 제 말이 맞다고는 절대 할 수 없다고 생각합니다.😅
이런 응집도와 분리의 적정선을 결정할 때 @ssi02014 님은 주로 어떤 기준(팀 컨벤션, 기준 등등)을 참고하시는지 여쭤봐도 괜찮을까요? 혹은 제가 이런 설계 감각을 익히기 위해 참고할 만한 좋은 공부 주제가 있다면 추천 부탁드리고 싶습니다! 😭
📝 작업 내용 (Description)
features/shop으로 통합 관리하도록 리팩토링했습니다.✨ 변경 사항 (Changes)
ProductCard,ShopProfilePanel,ShopPurchaseModal등 API 데이터 기반으로 전환features/cart,features/shop-purchase기능을features/shop으로 병합하여 응집도 향상useShop.ts)📸 스크린샷 (Screenshots)
✅ 리뷰어 체크리스트 (Reviewer Checklist)
📢 참고 사항 (Notes)
🧐 이슈 (Related Issues)