프로젝트 피드백 - 작성자: 정영훈 튜터

[minzunim]

Overview

git

• 머지가끝난 브랜치들은 삭제해주세요~
• config.json 에 "host": "express-database.cg21gzirpdhp.ap-northeast-2.rds.amazonaws.com", "password": "wnghqkd726" 이러한 db정보가 있으면 안됩니다. config.json 을 js 파일로 바꾸고 .env를 활용해서 바꾸주세요. 4조가 후발대에서 알려드린 방법대로 잘 사용하셔서 링크 - https://github.com/muja-code/closer4u_service/blob/main/config/config.js 참고 하시면 좋습니다. => models/index.js 에서 config.json -> config.js 로 변경해야 정상작동합니다.(링크2 9번라인 참고 - https://github.com/muja-code/closer4u_service/blob/main/models/index.js)
• README.md 파일 만들어서 프로젝트소개, 폴더구조, 실행법, api명세, erd 올려주면 좋습니다.

프로젝트 결과물 정리본

• 권한이 없어서 ERD, API 는 생략하겠습니다. 요청드렸으니 나중에 확인해주세요.

코드리뷰

• 아래에 작성, 메인브랜치가 최신인거 같아서 메인브랜치 기준으로 남겨드립니다.

[minzunim]

controllers

https://github.com/Joto-Han/3week_project/blob/bf983e178a95d32a60191ed55b7b8a08a1e4e8e3/controllers/wash_add.controller.js#L5

사용하지 않는 매개변수 next 삭제

https://github.com/Joto-Han/3week_project/blob/bf983e178a95d32a60191ed55b7b8a08a1e4e8e3/controllers/wash_add.controller.js#L5-L19 washAdd 및 다른 컨트롤러, 서비스, 레파지토리 메서드들 모두 예외처리(try catch)가 필요합니다. 다른조의 소스 참고해서 예외처리 해주세요. 참고

• https://github.com/muja-code/closer4u_service/blob/main/controllers/orders/orders.js
• https://github.com/muja-code/closer4u_service/blob/main/services/orders/orders.js
• https://github.com/muja-code/closer4u_service/blob/main/repositories/orders/orders.js

[minzunim]

services

https://github.com/Joto-Han/3week_project/blob/bf983e178a95d32a60191ed55b7b8a08a1e4e8e3/services/login_shop.service.js#L13-L16

main브랜치에선 이러한 주석 및 console.log들을 제거해주세요.

https://github.com/Joto-Han/3week_project/blob/bf983e178a95d32a60191ed55b7b8a08a1e4e8e3/services/register_shop.service.js#L12 shop_id가 .null로 가져올 수 있나요? user_id나 다른부분도 id값이 잘 가지고 와지는지 테스트해보세요~

[minzunim]

routes

https://github.com/Joto-Han/3week_project/blob/bf983e178a95d32a60191ed55b7b8a08a1e4e8e3/routes/login_shop.routes.js#L4

사용하지 않는 변수들 jwt, authmiddleware, Op, User 제거해주세요~

[minzunim]

migrations

https://github.com/Joto-Han/3week_project/blob/bf983e178a95d32a60191ed55b7b8a08a1e4e8e3/migrations/20221230073005-create-user.js#L6

models 에보면 user모델 id는 user_id 입니다. 마이그레이션에 user update 추가해서 id -> user_id로 변경해주세요.