Skip to content
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

[4팀 최하늘] [Chapter 2-2] 디자인 패턴과 함수형 프로그래밍 #27

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

haaneul11
Copy link

@haaneul11 haaneul11 commented Jan 16, 2025

과제 체크포인트

기본과제

  • React의 hook 이해하기

  • 함수형 프로그래밍에 대한 이해

  • Component에서 비즈니스 로직을 분리하기

  • 비즈니스 로직에서 특정 엔티티만 다루는 계산을 분리하기

  • Component에서 사용되는 Data가 아닌 로직들은 hook으로 옮겨졌나요?

  • 주어진 hook의 책임에 맞도록 코드가 분리가 되었나요?

  • 계산함수는 순수함수로 작성이 되었나요?

심화과제

  • 뷰데이터와 엔티티데이터의 분리에 대한 이해

  • 엔티티 -> 리파지토리 -> 유즈케이스 -> UI 계층에 대한 이해

  • Component에서 사용되는 Data가 아닌 로직들은 hook으로 옮겨졌나요?

  • 주어진 hook의 책임에 맞도록 코드가 분리가 되었나요?

  • 계산함수는 순수함수로 작성이 되었나요?

  • 특정 Entitiy만 다루는 함수는 분리되어 있나요?

  • 특정 Entitiy만 다루는 Component와 UI를 다루는 Component는 분리되어 있나요?

  • 데이터 흐름에 맞는 계층구조를 이루고 의존성이 맞게 작성이 되었나요?

과제 셀프회고

과제에서 좋았던 부분

  • 어떤 파라미터를 받아 어떤 규모의 함수로 분리해야 하는지 배운것 같다.

과제를 하면서 새롭게 알게된 점

  • hook으로 분리해서 리팩토링 하는 방법? 어떤 기능을 분리해서 hook으로 해야하는지 화면과 기능을 분리해야하는지에 대해 맛보기 한 것 같다.

과제를 진행하면서 아직 애매하게 잘 모르겠다 하는 점, 혹은 뭔가 잘 안되서 아쉬운 것들

  • 드디어 이번주차에 과제에 대한 피드백을 받을 수 있게 Basic은 완수한 나에게 일단...칭찬.. ㅋㅋㅋ 담주에는 설 연휴로 한주 쉬니까 그동안 못한 과제와 심화과제까지 도전해보는 것이 목표이며
  • 코드를 돌아가게는 짜는건 잘하지만 어떤 코드가 보기 좋고 클린한지에 대한 개념이 아직 익숙지 않아 어느 정도로 코드를 리팩토링해야하는지 잘 모르겠어서 이부분에 대한 리뷰를 받고싶습니다!!

리뷰 받고 싶은 내용이나 궁금한 것에 대한 질문

  • 지금은 코드를 테스트만 통과하게 일단 구현을 직관적으로 했지만 이 코드에 대해 어떻게 디벨롭을 하면 좋을지 (코드를 구구절절 나열하여 구현하기보다 한줄로 코딩하면 좋은지 -> 제 경우에는 이러면 코드를 직관적으로 파악하기 어려워 나누긴했습니다!)
  • 그리고 코드를 단순히 함축해서 한줄로 만드는 것이 좋은지 (저는 조금씩 나눠서 쪼개서 했습니다. 여러줄로)
  • 그리고 CartItme[] 배열에 product 등등이 있는데 이걸 구조 분해 할당하여 쓰는것이 좋은지 아님 cartItem.product.id 이런식으로 쓰면 좋을지!!

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.

1 participant