forループの代わりに補題を使用する

Question

以下は、私のコードの簡単な例です。

>>> def action(num): 
     print "Number is", num 
>>> items = [1, 3, 6] 
>>> for i in [j for j in items if j > 4]: 
     action(i) 
Number is 6 

私の質問は以下の通りです：は、単にまだaction関数を呼び出します理解してforループを交換する（このようなコードの明確さなどの理由で）悪い習慣ですか？つまり：

>>> (action(j) for j in items if j > 2) 
Number is 6 

Answer 1

これは、発電機または理解度を全く使用しないでください。

def action(num): 
    print "Number is", num 

items = [1, 3, 6] 
for j in items: 
    if j > 4: 
     action(i) 

ジェネレータは遅延して評価されます。式(action(j) for j in items if j > 2)は単にジェネレータ式を呼び出し側に返します。あなたが明示的にそれを使い切っていない限り、何も起こりません。リスト内包表記は熱心に評価されますが、この特定のケースでは、目的のないlistが残ります。ちょうど通常のループを使用してください。

Answer 2

私は個人的にはTigerhawk's solutionですが、彼とwillywonkadailyblahの解決策（今削除されています）の中間になるかもしれません。

willywonkadailyblahのポイントの1つであった：

なぜだけではなく、古いものを使用しての新しいリストを作成しますか？あなたはすでに正しい要素を除外するための条件を持っています。なぜそれらをメモリに入れて戻すのですか？

この問題を回避する1つの方法、すなわちフィルタリングはジェネレータ式のフィルタリング一部ではなく、リスト内包することによって、ループのために使用した反復処理する場合にのみ行っているフィルタリングの遅延評価を使用することです。

すべての正直で出力

Number is 6 

for i in (j for j in items if j > 4): 
    action(i) 

、私はTigerhawを考えますしかし、kの解がこれには最高です。これはただ一つの可能​​な選択肢です。

私が提案した理由は、C＃でLINQクエリを思い出させるため、1つのステートメント（LINQ式）内のシーケンスから要素を抽出、フィルタリング、投影するという怠惰な方法を定義し、そのクエリーで別のfor eachループを使用して、各要素に対して何らかのアクションを実行します。

Answer 3

これは悪い習慣です。第1に、あなたのコード断片は、望ましい出力を生成しません。代わりに、<generator object <genexpr> at 0x03D826F0>のようなものが得られます。

第2に、リストの理解はシーケンスを作成するためであり、ジェネレータaはオブジェクトのストリームを作成するためです。典型的には、副作用はない。あなたのアクション関数は、副作用の主要な例です。それは入力を出力し、何も返しません。むしろ、ジェネレータは生成する各アイテムに対して、入力を取り、出力を計算する必要があります。例えば。あなたがグローバル状態（何かを印刷する）を変異させ、およびオブジェクトのストリームを作成することはなく、あなたのコードの目的は、難読化されている発電機を使用することにより

doubled_odds = [x*2 for x in range(10) if x % 2 != 0] 

。 一方、forループを使用するだけで、コードはわずかに長くなります（基本的には空白になります）。しかし、すぐに目的は（アイテムの新しいストリーム/リストを作成するのではなく）アイテムを選択して、 。

for i in items: 
    if i < 4: 
     action(i) 

発電機はまだループ構造、基礎となるバイトコードが同じ多かれ少なかれあること（どちらかといえば、発電機はわずかに効率が低い）、とあなたは明快さを失うされていることに注意してください。ジェネレータとリスト内包はすばらしいですが、これは正しい状況ではありません。