リストから重複を削除するより速い方法

Question

APIを使用してWebサイトからチケット番号をダウンロードし、作成した一意のチケットIDを1つのIDインスタンスのみを受け入れるようカウントします。

これは非常に遅いです...私はセットを使用しようとしましたが、それは悪化しています。私はitertoolsなどの他のツールについては言及していますが、私のような状況でどのように使用できるかについて明確な説明はありません。

良い変数と悪い変数は、私が知ることのできるジェネレータのタイプを格納します。あなたの助けをありがとう！ listとしてgood_arrayと

good = self._client.satisfaction_ratings(score="good",sort_order="asc",start_time=thirty_days,end_time=today) 
    bad = self._client.satisfaction_ratings(score="bad", sort_order="asc", start_time=thirty_days, end_time=today) 
    good_array = [] 
    bad_array = [] 
    for g in good: 
     if g.ticket.id not in good_array: 
      good_array.append(g.ticket.id) 
    for b in bad: 
     if b.ticket.id not in bad_array: 
      bad_array.append(b.ticket.id) 
    goodnum = len(good_array) 
    badnum = len(bad_array) 

Answer 1

if g.ticket.id not in good_arrayは、それ以降のすべての権利は、線形検索を使用して低速である（と私は順序は重要ではありません理解）setを使用して

だけのユニークな要素を維持するのに最適な方法である（と

： が遅くなることはないことを、あなたは

今、ちょうどループを削除し、2セットの内包表記を作成する）間違った方法でそれらを使用している必要がありますあなたが行ったように

good_array = {g.ticket.id for g in good} 
bad_array = {g.ticket.id for g in bad} 

その後、カウント：

goodnum = len(good_array) 
badnum = len(bad_array) 

Answer 2

セットは、重複を許可しないのpythonのデータ型です。 は、単にあなたの良いと悪いリストを設定するにはキャスト -

unique_good = set(good) 
unique_bad = set(bad) 

あなたはあなたがリストのためになったのと同じ方法でセットの長さを得ることができます。