副作用のためにリスト内包表記を使用するのはPythonicですか？

戻り値ではなく、副作用のために呼び出す関数について考えてください（画面への出力、GUIの更新、ファイルへの出力など）。

def fun_with_side_effects(x):
    ...side effects...
    return y

さて、リストの内包表記を使用してこの関数を呼び出すのはPythonicですか？

[fun_with_side_effects(x) for x in y if (...conditions...)]

リストはどこにも保存しないことに注意してください

または、このfuncを次のように呼び出す必要があります。

for x in y:
    if (...conditions...):
        fun_with_side_effects(x)

どちらがより良いのですか、そしてなぜですか？

そうすることは非常に反Pythonicであり、ベテランのPythonistaはあなたに地獄を与えるでしょう。中間リストは作成後に破棄されるため、非常に大きくなる可能性があり、そのため作成にコストがかかる可能性があります。

リスト内包表記は使用しないでください。人々が言っ​​たように、必要のない大きな一時リストが作成されるからです。次の2つの方法は同等です。

consume(side_effects(x) for x in xs)

for x in xs:
    side_effects(x)

定義とconsumeからitertoolsmanページ：

def consume(iterator, n=None):
    "Advance the iterator n-steps ahead. If n is none, consume entirely."
    # Use functions that consume iterators at C speed.
    if n is None:
        # feed the entire iterator into a zero-length deque
        collections.deque(iterator, maxlen=0)
    else:
        # advance to the empty slice starting at position n
        next(islice(iterator, n, n), None)

もちろん、後者の方が明確で理解しやすいです。

リスト内包表記は、リストを作成するためのものです。また、実際にリストを作成しているのでない限り、リスト内包表記を使用しないでください。

だから私は2番目のオプションを得るために、リストを反復して、条件が適用されたときに関数を呼び出します

二番目は良いです。

あなたのコードを理解する必要がある人を考えてください。あなたは最初の:)で簡単に悪いカルマを得ることができます

filter（）を使用して、2つの中間に移動できます。例を考えてみましょう：

y=[1,2,3,4,5,6]
def func(x):
    print "call with %r"%x

for x in filter(lambda x: x>3, y):
    func(x)

あなたの目標に依存します。

リスト内の各オブジェクトに対して何らかの操作を実行しようとしている場合は、2番目のアプローチを採用する必要があります。

別のリストからリストを生成しようとしている場合は、リスト内包表記を使用できます。

明示的は暗黙的よりも優れています。シンプルは複雑よりも優れています。（Python Zen）

できるよ

for z in (fun_with_side_effects(x) for x in y if (...conditions...)): pass

しかし、それは非常にきれいではありません。

副作用のためにリスト内包表記を使用することは醜く、Python以外の方法で非効率的であり、私はそれをしません。forループは、for副作用が重要な手続き型の信号であるため、代わりにループを使用します。

ただし、副作用のためにリスト内包表記の使用を強く主張する場合は、代わりにジェネレータ式を使用して非効率を回避する必要があります。このスタイルを絶対に主張する場合は、次の2つのいずれかを実行してください。

any(fun_with_side_effects(x) and False for x in y if (...conditions...))

または：

all(fun_with_side_effects(x) or True for x in y if (...conditions...))

これらはジェネレータ式であり、破棄されるランダムリストを生成しません。allどちらのフォームも混乱するので使用しないでください。

これは醜いと思いますし、実際にコードで行うことはありません。しかし、この方法でループを実装することを主張する場合、それが私が行う方法です。

リスト内包表記とその同類は、機能的なスタイルに少なくともほのかに似ているものを使用する試みを示すべきであると私は感じる傾向があります。その仮定を破る副作用のあるものを置くと、人々はあなたのコードをもっと注意深く読まなければならず、それは悪いことだと思います。