85

Considering the following code snippet:

# directorys == {'login': <object at ...>, 'home': <object at ...>}
for d in directorys:
    self.command["cd " + d] = (lambda : self.root.change_directory(d))

I expect to create a dictionary of two function as following :

# Expected :
self.command == {
    "cd login": lambda: self.root.change_directory("login"),
    "cd home": lambda: self.root.change_directory("home")
}

but it looks like the two lambda function generated are exactly the same :

# Result :
self.command == {
    "cd login": lambda: self.root.change_directory("login"),
    "cd home": lambda: self.root.change_directory("login")   # <- Why login ?
}

I really don't understand why. Do you have any suggestions ?

4

4 回答 4

107

您需要为创建的每个函数绑定 d 。一种方法是将其作为具有默认值的参数传递:

lambda d=d: self.root.change_directory(d)

现在函数内部的 d 使用参数,即使它具有相同的名称,并且在创建函数时评估它的默认值。为了帮助您看到这一点:

lambda bound_d=d: self.root.change_directory(bound_d)

记住默认值是如何工作的,例如列表和字典等可变对象,因为您正在绑定一个对象。

这种具有默认值的参数习语很常见,但如果您自省函数参数并根据它们的存在确定要做什么,则可能会失败。您可以使用另一个闭包来避免该参数:

(lambda d=d: lambda: self.root.change_directory(d))()
# or
(lambda d: lambda: self.root.change_directory(d))(d)
于 2013-11-07T13:52:05.950 回答
29

这是由于 d 被绑定的点。lambda 函数都指向变量 d而不是变量的当前,因此当您d在下一次迭代中更新时,此更新会在您的所有函数中看到。

举一个更简单的例子:

funcs = []
for x in [1,2,3]:
  funcs.append(lambda: x)

for f in funcs:
  print f()

# output:
3
3
3

您可以通过添加一个附加函数来解决这个问题,如下所示:

def makeFunc(x):
  return lambda: x

funcs = []
for x in [1,2,3]:
  funcs.append(makeFunc(x))

for f in funcs:
  print f()

# output:
1
2
3

您还可以修复 lambda 表达式中的范围

lambda bound_x=x: bound_x

但是,通常这不是一个好习惯,因为您已经更改了函数的签名。

于 2013-11-07T13:47:35.783 回答
8

lambda或者,您可以使用which代替functools.partial,在我看来,它具有更简洁的语法。

代替:

for d in directorys:
    self.command["cd " + d] = (lambda d=d: self.root.change_directory(d))

这将是:

for d in directorys:
    self.command["cd " + d] = partial(self.root.change_directory, d)

或者,这是另一个简单的例子:

numbers = [1, 2, 3]

lambdas = [lambda: print(number) 
           for number in numbers]
lambdas_with_binding = [lambda number=number: print(number) 
                        for number in numbers]
partials = [partial(print, number) 
            for number in numbers]

for function in lambdas:
    function()
# 3 3 3
for function in lambdas_with_binding:
    function()
# 1 2 3
for function in partials:
    function()
# 1 2 3
于 2019-07-31T10:03:08.820 回答
2

我遇到了同样的问题。选择的解决方案对我帮助很大,但我认为有必要添加一个精度以使问题的代码发挥作用:在循环之外定义 lambda 函数。顺便说一句,默认值不是必需的。

foo = lambda d: lambda : self.root.change_directory(d)
for d in directorys:
    self.command["cd " + d] = (foo(d))
于 2017-10-20T10:27:41.890 回答