-
Notifications
You must be signed in to change notification settings - Fork 46
[1팀 안재현] Chapter 3-3 기능 중심 아키텍처와 프로젝트 폴더 구조 #31
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
JaeHyunGround
wants to merge
43
commits into
hanghae-plus:main
Choose a base branch
from
JaeHyunGround:main
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- 전역 레이아웃을 담당하는 요소라고 판단하여 app 레이어에 배치했습니다.
기존 /components/index.tsx 에 있던 컴포넌트 무더기 코드를 shared 레이어 ui 세그먼트로 분리했습니다.
- api 호출 로직 - 타입 을 entities 레이어에 생성했습니다.
- api 호출 로직 - 타입 을 entities 레이어에 생성했습니다.
- api 호출 로직 - 타입 을 entities 레이어에 생성했습니다.
- api 호출 로직 - 타입 을 entities 레이어에 생성했습니다.
header/footer)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
배포 링크 : https://jaehyunground.github.io/front_7th_chapter3-3/
과제 체크포인트
기본과제
목표 : 전역상태관리를 이용한 적절한 분리와 계층에 대한 이해를 통한 FSD 폴더 구조 적용하기
체크포인트
심화과제
목표: 서버상태관리 도구인 TanstackQuery를 이용하여 비동기코드를 선언적인 함수형 프로그래밍으로 작성하기
체크포인트
최종과제
과제 셀프회고
폴더 구조
FSD 분리 기준
위와 같이 FSD 분리 기준을 잡고 과제를 진행했습니다.
과제를 진행하면서 어려웠던 부분은
entities와features의 분리 기준이 적절한지? 로직을 분리하면서도어느 레이어에 넣어야하지와 같은 고민을 반복적으로 하게 됐습니다.저만의 분리 기준을 세웠음에도 불구하고
모호함의 영역이 해소되지 않았던 점이 답답하면서도 모호한 기준을 가지고 고민하는 과정을 거쳐가면서 어느정도 가닥이 잡혀가는 것이 흥미로웠습니다.완벽한 FSD 분리 기준을 세우지는 못했지만, 레이어를 구분 함으로써 단방향 참조를 신경쓰고 명확한 폴더 구조를 잡아 가는 것에 대한 중요성을 느낄 수 있었습니다.
이번 과제를 통해 이전에 비해 새롭게 알게 된 점이 있다면 적어주세요.
FSD 분리 기준에 대해서 생각해볼 수 있었습니다. 이전에 회사 업무에 FSD를 도입해 본 경험이 있습니다. FSD를 처음 접해봤기도 했고, 팀원들 간 분리 기준에 대해 상이했던 부분이 많아 결국 폴더 구조가 엉망진창이 되었던 기억이 납니다.
이전까지 모호했던 레이어 - 슬라이스 - 세그먼트 의 개념에 대해서 이해할 수 있었고, 이전에 FSD를 도입했을 때 문제점이 무엇이었으며, 다음 프로젝트에선 어떻게 FSD를 도입해볼 지 고민해볼 수 있었습니다.
본인이 과제를 하면서 가장 애쓰려고 노력했던 부분은 무엇인가요?
다른 사람이 봤을 때 이해하기 쉬운 폴더 구조를 가져가고자 노력했습니다.
처음에는 CRUD 중에 R(useQuery)을 entities 레이어에, 나머지(useMutation)는 features 레이어에 두려다가 CRUD 기능을 분리하여 관리하는 것이 어색하다 생각해서
entities 레이어에 api 함수를,features 레이어에 useQuery, useMutation 로직을 넣어도 보고,단순 데이터를 props로 받아 view의 역할만 하는 ui는 entities, 데이터의 변화을 일으키는 ui 요소는 features 레이어로 분리도 해보면서
많은 분들이 헷갈려했던
entities, features 레이어에는 각각 어떤 로직들이 들어가야 할까?에 대해 한 가지 방향성을 제시할 수 있을만한 폴더 구조를 짜고자 노력했던 것 같습니다.아직은 막연하다거나 더 고민이 필요한 부분을 적어주세요.
post/ui/post-tag.tsx 와 /widgets/posts-table/ui/post-table-row.tsx이번에 배운 내용 중을 통해 앞으로 개발에 어떻게 적용해보고 싶은지 적어주세요.
회사 업무에 FSD를 재적용하여 성공적인 FSD 도입 사례를 남겨보고 싶습니다.
팀원 간 폴더 구조 컨벤션을 명확히 하고 이를 문서화하여 팀원 간 생각의 차이가 많이 나지 않도록 하고, 커뮤니케이션 비용을 줄여 빠른 생산성을 직접적으로 느껴보고 싶습니다.
챕터 셀프회고
클린코드: 읽기 좋고 유지보수하기 좋은 코드 만들기
코드가 눈에 안 들어오고, 서비스의 기능은 어떤 것이며 어디까지 특정 함수에 대한 로직인지 파악이 잘 안됐습니다. 코드를 보자마자 들었던 생각은
아.....였던 것 같네요 ㅎㅎ클린코드를 작성해야 하는 이유는
내가 아닌 다른 사람이 작성한 코드를 볼 때 이해하기 쉽게 하기 위함이라고 생각합니다. 개발은 혼자 보단 여러 사람과 협업을 하는 경우가 훨씬 많고, 코드 구조를 구상하고 작성하는 시간보다이 코드는 무슨 역할을 하는 코드지?라는 생각을 더 많이 하게 되는 순간 커뮤니케이션 비용도 비싸지고 결국에는 효율성이 떨어지는 상태가 지속될 것이라고 생각합니다.따라서 클린 코드를 작성함으로써 팀원간 커뮤니케이션 비용을 줄이고, 전반적인 코드 스타일을 맞추어 같은 서비스 내의 코드에서도 이질감이 느껴지지 않게 하며, 코드를 이해하기 위해 많은 노력을 기울이지 않아도 되는 상황을 만드는 것이 이상적인 협업 과정이라고 생각했습니다 !
결합도 낮추기: 디자인 패턴, 순수함수, 컴포넌트 분리, 전역상태 관리
컴포넌트를 분리함으로써 컴포넌트의 재사용성을 늘리고, 계산 로직은 utils의 개념으로 액션 로직은 hooks의 개념으로 분리해보며 역할에 따라 명확하게 코드를 분리했을 때 가져오는 이점에 대해 체감할 수 있었습니다. (이점 : 관심사가 분리되어 원하는 로직을 빠르게 찾을 수 있다, 비즈니스 로직과 UI 로직을 분리함으로써 각 코드가 집중해야 할 부분을 명확히 함으로써 코드 가독성이 높아진다.)
응집도 높이기: 서버상태관리, 폴더 구조
다음에 FSD를 사용한다면 entities, features 레이어를 통합해서 써볼 수 있을 것 같고, widgets의 분리 기준을 명확히 할 수 있다면 지금보다 더 적절한 폴더구조를 가져갈 수 있을 것이라 생각했습니다.
폴더 구조를 짤 때 이전까진
남들이 했던 방법대로 따라했던 경우가 많았는데, 레이어 분리에 대한 저만의 기준을 세우고 그 기준에 따라 코드를 분리하는 경험을 함으로 인해 개발자로써 줏대(?)가 조금은 생긴 것 같습니다. ㅎ.ㅎ리뷰 받고 싶은 내용이나 궁금한 것에 대한 질문
코치님의 FSD 레이어 분리 기준
많은 분들께 여쭤봤을 때 FSD 레이어를 분리하는 기준이 다른 경우가 많았어서 흥미로웠습니다. 코치님께서는 프로젝트에서 FSD를 도입하신다고 하면
어떤 기준을 가지고 레이어를 분리하실 것인지,슬라이스(도메인)은 어떤 기준으로 나눌 것 같으신지궁금합니다 !entities, features의 슬라이스 에는 도메인 네이밍이, widgets의 슬라이스에는 UI 요소의 이름을 작성했는데 올바른 방법일지 궁금합니다 !
post/ui/post-tag.tsxposts-table/ui/post-table-row.tsx와 같이 entities 슬라이스는 도메인 네이밍, widgets 슬라이스는 UI 네이밍을 작성했는데 올바른 방법인지 궁금해요...!!