CODEWITHSUNDEEP
pythonlambdatkinter
apalopohapa
apalopohapa

Reputation: 5183

Python's lambda iteration not working as intended

In the code below I intend to have two buttons, and when each is pressed '0' and '1' are to be printed to stdout, respectively. However when the program is run, they both print '1', which is the last value i had in the for iteration. Why?

import Tkinter as tk
import sys

root = tk.Tk()

for i in range(0,2):
    cmd = lambda: sys.stdout.write(str(i))
    tk.Button(text="print '%d'" % i,command=cmd).pack()

root.mainloop()

Upvotes: 1

Views: 521

Answers (3)

John La Rooy
John La Rooy

Reputation: 304205

I think it's a bit odd to use an anonymous function just to then give it a name. Why not write it like this?

for i in 0,1:
    def cmd():
        return sys.stdout.write(str(i))
    tk.Button(text="print '%d'"%i, command=cmd).pack()

Upvotes: 1

Brian
Brian

Reputation: 118865

Surely it's the issue in

On lambdas, capture, and mutability

that comes up over and over...

Upvotes: 3

newacct
newacct

Reputation: 122449

The i is not captured in the lambda when you create it (as you wanted). Instead, both functions refer back to the i in the external for loop, which changes after the function is created and before it is run. To capture it, you can use a default value:

for i in range(0,2):
    cmd = lambda i=i: sys.stdout.write(str(i))
    tk.Button(text="print '%d'" % i,command=cmd).pack()

Upvotes: 5

Related Questions