[django] 1st project review(views/urls)

🖥 1차 프로젝트 진행하면서 받은 피드백 정리

• 1차 프로젝트: fruitz_company
• scope: 카테고리/제품 views/urls

.gitignore
: 예를 들어 my_settings.py 나 data.py 등을 .gitignore에 넣고 PR 올린 다음, pull을 다시 받으면 해당 파일들을 보여지지 않는다.
: 그래서 해당 파일 내용은 따로 빼뒀다가, pull 받은 후에 다시 작성해야 한다. (secret_key는 랜덤으로 사이트에서 get)


migrate
: migrate파일은 서로 dependency 특성이 있어 삭제하면 안된다
: 그래서 보통 migrate 파일만 관리하는 별도의 branch를 만들어놓는다고 한다
: column이 추가되어(모델링에 변경사항이 생겨서) migrate 안될 경우, 일단 default=0 등을 넣어서 추가한 후 migrate 하고 > 다시 default = 0 삭제 > 다시 migrate 하는 것도 방법이다

request.GET.get
:request.GET은 django에서 제공하는 메소드
:python에서 제공하는 get()은 딕셔너리 형태일 때만 사용하지만, request.GET은 딕셔너리가 아니어도 읽을 수 있다. (사전형으로 바꿔주기 때문. request는 딕셔너리 형태가 아니다)

# http://localhost:8000/fbv/?q=100
def fbv(request):
	print(request.GET['q'])
    return HttpResponse('')
#반환값: 100

:get()을 붙여서 request.GET.get()을 사용하면, value값을 찾아줄 수 있고 key가 존재하지 않으면 None을 반환한다
(딕셔너리 자료형에서 data['q'] 형태로 작성하면 해당 key가 없을 때 에러가 뜬다. 하지만 get은 None을 반환)

토큰 유효기간 설정 및 에러 설정
: token을 생성할 때, exp를 추가로 넣으면 유효기간 설정할 수 있음
: https://velog.io/@devmin/JWT-token-expired-date-with-timedelta

BASE_DIR

BASE_DIR = Path(__file__).resolve().parent.parent

• Path(__file__) : settings.py의 절대 경로를 인스턴스화함
• resolve().parent.parent : resovle()를 통해 절대경로를 반환 후, 부모 경로로 2번 옮기면 django project가 생성된 위치가 되고 이것이 BASE_DIR에 담겨짐
  
  

컨벤션
: 데이터를 기준으로 잘 생각하며 엔드포인트를 만들어야.
ProfileCheck > UserView 로 변경
(그래야 사용자 정보 업데이트가 필요할 경우 새로운 class를 만들 필요 없이 하위 method만 추가하면 됨)
: 리소스가 정확히 무엇을 가리키는지 파악하고, 코드를 짜야 한다
예를 들어, 카테고리 별 전체 제품을 조회한다고 했을 때 기준은 Product 클래스가 되어야 하지 Category 클래스가 아니다. 불필요한 내용이 추가되는데다가 가독성도 좋지 않다.

참고 자료:
https://intrepidgeeks.com/tutorial/django-requestgetget