2nd [Hines] 💫9Day

어제 추가로 프론트분께서 상품 디테일 정보를 2가지 추가 요청이 왔다.
카테고리 이름과 서브 카테고리 이름 정보도 요청하셔서 오늘 아침에 이전 코드에 추가 및 postman으로 확인해본 결과이다(+ Json으로 넘겨주는 key값이 "message"로 되어 있어서, "result"로 명칭도 같이 수정하였다.")

---

원래 수요일 오전 10시반에 예정이엿던 코드리뷰가 오늘 오후 2시반으로 앞당겨졌다.

라이브 코드 리뷰 전까지 작성한 부분은
<상품 상세페이지 기능>, <리뷰 등록 기능>, <리뷰 정보 불러오기 기능 작성중>...상태엿고,
아직 리뷰 정보 불러오기 기능 구현 중에서 5개씩 리뷰 전달하는 기능을 구현하지 못해서 이번 라이브 코드 리뷰 시간에 <상품 상세페이지 기능>과 <리뷰 등록 기능에 부분>만 리뷰를 받게 되었다.

우리 팀원의 <장바구니>, <주문 기능>과 나의 나머지 담당 부분인 <리뷰 정보 불러오기>는 목요일에 다시 라이브 코드 리뷰를 받기로 했다.

오늘 진행한 라이브 코드 리뷰에 대해서 다시 정리를 해보려고한다.
(팀원분들이 담당한 기능들은 나중에 다시 코드를 뜯어보면서 정리해보기로 하자)

1. products app의 veiws.py : ProductDetailView

1. 불필요한(미사용) import 삭제
   [수정 전]
   

   [수정 후]
   

1) for문 -> list comprension 적용해보기
2) 불필요한 공백(tab) 삭제
3) (+ 프론트의 요구 : sub_category_name과 category_name 추가하기,
수정 필요 : 기존의 'name' key -> 'product_name' key 수정,
json key 수정 : 기존의 'message' -> 'result' 수정)

[수정 전]

[수정 후]

---

2. products app의 veiws.py : ReviewView

1. decorator의 error message 중복 삭제 필요

[수정 전]

[수정 후]

애초에 login decorator를 사용하여 유저가 로그인을 성공하지 않는다면,

if not User.objects.filter(id=request.user.id).exists():
    return JsonResponse({'message':'UNAUTHORIZED_USER'}, status=401)

위에 작성한 class의 함수를 실행될 수가 없다.

왜??

:: login decorator 부분에서 로그인 정보가 없는 유저를 먼저 걸러주기 때문이다.
(그렇기 때문에, 로그인을 하지 않은 유저의 정보로는 ReviewView의 함수가 실행 될 수가 없을 뿐더러, 유저 여부에 대한 조건과 메세지는 Review view에서는 작성할 필요가 없다.)

아마,
login decorator를 다른 팀원이 구현중이여서...
해당 리뷰 기능 구현할때,
"유저 정보가 없는 경우에는??" 이라는 물음표가 생겨서 작성한 것같다..

하지만,

login decorator의 작성 의도를 정확히 알고 있었다면,
불필요한 저 코드를 작성하지 않았을 거라는 생각이 든다... (반성...)
1차때 login decorator를 작성해보았음에도 불구하고 이런 코드를 작성했다는 점을 반성의 필요성을 느낀다...

2. get() -> filter() 수정

[수정 전]

if not Product.objects.get(id=product_id):
    return JsonResponse({'message':'NO_PRODUCT'}, status=400)

[수정 후]

if not Product.objects.filter(id=product_id).exitsts():
    return JsonResponse({'message':'NO_PRODUCT'}, status=400)

• get()과 filter().exists()

  • 방법1) objects.get() 사용시 데이터가 없는 경우, exception(예외)처리 필요
    => 해당 조건문의 반환값을 반환하지 못하고, 예외 발생
    (즉, 지금 작성한 if 조건문을 작성한 의미 X)
  • 방법2) if 조건문 사용 -> 예외가 발생하지 않게 조건 작성 필요

  <결국, if문을 작성하던지, execpion 처리 하던지 어느 방법이든 무관>

• 해당 조건을 작성했던 이유는 단순하게,

  "만약에
  프론트단에서 등록되지 않은 product_id값이 아닌 정보를 전달해줄 경우가 혹시라도 발생하면,
  그 에러처리는 따로 안해줘도 무방한가...?"

  라는 생각으로 작성하게 되었다.

  이렇게 작성하는게 맞는건지 동기 몇명께 여쭤보았을땐,
  "없는 product는 리뷰를 작성도 못할텐데 저 조건을 작성하는게 말이 안되는 것 같다...?"라는 류의 답변을 받았다.
  나도 이런 생각을 가져보기도 했기도 했고, 작성한 코드에 대해서 확신을 가지고 작성한게 아니여서
  멘토님께 이 조건식을 작성하는게 맞는건지 궁금해서 여쭤보기로 했다.

---

내일은
오늘 코드가 추가된 ProductDetailView의 unit test까지 완료하고,
ReviewView의 get()메소드의 offset:limit을 추가시켜보기로...

우선, 기업협업 이력서를 완료시키고 자는걸로...