我有一个脚本可以创建一堆线程,运行一个程序来使用线程从队列中运行任务,并从每个线程返回一些东西。我想计算其中有多少成功返回,所以我设置了一个变量" success = 0"每次队列报告任务成功完成时,都会增加它。
然而,我得到了#34; UnboundLocalError:本地变量'成功'在分配之前引用"
发生了什么?
以下是一些示例代码:
successful = 0
q = Queue(200)
for i in range(100):
t=Thread(target=foo)
t.daemon=True
t.start()
def foo():
while True:
task=q.get()
#do some work
print task
successful+=1 # triggers an error
q.task_done()
for i in range(100):
q.put("Foo")
q.join()
print successful
答案 0 :(得分:17)
successful+=1
不是线程安全的操作。如果多个线程试图递增共享的全局变量,则可能发生冲突,并且successful
将无法正确递增。
要避免此错误,请使用锁定:
lock = threading.Lock()
def foo():
global successful
while True:
...
with lock:
successful+=1
以下是一些代码,用于演示x + = 1不是线程安全的:
import threading
lock = threading.Lock()
x = 0
def foo():
global x
for i in xrange(1000000):
# with lock: # Uncomment this to get the right answer
x += 1
threads = [threading.Thread(target=foo), threading.Thread(target=foo)]
for t in threads:
t.daemon = True
t.start()
for t in threads:
t.join()
print(x)
的产率:
% test.py
1539065
% test.py
1436487
这些结果不一致且小于预期的2000000.取消注释锁定会产生正确的结果。
答案 1 :(得分:5)
问题发生是因为在函数内部分配的变量被认为是该函数的本地变量。如果要修改在successfull
之外创建的变量foo
的值,则需要明确告知解释器您将在函数内使用全局变量。这可以通过以下方式完成:
def foo():
global successfull
while True:
task=q.get()
#do some work
print task
successful+=1 # triggers an error
q.task_done()
现在代码应该按预期工作。
答案 2 :(得分:0)