dreamherb/node.js_secondweek

node.js_secondweek's Issues

안녕하세요! 과제 코멘트 남겨드리고 갑니다!

안녕하세요! 항해99 2기 수료생 이용우입니다. 서비스 잘 보고 갑니다.

몇 가지 궁금한 점이 있어 질문드립니다.

1. Router에 접근하기 위한 URI를 Restrul API에 맞게 구성하면 좋을 것 같습니다.

   • URI만으로 접속한 페이지에서 어떠한 행동을 할 지 예측하기 힘든 것 같습니다.

   // postings.js
   // Before
   router.post("/newPostings", async (req, res) => { }
   router.get("/showPostings", async (req, res) => { }
   router.put("/updatePostings", async (req, res) => { }
   
   // After
   router.post("/posts", async (req, res) => { }
   router.get("/posts", async (req, res) => { }
   router.put("/posts", async (req, res) => { }

   • 참고: [RESTful API 설계 가이드](https://sanghaklee.tistory.com/57)

2. 예외 처리되지 않은 라우터가 많은 것 같습니다.

   • 예외 처리되지 않은 라우터에서 에러가 발생할 경우 서버가 멈출 수 있습니다.
   • 에러 상황은 언제든지 발생할 수 있으므로 예외 처리는 필수적으로 작성되어야 합니다.

3. /signin 페이지에 접속 되지 않습니다.

   • URI가 존재하지 않는다는 에러 메시지가 반환되고 있습니다.
   • 회원가입을 성공했을 경우 이동하는 페이지의 URI와 Router에 작성한 URI가 일치하지 않는 것 같습니다.

4. .gitignore를 이용하여 실행에 필요없는 정보는 예외 처리하면 좋을 것 같습니다.

   • node_modules폴더는 Pacakge.json 파일로 다운로드 받을 수 있으므로 삭제해도 무방합니다.
   • 참고: .gitignore-적용하기

과제 수행하시느라 고생많으셨습니다!

안녕하세요 잘보고갑니다!

안녕하세요 항해3기 황창환입니다.

부족하지만 조금이나마 도움 드리고자 코멘트 작성해봅니다!

1. res.send 와 json을 혼합해서 사용하고 계신데 객체로 response를 작성할 거라면 send보다는 json을 쓰는게 조금 더 좋지 않을까라는 생각이 듭니다!
   (이유는 아래 참고를 보시면 조금 도움이 될 것 같습니다.)

참고: https://haeguri.github.io/2018/12/30/compare-response-json-send-func/

2. try/catch를 이용하여 에러핸들링을 하면 조금 더 좋은 코드를 짤 수 있습니다!
   참고:try/catch
   참고:express error-handling

3. api end-point를 REST API를 참고하여 작성한다면 조금 더 좋지 않을까 생각이 듭니다 :)

4. 회원가입 성공시에 404 에러가 뜹니다.. 경로 수정해주시면 좋을것 같아요!

5. 로그아웃도 구현해주시면 좋지 않을까 싶습니다 (굉장히 쉬우니 한번 혼자 생각해보심이 좋을것 같습니다~)