この例ではスレッドロックを使用する必要がありますか？

Question

1つのスレッドでネストされたリストに値を追加し、ネストしたリストをメインスレッドのリスト内包にコピーします。私の例ではスレッドロックを使用する必要がありますか？私はlist.append()メソッドis thread-safeを知っていますが、リスト内包でデータをコピーするときにロックを使用する必要がありますか？

ロックを使用する必要がある場合、copy_data()でロックを使用するのは意味がありますが、GrowList._add_to_list()ではロックされていませんか？

import threading 
import time 

class GrowList(object): 
    def __init__(self): 
     self.data = [] 
     self.stop_event = threading.Event() 
     self.add_to_list() 

    def _add_to_list(self): 
     sample = [1, 2, 3, 4] 
     while not self.stop_event.is_set(): 
      self.data.append(sample) 
      time.sleep(1.0) 

    def add_to_list(self): 
     self.t = threading.Thread(target=self._add_to_list) 
     self.t.start() 

    def stop(self): 
     self.stop_event.set() 
     self.t.join() 


def copy_data(nested_list): 
    return [row[:] for row in nested_list] 

Answer 1

私は他に追加しながら、あなたは一つのスレッド に反復するために、少なくともロックを必要とするだと思う、と私はときにデータをコピーするとき、それはロックを使用する 意味がありませんと思うリストに追加するときには、 ロックを使用しないでください。ロックを使用するポイント は、リストの所有権を主張することです。

あなたが聞くかもしれないもう1つの質問は、ロックを使用して ではないことです。 をself.data = []で置き換えてself.data = MySafeList()と に置き換えるだけで、小さなスレッドセーフリストクラスをロックで個別に作成することで、コードをきれいに保つことができます。 数多くの @synchronizedデコレータの1つを使用して簡単に作成できます。たとえば、あなたの要件が何であるかに依存 が

@synchronized 
def __iter__(self): 
    return iter(list(list.__iter__(self)))