[Node.js_4기] TIL : 뉴스피드 프로젝트 회고 (24/02/14)

2024. 2. 15. 21:41공부/내배캠 TIL

목차

 

1. 문제

2. 결과

3. 배운점

 

1. 문제 

 

[Node.js_4기] TIL : 뉴스피드 프로젝트 (24/02/07) (tistory.com)

금일 프로젝트 발표를 끝내고 간이 피드백을 받았습니다.

Blue and White Minimal Professional Business Project Presentation - Presentation (canva.com)

 

프로젝트에서 팀원들과 제가 느낀 가장 큰 어려움은 아래와 같았습니다.

1. 모임 추천 기능

  • 반드시 똑같은 키워드를 입력해야 해당하는 모임을 추천받을 수 있음
  • 자연어 처리를 해보고싶었으나 실력의 한계

2. 초기 설계 대비 볼륨이 늘면서 생긴 크고 작은 오류들

  • 프론트엔드 도입 후 변화와 볼륨 증가
  • 프론트엔드와 백엔드 간의 통합 어려움

3. 타인의 코드를 활용하는 것에 대한 어려움

  • 팀원간 협력과 정보 공유를 통해 적극적인 의사소통의 중요성
  • 주석처리를 통해 코드를 읽기 쉽게 만들것

특히, 프로젝트 진행중 api에 맞춰 백엔드 사양을 맞추고 나서 시간이 남아 진행하게된 프론트엔드에 api를 붙이는 작업이 오류가 많았고 그에따라 트러블슈팅 기간이 굉장히 길어져 연휴기간에도 작업을 하게 되었습니다.

 

2. 결과 

 

피드백 내용 

 

잘한것 - 

기본적인 구현사항을 잘 구현해 주었다. 

또, 뉴스피드 프로젝트에 잘 어울리는 주제를 선정하여(모임을 만들어 모임 내에서의 닫힌 커뮤니케이션에) 좋다.

 

보완점 - 

response에 필요하지 않은 내용들이 남아있다던가, 정렬 기능이 부족해보이는 문제가 있다.

불필요할 정도로 api가 깊어진것으로 보인다.  

 

결론 - 

ux에 대한 고민이 엿보이는 프로젝트였다. 요구사항을 잘 구현했고, 추가적인 도전도 잘 구현했다.

그렇지만, 디테일적인 부분에서 부족함이 보였다.

 

 

# 피드백 추가

" 필수 요구 사항 뿐만 아니라 디테일한 부분까지 신경써서 잘 만들어주셨습니다.
도메인도 붙여서 배포해주신 부분 좋습니다만 배포 환경에 대한 테스트를 더 해주셔야될 것 같습니다,
pull_request_template.md를 이용해서 PR 관리를 하신 부분 아주 좋습니다.
S3 활용 좋습니다.

 

.env
.env 각 key에 어떤 값이 들어가는지는 빼고 key만 나열해둔 .env.example이라는 파일을 깃에 올려서 관리하시면 .env에 대한 형상관리도 가능합니다.


routers
routers 폴더에 index.js를 만들어서 라우터들을 정리하면 각 라우터에서 공통적으로 사용되는 경로를 쉽게 파악할 수 있습니다.

 

<나에게의 직접적인 피드백>
users.routes.js
verifyToken을 인증메일 송신 API의 결과로 보내주면 해당 결과를 사용자가 확인할 수도 있습니다. verifyToken은 이메일에 들어가서만 확인할 수 있도록 response에서 제거해주시기 바랍니다. 


community.routes.js
관심사에 따른 추천 로직도 좋은 시도인 것 같습니다. 다만 사용자 테이블에 id는 unique한데 where: { id: +userId }로 findMany() 메서드를 사용하면 배열로 결과가 나올 뿐 한명만 조회가 될 것 같습니다.
user_interest 테이블을 새로 만들어서 관리했으면 어떨까 싶습니다. 아마 조금 더 로직이 간단해졌을 것 같습니다. "

 

3. 배운점 

 

지난 팀과제가 끝나고, 이제 깃을 사용한 협업에 익숙해졌다 생각했지만, 어림도 없지. 바로 충돌 오류로 깃클론부터 다시세팅하기를 세번정도 반복 했습니다.

숙련주차 강의에서 배웠던 token을 이용한 인증 인가를 실제로 구현해보았고, 이메일 인증을 구현하면서 url에 정보를 담아 보내 현재 사용자와 비교하는등의 과정으로 인젝션이 어떻게 일어날 수 있을지도 생각해볼 수 있는 기회가 되었습니다. 

 

결론적으로 또 한번 크게 도약할 수 있었던 프로젝트였습니다.