bgg01555 / myblog Goto Github PK

article_detail 프론트 이용해서 글쓰기 프론트 부분 구현하기

안녕하세요 항해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)

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

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

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

1. 코드 스타일이 일관적이지 않아 가독성이 떨어지는 것 같습니다.
   • ESlint를 이용하여 코드 포맷팅을 맞추면 가독성이 좋을 것 같습니다.
   • VSCode나 WebStorm에서 자동 포맷팅 설정을 할 수 있으므로 설정하는 것을 고려하면 좋을 것 같습니다.
   • 참고: ESlint 시작하기, VSCode ESlint 설정
2. 예외 처리되지 않은 라우터들이 많은 것 같습니다.
   • 예외 처리되지 않은 라우터에서 에러가 발생할 경우 서버가 멈추거나, 종료될 수 있습니다.
   • 에러 상황은 언제든지 발생할 수 있으므로 예외 처리를 추가하면 좋을 것 같습니다.
   • 참고: expressjs Exception
3. .gitignore를 이용하여 js, .vscode 폴더와 같이 업로드 할 파일을 예외처리할 수 있습니다.
   • 프로젝트 실행에 필요하지 않는 정보들을 굳이 github에 업로드할 필요는 없습니다.
   • .gitignore를 이용하여 업로드 될 파일을 예외처리할 수 있습니다.
   • 참고: .gitignore-적용하기]
4. Router에서 Render를 하기위한 로직이 너무 많은 것 같습니다.
   • 현재는 ( 페이지 접속 → DB 검색 → Render)와 같이 구성된 것 같습니다.
   • 이후 프로젝트를 진행할 때 프론트 엔드와 협업하게 될 경우 동일한 방법을 사용하기 힘들 수 있습니다.
   • views/*.ejs 파일들에서 (Render → Javascript로 Router 호출 → DB 검색 → 반환된 정보를 .ejs 파일에 반영) 과 같은 로직을 사용하는것이 좋을 것 같습니다.

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

상세 페이지 이동시 글쓰기 버튼 숨기기

비밀번호 암호화 해서 저장하기