[원티드 프리온보딩] 사전과제 TODO 코드 리뷰 후기

 

과제 깃허브 레포지토리

GitHub - so0112/wanted-pre-onboarding-frontend: 투두리스트

 

사전과제 배포사이트(현재 api 배포 중단으로 작동 x)

Todo

 

과제를 진행하며 전체적으로 느낀점

기간내에 빠르게 동작하는 TODO LIST를 만드려다보니 디렉토리 및 파일구조, 리다이렉션 로직,

중복된 코드패턴이 꽤 많이 들어갔다. 다른 팀원분들께 피드백을 받아보니 그런 부분이 더욱 두드러졌다.

안티패턴으로 작성된 코드 -> 피드백 받았던 내용순으로 정리해보자면

• map, filter 메서드를 사용할때 element의 변수명을 el로 사용
  (피드백) el보다는 어떤 값이들어오는지 작성할 수 있도록 변수명 작성하기
  
  
• getTodo에서 datas라는 state의 사용
  (피드백) datas라는 단어는 없을뿐더러 어떤 값이들어오는지 모르니 유의미한 state명이 필요
  
  
• api의 인수로 어떤 타입의 값이 들어오는지 파악이 어려움
  (피드백)타입스크립트로 리팩토링해보기
  
  
• 기능별로 파일 분리는 잘 된 것 같은데 디렉토리가 너무 많고 난잡
  (피드백)기능별로 분리를 하되 직관적인 구조로 수정하기
  
  

이외에도 여러가지 피드백을 받았는데 내가 작성할 때는 잘 보이지 않았던 것들이
팀원분들께서 코드를 보며 확인을 해주시니 수정해야할 부분들이 확실히 들어왔다.
코드리뷰를 받고 리팩토링을하는 것이 실력 향상에 도움이 되겠다는 생각이 들었고
그리고 같은 과제에 대해 다른 사람들은 어떻게 코드를 작성했나 볼 수 있는 점이 정말 좋았다
한 달 짜리 짧은 과정이지만 원티드 프리온보딩을 통해 실력향상에 큰 도움이 될 수 있을 것 같다.
아직 부족한 실력이지만 이번 프리온보딩을 발판 삼아 더 실력을 쌓을 수 있도록 열심히 해야겠다.