私はきちんとした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
から数字を追い出します。
使用しているものは、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
値のリストを取得します。
リストの補完は機能しますが、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]
_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)
_