web-dev-qa-db-ja.com

インラインforループ

私はきちんとしたPythonのやり方を学ぼうとしていますが、なぜ私のforループをこのようにリファクタリングできないのか疑問に思っていました:

q  = [1, 2, 3, 4, 1, 2, 5, 1, 2, 3, 4, 5]
vm = [-1, -1, -1, -1]

for v in vm:
    if v in q:
        p.append(q.index(v))
    else:
        p.append(99999)

vm[p.index(max(p))] = i

Forループを次のように置き換えてみました:

[p.append(q.index(v)) if v in q else p.append(99999) for v in vm]

しかし、それは機能しません。 for v in vm:ループは、vmで次に来るときに基づいて、qから数字を追い出します。

13
Will

使用しているものは、Pythonで list comprehension と呼ばれ、インラインforループではありません(1つに似ていますが)。ループは、リスト内包表記として次のように記述します。

_p = [q.index(v) if v in q else 99999 for v in vm]
_

リスト内包表記を使用する場合、リストは内包表記自体から作成されるため、_list.append_を呼び出しません。リスト内の各項目は、forキーワードの左側の式(この場合はq.index(v) if v in q else 99999)によって返されるものになります。内包的に、内包内で_list.append_を使用すると、Noneメソッドが常に返すものであるため、append値のリストを取得します。

33
iCodez

enumerate を使用できます。エレメントのind/indexをvmに保持します。vm aを作成すると set0(1)ルックアップ:

vm = {-1, -1, -1, -1}

print([ind if q in vm else 9999 for ind,ele in enumerate(vm) ])
2

リストの補完は機能しますが、appendはNoneを返すためNoneのリストを返します:

デモ:

>>> a=[]
>>> [ a.append(x) for x in range(10) ]
[None, None, None, None, None, None, None, None, None, None]
>>> a
[0, 1, 2, 3, 4, 5, 6, 7, 8, 9]

このように使用するより良い方法:

>>> a= [ x for x in range(10) ]
>>> a
[0, 1, 2, 3, 4, 5, 6, 7, 8, 9]
2
Hackaholic
_q  = [1, 2, 3, 4, 1, 2, 5, 1, 2, 3, 4, 5]
vm = [-1, -1, -1, -1,1,2,3,1]

p = []
for v in vm:
    if v in q:
        p.append(q.index(v))
    else:
        p.append(99999)

print p
p = [q.index(v) if v in q else 99999 for v in vm]
print p
_

出力:

_[99999, 99999, 99999, 99999, 0, 1, 2, 0]
[99999, 99999, 99999, 99999, 0, 1, 2, 0]
_

リスト内包表記でappend()を使用する代わりに、pを直接出力として参照し、LCでq.index(v)および_99999_を使用できます。

これが意図的なものであるかどうかはわかりませんが、q.index(v)vに複数ある場合でも、qの最初の出現を検出することに注意してください。 v内のすべてのqのインデックスを取得する場合は、enumeratorと既にアクセスしたindexesのリストの使用を検討してください

それらの行にあるもの(擬似コード):

_visited = []
for i, v in enumerator(vm):
   if i not in visited:
       p.append(q.index(v))
   else:
       p.append(q.index(v,max(visited))) # this line should only check for v in q after the index of max(visited)
   visited.append(i)
_
1
f.rodrigues