This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author alan.hoover
Recipients alan.hoover
Date 2010-04-01.18:25:59
SpamBayes Score 7.313761e-12
Marked as misclassified No
Message-id <1270146362.25.0.403939205554.issue8283@psf.upfronthosting.co.za>
In-reply-to
Content
Background:
building a screen using Tkinter based on information from a database to let user take action on the database rows.  I don't know how many rows there will be, so I'm storing the widgets in arrays for each column.

Code:
for row in getDBrows():
    self.buttons.append(Tkinter.Button(self,text='UnLoad it', command=lambda :self.unload(row[0])))

Problem:
When executing the above code, all the buttons have the key to the database table that belongs to the last row.  I found a work around -- by moving the call to create the button (containing the lambda) to a separate function and call that function to create the button instead of creating the button directly, the buttons then make their callback with a correct database key.

Workaround:
for row in getDBrows():
    self.buttons.append(self.buildbutton(row[0]))
.
.
def buildbutton(self,key):
    return Tkinter.Button(self,text='UnLoad it', command=lambda: self.unload(key))

When using the workaround code instead of the original code, the button
for each row has the appropriate key to the database.

Speculation:
It acts like the lambda definitions don't get solidified until the containing block exits; at that time, the lambda definition(s) get locked in with the current value of the variable getting passed into the lambda as a parameter.  By moving the lambda call to a different block (separate function), the lambda gets "locked" when that block (function) exits instead of when the containing block (loop) exits.
History
Date User Action Args
2010-04-01 18:26:02alan.hooversetrecipients: + alan.hoover
2010-04-01 18:26:02alan.hooversetmessageid: <1270146362.25.0.403939205554.issue8283@psf.upfronthosting.co.za>
2010-04-01 18:26:00alan.hooverlinkissue8283 messages
2010-04-01 18:25:59alan.hoovercreate