delはリストから何も削除していないようです

Question

for x,y in words: 
    for z in x: 
     if z in stopwords: 
      del x[x.index(z)] 

これは私のコードです。

(list of words, metadata) 

私のコードの目的は、単語のリストからすべてのストップワードを削除することです：言葉のデータはタプルは次のようになりタプルのリストです。 これは唯一の問題は、ストップワードが後で削除されないことです...

私は間違って何をしましたか？ 私はすでに

x.pop(x.index(z)) 

でそれを実行しようとしましたが、それは違いを確認していないようです。例えば

stopwords = set(stopwords) # just so "in" checks are faster 
result = [([word for word in x if word not in stopwords], y) for x, y in words] 

：

>>> stopwords = ['stop'] 
>>> words = [(['hello', 'you', 'stop'], 'somemeta')] 
>>> stopwords = set(stopwords) # just so "in" checks are faster 
>>> result = [([word for word in x if word not in stopwords], y) for x, y in words] 
>>> result 
[(['hello', 'you'], 'somemeta')] 

注あなたは、一般的にリストあなた」を変更しないでください

Answer 1

あなたは、単にネストされたリストの内包表記を使用してストップワードなしで新しいリストを作成することができます繰り返すそれはバグを追跡するのが大変なことにつながります。

Answer 2

for x,y in words: 
    for z in x: 
     if z in stopwords: 
      del x[x.index(z)] 

最も外側のループは、単語リストの1つにxを割り当てます。我々はすぐにyを無視します。 2番目のループはその単語リストを繰り返します。 removing elements from a list you're iterating over causes peculiar behaviour。特定の単語をスキップする可能性があります。これは、del、pop、removeおよびsliceのすべての置換に適用されます。

stopwordsがsetであることを保証し、それに基づいて各単語をフィルタリングすると、内部ループの代わりにx[:] = [w for w in x if w not in stopwords]が効率的になります。ここでのスライス置換は、純粋にxが同じオブジェクトのままであることを保証するためのものであり、この場合はwords内のエントリを確実に変更します。 これは、前述の反復問題には実行されません。なぜなら、リストの理解は、割り当てによってスライスに格納される前にそのリストを構築するためです。