2nd [Hines]💫 4Day

3일차에 올렸던 PR에 받은 리뷰 내용에서

1. 결과 값을 리스트로 감싸서 반환하는 이유는 무엇인가요?
2. product 객체만 사용해서 image_url을 가져올 수 있게 수정해주세요.

• 1번
  : '{ }가 한 개 이상일 경우, [ ]에 감싸서 반환하고
  { }가 한 개일 경우, 바로 { } 형태로 반환한다.'
  라는 것을 구글링을 통해 해결하고,

• 2번
  : 역참조의 개념이 부족하여 추가로 공부 후, product.image_set.all()로 코드를 수정하였다.

위의 리뷰를 반영하여 수정된 코드는 아래(👇)와 같다.

import json

from django.http            import JsonResponse
from django.views           import View

from products.models        import Product

class ProductDetailView(View):
    def get(self, request, product_id):
        try: 
            product = Product.objects.get(id=product_id)

            result = {
                        'name'                : product.name,
                        'price'               : product.price,
                        'brand'               : product.brand,
                        'description'         : product.description,
                        'thumbnail_image_url' : product.thumbnail_image_url,
                        'image_url'           : product.image_set.all()[0].image_url
                    }
                    
            return JsonResponse({'message' : result}, status=200)     

        except Product.DoesNotExist:
            return JsonResponse({'message' : 'PRODUCT_DOESNOT_EXIST'}, status=400) 

---

위의 리뷰를 반영 후에,
오후 수업때 배운 unit test를 작성하는 과정에서 오류를 발견하게 되었다.

작성했던 product의 views.py를 수정해야 할 점을 발견하게 된 것이다.

위의 리뷰를 받아 수정했던 코드에서 product의 image를 한 개만 반환하고 있었던 것이다!!(아니 이게 무슨일이야....?😱😱😱😱😱😱 이 상태서 unit test를 했단 말인가....

그리하여!!
여러 이미지를 담을 수 있게 for문을 사용하여 빈 list[ ]를 만들고,
append() 함수를 이용하는 로직을 작성하기로 했다.
(+ 추가로 list compresion으로 업데이트를 해보기로!)

---

product의 image를 한 개만 담았던 코드를 수정하는 과정을 아래에 작성해보도록 하겠다.

우선 image를 QuerySet을 image_url 이라는 변수에 담는다

image_url = product.image_set.all() #--> QuerySet 형태

빈 list의 변수인 image_result를 지정해 주었다.

image_result = []

그리고,
이제 for문을 통해 빈 list에 append( ) 함수를 이용하여 로직을 작성하면, 아래의 코드(변경 전👇)와 같다.

import json

from django.http            import JsonResponse
from django.views           import View

from products.models        import Product

class ProductDetailView(View):
    def get(self, request, product_id):
        try: 
            product = Product.objects.get(id=product_id)
            image_url  = product.image_set.all() #--> 쿼리셋[리스트형태]
            
            image_result = []
            for image in image_url: #--> 쿼리셋에서 한개씩 꺼내도, 그냥 객체가 꺼내져나옴...
                image_result.append(image)

쉘에서 확인을 해본 결과,
image_url은 QuerySet type으로 나오고
for문을 통해 하나씩 꺼내온 것은 객체이며

위의 내가 작성한 코드는 image_result 변수에 곧바로 객체를 담은 상황이 된것이다.

Json은 객체나 쿼리셋의 type로는 변환시킬 수 없기때문에,
for문을 통해서 image를 Json이 변환이 가능한 [ ] or { } 형태로 만들어줘야 한다는것!

위의 같은 코드로는 for문을 작성했지만,
Json Serisable Error가 발생하게 된 것이다.

그래서,
append( ) 함수를 이용하여 빈 list에 넣어주고 싶은 값은,
한 개씩 나온 그 객체가 가지고 있는 image_url이라는 정보라는 것이다.

위의 사항을 반영하여,
수정한 코드(수정 후👇)는 아래와 같다.

import json

from django.http            import JsonResponse
from django.views           import View

from products.models        import Product

class ProductDetailView(View):
    def get(self, request, product_id):
        try: 
            product = Product.objects.get(id=product_id)
            image_url  = product.image_set.all() #--> 쿼리셋[리스트형태]
            
            image_result = []
            for image in image_url: #--> 쿼리셋에서 한개씩 꺼내도, 그냥 객체가 꺼내져나옴...
                image_result.append(image.image_url)

위의 코드에 추가로
멘토님의

누구나 변수명만 보고도
해당 변수에 어떤 값이 담겼는지 알 수 있게
변수명을 수정하는게 좋을 것 같습니다.

라는 조언을 받아,

`image_url`     --> `images`
`image_result`  --> `image_list`

아래(👇)의 사진과 코드가 변수명에 대한 리뷰를 반영한 것이다.

import json

from django.http            import JsonResponse
from django.views           import View

from products.models        import Product

class ProductDetailView(View):
    def get(self, request, product_id):
        try: 
            product = Product.objects.get(id=product_id)
            images  = product.image_set.all() #--> 쿼리셋[리스트형태]
            
            image_list = []
            for image in images: #--> 쿼리셋에서 한개씩 꺼내도, 그냥 객체가 꺼내져나옴...
            image_list.append(image.image_url)

            result = {
                         'name'                : product.name,
                         'price'               : product.price,
                         'brand'               : product.brand,
                         'description'         : product.description,
                         'thumbnail_image_url' : product.thumbnail_image_url,
                         'image_url'           : image_list
                  }

		return JsonResponse({'message' : result}, status=200)     

	  except Product.DoesNotExist:
      		return JsonResponse({'message' : 'PRODUCT_DOESNOT_EXIST'}, status=400) 
            

---

product datail view를 작성 완료 후에,
다시 unit test를 진행하게 되었다.

해당 unit test code를 작성하는 과정을 모두 정리하지 못햇지만,
내가 실수한 부분, 과정에 발생한 에러, 해결?에 대한 작성은 나의 벨로그 링크(👇)로 첨부하도록 하겠다.

👀 unit test 작성 과정 기록 👀

---

<잠깐... 끄적이는 말...>
uni test code를 작성하면서, 이미지에 대한 for문을 돌리는 이유는 알게 되었다.

위에서 벨로그를 작성하면서 설명했지만,
all()은 QuerySet type이기 때문에 json이 변환을 할 수 없다.

그렇기 때문에 for문을 통해서 객체(인스턴스)를 Json이 변환할 수 있는 형태인 list나 dictionary type으로 바꿔줘야 한다는 것을 배웠다.