Блог

Я хочу создать функцию, в которой пользователь получает список пользователей из базы данных и может выбрать удаление одного из них из базы данных. Это выглядит так, как я хочу, я получаю все элементы в строках с кнопкой удаления для каждого, и они называются правильно. Но функция, похоже, работает не так, как я хочу. Если я нажму любую кнопку удаления, последний пользователь будет удален. Я новичок в этом, поэтому уверен, что совершил ошибку, но не знаю где.

 def delete():
    conn = sqlite3.connect("database.db")
    c = conn.cursor()
    c.execute("DELETE FROM databasetable WHERE oid = "   str(i[5]))
    conn.commit()
    conn.close()

conn = sqlite3.connect("database.db")
c = conn.cursor()
c.execute("SELECT *, oid FROM databasetable")
listfromdatabase = c.fetchall()
conn.commit()
conn.close()

for i in listfromdatabase:
    label = Label(frameskidlararenamn, text=i[0])
    label.pack()
    button = Button(frameskidlarareknapp, text="Delete "   str(i[0]), command=delete)
    button.pack()
 

Есть какие-нибудь идеи?

В конце цикла for i in listfromdatabase: i находится последний элемент. Поэтому функция удаления удаляет этот последний элемент.

Проверьте, работает ли это, передав аргумент команде с помощью лямбда-функции.
изменилось lambda : delete(i[5]) на lambda item=i[5]: delete(item)

 button1 = [None for _ in range(len(listfromdatabase))]
for n,i in enumerate(listfromdatabase):
    label = Label(frameskidlararenamn, text=i[0])
    label.pack()
    button1[n] = Button(frameskidlarareknapp, text="Delete "   str(i[0]),command=lambda item=i[5]: delete(item))
    button1[n].pack()
 

 def delete(item):
    conn = sqlite3.connect("database.db")
    c = conn.cursor()
    c.execute("DELETE FROM databasetable WHERE oid = "   str(item))
    conn.commit()
    conn.close()
 

Просто чтобы быть уверенным, проверьте с помощью двух кнопок с одинаковым названием, но с разными функциями. Нравится

 def printnum(num):
    #print num on a label or something
button = Button(frameskidlarareknapp, text="print num 1", command=lambda: printnum(1))
button.pack()
button = Button(frameskidlarareknapp, text="print num 2", command=lambda: printnum(2))
button.pack()
 

Ваше определение удаления настроено так, чтобы оно было нацелено на пятый элемент i

 def delete():
    conn = sqlite3.connect("database.db")
    c = conn.cursor()
    c.execute("DELETE FROM databasetable WHERE oid = "   str(i[5]))
    conn.commit()
    conn.close()
 

Вместо этого вы должны передать oid для удаления, изменив определение

 def delete(oid):
    conn = sqlite3.connect("database.db")
    c = conn.cursor()
    c.execute("DELETE FROM databasetable WHERE oid = "   str(oid))
    conn.commit()
    conn.close()