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

[changchanghwang]

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

별건 아니지만 조금 도움이 되고자 코멘트 달아봅니다!

1. myblog/schemas/index.js

   Lines 10 to 15 in 61f28df

   	// //스키마 생성
   	// const mongoose = require("mongoose");
   	// const connect = () => {
   	// mongoose.connect("mongodb://bgg01578:wngus4582@localhost:27017/my_blog?authSource=admin", { ignoreUndefined: true }).catch(err => console.log(err));
   	// };//mongoose를 mongoDB와 연결해주어야한다

   
   위 코드에 주석처리되어있는 부분은 보안상 지우는게 더 좋지 않을까 싶습니다 :)

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

3. res.send와 res.json을 혼용해서 사용하시고 계신데 어차피 객체로 response를 보낸다면 res.json이 조금 더 좋은 선택이 될 수 있습니다.

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

4. myblog/routes/articles.js

   Line 39 in 61f28df

   const [article] = await Article.find({ _id: articleId });

   
   이 코드같은 경우에 한가지 객체만 찾는거라면 findOne 메서드를 사용하는것이 조금 더 좋지 않을까 싶습니다!

5. myblog/routes/users.js

   Line 41 in 61f28df

   const token = jwt.sign({ username }, "secret-juhyeon");

   
   위 코드같은 경우에 secret key 같은 경우 보안상으로 올라가면 좋지 않기때문에 환경변수를 이용하면 더 좋지 않을까 생각이 듭니다 :)
   (참고:https://www.npmjs.com/package/dotenv)

과제 너무 고생하셨고 남은 항해기간도 화이팅하세요 :)