ジャンゴ：入力

Question

のレコードのレコードまたはスラグを受け入れるのダックタイピングに優しい方法で私はこのような機能を持っている：bar今

def foo(bar): 
    ... 

は、いずれかのDjangoのレコードまたはレコードを指しているスラグすることができます。実際のレコードがあるときにこの関数を呼び出すことができるように、またはデータベースから取り出した文字列としてしか利用できないより一般的な関数から呼び出すことができるように、この柔軟性が必要です。

私はのような何かを行うことができます実現：

def foo(bar): 
    try: 
     bar.pk 
    except AttributeError: 
     bar = Bar.objects.get(slug=bar) 
    ... 

しかし、これはすべてのでエレガントないないようです。

私はisinstanceを使用しないでください。

Answer 1

あなたはダックタイピングを使用していない定義です。ダックタイピングは、「もしアヒルのように話し、アヒルのように見えたら、それはアヒルです」と言います。

ダックタイピングは、全く異なるクラスの2つのオブジェクトをメソッドに渡し、両方が同じメソッド/属性を実装する（または欠落しているものをうまく処理する）ため、機能することを意味します。これは、メソッドがどのタイプを取得するかについて決して気にしないことを意味します。つまり、渡すオブジェクトには、使用すると予想される属性があります。

場合によっては、オブジェクトを時々渡したい場合や、そのオブジェクトを別の時間にルックアップするために使用できる文字列を渡したい場合があります。これはDuck Typingとは関係ありません。

isinstanceはこれを解決する正しい方法です。この場合、これは問題を解決する最も明白な方法であり、他のものはより複雑になり、理解しにくいという利点はありません。あなたは属性やhasattrに対してtry/exceptを使うことができますが、それは将来の開発者を何か他のものよりも混乱させる可能性があります。 Duck Typingは、さまざまなサブクラスをいくつかの特定の関数にマッチさせる代わりにキャストを置き換えることは素晴らしいですが、この場合はダックタイピングは適用されません。

要するに、ちょうどisinstanceを使用してください。あなたの場合は、それを行うには正しい（ピジョン）方法です。

Answer 2

私はそれがあることを処理する恐ろしい方法であることはよく分からないが、私は似た何かをしたい場合、私はおそらくhasattrを使用します。

def foo(bar): 
    if hasattr(bar,"pk"): 
     bar.pk 
    else: 
     # I include the str in case some other object with a __str__ happens 
     # to come through. 
     bar = Bar.objects.get(slug=str(bar)) 

Answer 3

これは、同じことをしたい他の機能に役立つ別の方法です。あなたが使用しているモデルの名前が「アイテム」であることを告げるつもりです。

def slug_resilient_decorator(class_type): 

    def slug_resilient_wrapper(obj): 

      if obj.has_attr('pk'): 
        return obj 
      else: 
        return class_type.objects.get(slug=obj) 

    return wrapper 

@slug_resilient_decorator(Item) 
def slug_resilient_detail_view(obj): 

    ...