발표 피드백

20230417 장고 프로젝트 발표 피드백

 

공통사항

• 테스트할 때 사용한 코드는 삭제하는 것을 권장
• 인덴트 안맞는 부분 수정
• 시크릿키는 깃허브에서 관리하지 않도록
• url나눌때 post-delete, post-update 이런식으로 나눌 필요 X
  http메소드로 구분
• 사용되지 않는 코드, 폴더, 이미지들 삭제
• 버전 관리 확실하게 하자(Semantic versioning) ex)v.1.0.0
• try except 구문을 잘 사용할 것
• 메인 브랜치에는 최종 코드만 올리는 것이 좋다(충돌 방지 목적)

1조

• 조회수 기능 구현할때, 세션을 활용해 중복으로 처리 안되게 한 것 좋았다.
  하지만 우회가능
• 필드 네이밍 시 중복되는 사항들은 제외
  post 테이블 안에 굳이 필드명을 post_title로 할 필요없이, title로 하면 된다.

2조

• 검색기능 코드를 보니까 ==으로 비교하고 있다.
  검색한 기능과 완전히 일치해야만 뿌려주는데, 포함했을때도 띄워주면 좋겠다.
• 테이블에 id를 직접추가해줄 필요는 없다. 자동적으로 장고에서 생성해준다.
  pk를 직접 추가하는 방식은 사용하지 않는 것이 좋다. 에러가 발생할 우려가 있다. 

3조

• 이메일 검증 과정이 프론트에만 있다. 누군가 악의적으로 데이터를 날리면 통과될 것이다.
  밸리데이션 할때는 프론트 뿐만아니라 백엔드에서 리젝시키는 과정도 중요하다.
• 게시글 하나를 보는데 요청을 3번이나 하고있다. (이전게시글, 다음게시글을 표시하기 위해)
  데이터가 많아지면 데이터 부하가 생길 수 있다.

4조

• 시크릿 키 뿐만 아니라 유튜브api 엑세스 토큰도 GIT에 올라가지 않도록 하는 것이 좋다.
• 이미지 저장 FileSystemStorage 장고에서 알아서 해준다. 
• 깃허브 커밋메시지 신경을 잘 써주는게 좋다(컨벤션 정하는 거 좋다)
• 도커 강의에서 한달뒤에 스태틱과 css를 어떻게 배포해야할지에 대해 배울 것이다. 
  디버그 플래그에 따라서 다르게 설정해야한다.

5조

• qquery?, __contains 활용한 점 좋다 
• python_posts = Post.objects.filter(title__contains='python')
• 템플릿 태그가 10개를 넘게 타고 들어간다. 무한 div 줄이는 것이 좋다.

6조

• 공통사항들

7조

• 네이밍이 직관적이지 않다.
• 유저 마이페이지 구현할 때, id를 받을 필요없다.
  그냥 로그인한 유저의 데이터를 보여주면 된다. (user = request.user)

8조 

• 추가기능중 심화내용에서 배울 내용이 있으니 지금 미리 할 필요까지는 없다. 
• if문에서 바다코끼리( := )같은 최신 문법 사용하는거 좋다.
  최신 문법 사용 시 어떤 버전부터 사용가능한지 파악 후 쓰는 것이 좋다.
• 네이밍 컨벤션 안맞는 부분 있다.

9조

• 게시글 삭제 기능 프론트에서는 검증하나, 백엔드에서 실제로 검증하는 과정이 없다.
  이런 로직은 프론트와 백엔드에 둘다 있어야한다.
• 게시글 카운트할 때, 변수 새로 만들지 말고 역참조로 한번 해보는 것도 좋을 것 같다.