February 10, 2022

👩🏻‍💻 ****과제 리뷰

json-server를 활용하지 않고 axios를 사용함

• 요구사항을 숙지하지 못함

CSS 디테일이 떨어지는 경우 (감점사항)

• 여백, 체크박스 네모의 크기, cursor:pointor

컨벤션, 협업 규칙을 정하지 않은 경우

• type 정하기, api 정하기 ⇒ 무조건 팀원과 같이 해야함, 커뮤니케이션 강조
• https://github.com/Free-Onboarding-team4/estimate-board ⇒ 리드미 참고 👀

TypeScript 아쉬운점

• union, optional 너무 많이사용 ⇒ 자유도 높아짐 ⇒ 안좋음🚨
• 타입을 시작할때부터 정의를 잘하고 가는게 좋다
• 코드 중복 ⇒ 협업이 잘 안된다고 생각함 ❌
• 하나에서만 쓰는 props type을 공용 파일에 정의하는 경우
  • 응집도를 최대한 생각하는게 좋음
  • 한 파일 안에서만 쓰이면 따로 분리하지 않아도 좋다

좋은코드 작성하기

• 의도가 드러나는 이름(변수명)을 작성하자
• IProps(요즘은 유행 지남) ⇒ 팀 안에서 정하는 컨벤션이 가장 높은 우선순위
  • 요즘에는 MaterialProps 등으로 사용
• hooks를 쓰려는 노력

💁🏻‍♂️ 멘토 가이드