Intro.
프로젝트를 같이 했던 R과 꼭 함께 리팩토링을 하고 싶었는데 결국 혼자 하게 되었다. R에게 DM 보내고 한 달이 꼬박 넘어서야 연락이 닿았는데, 대학원때문에 너무 바빠 도저히 시간이 안 될것 같다고 했다. 대신 너무 고맙게도 어떤 점을 개선해야 할지 같이 리뷰 할 시간을 내주었다. R과의 짧은 미팅으로 정리한 내용은 아래와 같다.
목표와 개선할 점
본격적으로 코드 고치기 전에 가장 큰 목표를 설정하고, 그에 맞춰 개선해야 할 점들을 정리했다.
목표 : 유저가 실제로 사용할 수 있는 서비스 만들기
개선할 점 :
1. 가장 눈에 띄는 부분
-
Testimonial 지우기
Testimonial 자체가 mock data 이기 때문에 삭제할 것 -
Menu bar 오류 해결하기
윈도우 사이즈 줄였다가 다시 늘리면 오류 발생 -
Firebase 로그인 서비스 다시 구현하기
Google에서 블락 당한 것 새로 셋업하기 -
User Profile 개선하기
관심있는 장르와 프로필 이미지를 저장할 수 있는 페이지인데, 다른 유저와 인터랙션 할 수 있는 기능이 아직 없어 무의미함.
유저 입장에서 좀 더 의미있는 페이지로 바꾸기
2. 코드 개선
-
Search page에 Context API 쓰인 부분 정말 필요한지 고민해 보고 개선하기
-
Firebase updated version에 맞게 코드 개선하기 (프로젝트 당시 version 7 → 현재 version 9)
-
저장된 마이 리스트 페이지 개선하기
-
Modal 등 유지보수성 고려해 컴포넌트 분리하기
리뷰 후기 - 반성할 점
리뷰 같이 진행하면서 서로 그 동안 성장했다고 느끼는 동시에 반성도 많이 했다.
서로가 어떻게 코드를 짰는지 자세히 들여다 보지 않았던 것과, 각자의 선택에 대한 충분한 토의가 없었던 게 가장 큰 실수였다는 점에 둘 다 동의했다. 돌아보면 우리가 구현하려고 했던 기능이 일단 동작하기만 하면 너무 기쁜 나머지, ‘이게 과연 좋은 방법인가? 더 개선할 수는 없을까?’ 같은 고민들을 안 했던 것 같다.
항상 유저 입장에서 생각하겠다고 다짐해놓고, 유저 입장에서 전혀 필요하지 않은 기능들을 만든 부분도 반성했다. 그래서 이번에는 리팩토링 버전 배포한 후에 실제 유저 피드백을 받아서 기능들을 더 개선해 나가기로 했다.
