Pythonでスレッドとキューを使ってcaesar-cipherを実装するプログラムを作成しました。私は自分のコードでマルチプロセッシングを使ってすべてのスレッディング作業を変更したいと思います。実装を開始する方法については、&の説明ができれば幸いです。ここでは、コードは次のようになります。コード(Python)でマルチプロセスを使ってマルチスレッドを変更するには
import threading
import Queue
import sys
import string
lock = threading.Lock()
def do_work(in_queue, out_queue, shift):
while True:
lock.acquire()
item = in_queue.get()
result = caesar(item, shift)
out_queue.put(result)
in_queue.task_done()
lock.release()
def caesar(plaintext, shift):
plaintext = plaintext.upper()
alphabet = string.ascii_uppercase
shifted_alphabet = alphabet[shift:] + alphabet[:shift]
table = string.maketrans(alphabet, shifted_alphabet)
return plaintext.translate(table)
if __name__ == "__main__":
if len(sys.argv) != 4:
print("Duzgun giriniz: '<filename>.py s n l'")
sys.exit(0)
else:
s = int(sys.argv[1])
n = int(sys.argv[2])
l = int(sys.argv[3])
work = Queue.Queue()
results = Queue.Queue()
myfile=open('metin.txt','r')
text_data=myfile.read() # <=== here load file
index=0
for i in xrange(n):
t = threading.Thread(target=do_work, args=(work, results, s))
t.daemon = True
t.start()
for i in range(0, len(text_data), l):
work.put(text_data[index:index + l])
index += l
work.join()
index=0
output_file=open("crypted"+ "_"+ str(s)+"_"+str(n)+"_"+str(l)+".txt", "w")
for i in range(0, len(text_data), l):
output_file.write(results.get())
index += l
sys.exit()
「do_work」のロックが問題です。しかし、それはとにかくそこにあるべきではありません。 'Queue.get'はすでにスレッドセーフなので、あなたはそれを保護していません。代わりに、1つのスレッドが作業をしている間に、すべてのスレッドがそのロックを待機します。マルチスレッドアプリケーションを効果的にシングルスレッド化しました。 – tdelaney
ほとんどのコードを 'multiprocessing.Pool'プールに置き換え、' map'メソッドを使うことができます。追加の利点として、 'multiprocessing.pool.ThreadPool'は同じインタフェースを持つスレッドバージョンを実装します。 – tdelaney