DRFこのクエリを最適化するにはどうすればいいですか？

Question

私は、心の中でクエリを持っているDjangoのORMとシェルで期待どおり、現在それが動作します：

>>> Place.objects.all()[0].images.filter(order=0)[0].filename 
'y5IUMPyv.jpg' 

しかし、私はDRFのSerializerMethodFieldでそれを実装する方法がわかりません。これは、私はそれを把握することができるまで私はその間に使用しているものです：

class CardSerializer(serializers.Serializer): 
    image = serializers.SerializerMethodField() 

    @staticmethod 
    def get_image(obj): 
     for d in obj.images.all(): 
      if d.order == 0: 
       return d.filename 

これは動作しません。「理想的な」方法であり、私にはわからない理由：

@staticmethod 
    def get_image(obj): 
     return obj.images.filter(order=0)[0].filename 

---

Traceback (most recent call last): 
    File "/home/admin/env/lib/python3.4/site-packages/django/core/handlers/base.py", line 149, in get_response 
    response = self.process_exception_by_middleware(e, request) 
    File "/home/admin/env/lib/python3.4/site-packages/django/core/handlers/base.py", line 147, in get_response 
    response = wrapped_callback(request, *callback_args, **callback_kwargs) 
    File "/home/admin/env/lib/python3.4/site-packages/django/views/decorators/csrf.py", line 58, in wrapped_view 
    return view_func(*args, **kwargs) 
    File "/home/admin/env/lib/python3.4/site-packages/django/views/generic/base.py", line 68, in view 
    return self.dispatch(request, *args, **kwargs) 
    File "/home/admin/env/lib/python3.4/site-packages/rest_framework/views.py", line 466, in dispatch 
    response = self.handle_exception(exc) 
    File "/home/admin/env/lib/python3.4/site-packages/rest_framework/views.py", line 463, in dispatch 
    response = handler(request, *args, **kwargs) 
    File "/home/admin/env/lib/python3.4/site-packages/rest_framework/generics.py", line 201, in get 
    return self.list(request, *args, **kwargs) 
    File "/home/admin/env/lib/python3.4/site-packages/rest_framework/mixins.py", line 45, in list 
    return self.get_paginated_response(serializer.data) 
    File "/home/admin/env/lib/python3.4/site-packages/serpy/serializer.py", line 134, in data 
    self._data = self.to_value(self.instance) 
    File "/home/admin/env/lib/python3.4/site-packages/serpy/serializer.py", line 123, in to_value 
    return [serialize(o, fields) for o in instance] 
    File "/home/admin/env/lib/python3.4/site-packages/serpy/serializer.py", line 123, in <listcomp> 
    return [serialize(o, fields) for o in instance] 
    File "/home/admin/env/lib/python3.4/site-packages/serpy/serializer.py", line 107, in _serialize 
    result = getter(self, instance) 
    File "/home/admin/src/places/serializers.py", line 72, in get_image 
    return obj.images.filter(order=0)[0].filename 
    File "/home/admin/env/lib/python3.4/site-packages/django/db/models/query.py", line 297, in __getitem__ 
    return list(qs)[0] 
IndexError: list index out of range 

Answer 1

アンダーソン・リマ@アイアンメイデンが指摘したように、これはあなたのデータに問題はなく、あなたのコードです。 order = 0の画像はありません。最初の方法では、最適でない場合は、その状況を正しく処理します。

@staticmethod 
    def get_image(obj): 
     for d in obj.images.all(): 
      if d.order == 0: 
       return d.filename 

     # returns None here if an object with order = 0 
     # does not exist in the database. 

しかし、2番目のアプローチでは、スライスを取っていますが、存在しないオブジェクトを取得しています。したがって、執行は、あなたが必要とするものがブロックを除いてちょうど試練であることを私たちに伝えます。

@staticmethod 
def get_image(obj): 
    try: 
     return obj.images.filter(order=0)[0].filename 
    except IndexError: 
     return None