62

I am trying to create buttons in tkinter within a for loop. And with each loop pass the i count value out as an argument in the command value. So when the function is called from the command value I can tell which button was pressed and act accordingly.

The problem is, say the length is 3, it will create 3 buttons with titles Game 1 through Game 3 but when any of the buttons are pressed the printed value is always 2, the last iteration. So it appears the buttons are being made as separate entities, but the i value in the command arguments seem to be all the same. Here is the code:

def createGameURLs(self):
    self.button = []
    for i in range(3):
        self.button.append(Button(self, text='Game '+str(i+1),
                                  command=lambda: self.open_this(i)))
        self.button[i].grid(column=4, row=i+1, sticky=W)

def open_this(self, myNum):
    print(myNum)

Is there a way to get the current i value, each iteration, to stick with that particular button?


This problem can be considered a special case of Creating functions in a loop. There's also What do lambda function closures capture?, for a more technical overview.

See also How to pass arguments to a Button command in Tkinter? for the general problem of passing arguments to Button callbacks.

8
  • A big thanks to both of you LukaD and BrenBarn, I have been fighting with that for several days now believe it or not. Both ways worked perfectly. I went with the i=i fix for now, but I'm definitely going to read up on the functools. I appreciate both answers.
    – Marcel
    Jun 2, 2012 at 19:24
  • If BrenBarns solution works for you then you should mark it as your accepted answer.
    – lukad
    Jun 3, 2012 at 14:25
  • @martineau Do you believe that the duplicate question answers this question? Feb 13, 2022 at 14:23
  • 1
    @Delrius: Oops, no, I accidentally marked it as a dup. and of the wrong question. Thanks for bring the mistake to to my attention.
    – martineau
    Feb 13, 2022 at 15:29
  • Also see stackoverflow.com/q/19693782/4014959
    – PM 2Ring
    Apr 29, 2022 at 18:23

4 Answers 4

145

Change your lambda to lambda i=i: self.open_this(i).

This may look magical, but here's what's happening. When you use that lambda to define your function, the open_this call doesn't get the value of the variable i at the time you define the function. Instead, it makes a closure, which is sort of like a note to itself saying "I should look for what the value of the variable i is at the time that I am called". Of course, the function is called after the loop is over, so at that time i will always be equal to the last value from the loop.

Using the i=i trick causes your function to store the current value of i at the time your lambda is defined, instead of waiting to look up the value of i later.

13
  • 1
    what if we wanted to pass two arguments to a function like open_this?
    – Amen
    Oct 14, 2014 at 17:10
  • 10
    @Amen: It depends what you want those arguments to be. If both are coming from some external loop and you want to "freeze" them both in way shown above, you would just do lambda x=x, y=y: self.open_this(x, y).
    – BrenBarn
    Oct 14, 2014 at 18:06
  • 3
    This is brilliant, straightforward and good explanation. This should be the answer.
    – Battleroid
    Mar 1, 2016 at 23:38
  • 1
    @BrenBarn Just now i came across same situation, its really amazing. I really weird how you all came across these kind of techniques. Thank you so much.
    – velpandian
    Mar 3, 2018 at 16:59
  • 1
    @RufusVS: It's just default argument syntax. It's the same as when you do def foo(x, y=3). It's just that here the name of the function argument is the same as the name of a variable in the enclosing scope.
    – BrenBarn
    Jun 28, 2020 at 23:23
12

This is how closures work in python. I ran into this problem myself once. You could use functools.partial for this.

for i in range(3):
    self.button.append(Button(self, text='Game '+str(i+1), command=partial(self.open_this, i)))
2
  • seemed easy, if you are using virtual environments you might get an issue, refer to the first on: self.button.append(Button(self, text='Game ' + str(i + 1), command=lambda x=i: self.open_this(x)))
    – Lucem
    Mar 29, 2020 at 9:15
  • 2
    @Lucem "if you are using virtual environments you might get an issue" There is no good reason why, as functools.partial is part of the standard library and has been for a long time. If you had difficulty with it then you should try to diagnose the problem and ask your own question. May 11, 2022 at 11:37
3

Simply attach your buttons scope within a lambda function like this:

btn["command"] = lambda btn=btn: click(btn) where click(btn) is the function that passes in the button itself. This will create a binding scope from the button to the function itself.

Features:

  • Customize gridsize
  • Responsive resizing
  • Toggle active state

#Python2
#from Tkinter import *
#import Tkinter as tkinter
#Python3
from tkinter import *
import tkinter

root = Tk()
frame=Frame(root)
Grid.rowconfigure(root, 0, weight=1)
Grid.columnconfigure(root, 0, weight=1)
frame.grid(row=0, column=0, sticky=N+S+E+W)
grid=Frame(frame)
grid.grid(sticky=N+S+E+W, column=0, row=7, columnspan=2)
Grid.rowconfigure(frame, 7, weight=1)
Grid.columnconfigure(frame, 0, weight=1)

active="red"
default_color="white"

def main(height=5,width=5):
  for x in range(width):
    for y in range(height):
      btn = tkinter.Button(frame, bg=default_color)
      btn.grid(column=x, row=y, sticky=N+S+E+W)
      btn["command"] = lambda btn=btn: click(btn)

  for x in range(width):
    Grid.columnconfigure(frame, x, weight=1)

  for y in range(height):
    Grid.rowconfigure(frame, y, weight=1)

  return frame

def click(button):
  if(button["bg"] == active):
    button["bg"] = default_color
  else:
    button["bg"] = active

w= main(10,10)
tkinter.mainloop()

enter image description here enter image description here

enter image description here

1

It's because the value for the name i changes and isn't captured by lambda:. (You can try that theory out by adding i = 1234 after the loop and seeing what happens.)

You'll need to write a function to wrap that i as a local name, then return a lambda in that function that captures i .

def make_button_click_command(i):
    return lambda: button_click(i)

# ...

btn = Button(..., command=make_button_click_command(i))

Another option is functools.partial, which does effectively the same thing:

command=functools.partial(button_click, i)

All in all, you can also simplify things a bit by using just range to get numbers from 0 to 10 and divmod to get the row and column in one function call:

from tkinter import Tk, Button


def button_click(i):
    print(i)


def make_button_click_command(i):
    return lambda: button_click(i)


root = Tk()

for i in range(10):
    value = (i + 1) % 10
    row, col = divmod(i, 3)
    btn = Button(root, text=value, padx=40, pady=20, command=make_button_click_command(value))
    btn.grid(row=row + 1, column=col)

root.mainloop()
0

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service, privacy policy and cookie policy

Not the answer you're looking for? Browse other questions tagged or ask your own question.