El error se debe al momento en el que es evaluada la línea:
datos=pro.get(), pre.get(), can.get(), var.get()
esta línea se evalúa al ejecutar la función agregar
, antes incluso de que la composición de la app esté completa y de que se muestre en pantalla (antes del mainloop). En ese momento todos los entries están lógicamente vacíos por lo que a crear
siempre se le manda una tupla con cuatro cadenas vacías.
Debes obtener los cuatro valores en el momento en el que se pulsa el botón, no cuando la función es llamada. Hay muchas formas de hacer esto, pero sin modificar tu idea original puede simplemente valerte de que las expresiones lambda se evalúan en diferido, es decir, cuando son llamadas y no al ser definidas:
import sqlite3
import tkinter as tk
def crear(datos1):
con = sqlite3.connect("Inventario")
cur = con.cursor()
try:
cur.execute("INSERT INTO INVENTARIO VALUES(NULL,?,?,?,?)", datos1)
con.commit()
tk.messagebox.showinfo("Agregar","Registro Agregado")
except:
tk.messagebox.showwarning("Error","Ha ocurrido un error")
finally:
cur.close()
con.close()
def agregar():
root = tk.Tk()
pro = tk.StringVar(root)
pre = tk.StringVar(root)
can = tk.StringVar(root)
var = tk.StringVar(root)
frame1 = tk.Frame(root)
frame1.pack()
entry1 = tk.Entry(frame1, textvariable=pro)
entry1.grid(row=1, column=1, padx=10, pady=10)
entry2 = tk.Entry(frame1, textvariable=pre)
entry2.grid(row=2, column=1, padx=10, pady=10)
entry3 = tk.Entry(frame1, textvariable=can)
entry3.grid(row=3, column=1, padx=10, pady=10)
rb1 = tk.Radiobutton(frame1, text="Producto", variable=var, value="Producto")
rb1.grid(row=4, column=1, sticky="w")
rb2 = tk.Radiobutton(frame1, text="Servicio", variable=var, value="Servicio")
rb2.grid(row=5, column=1, sticky="w")
label1 = tk.Label(frame1, text="Producto:")
label1.grid(row=1, column=0, padx=10, pady=10, sticky="w")
label2 = tk.Label(frame1, text="Precio:")
label2.grid(row=2, column=0, padx=10, pady=10, sticky="w")
label3 = tk.Label(frame1, text="Cantidad:")
label3.grid(row=3, column=0, padx=10, pady=10, sticky="w")
label4 = tk.Label(frame1, text="Tipo:")
label4.grid(row=4, column=0, padx=10, pady=10, sticky="w")
frame2 = tk.Frame(root)
frame2.pack()
crear_wrapper = lambda: crear((pro.get(), pre.get(), can.get(), var.get()))
boton1 = tk.Button(frame2, text="Agregar", command=crear_wrapper)
boton1.grid(row=1, column=0, padx=5)
boton2 = tk.Button(frame2, text="Cancelar", command=root.destroy)
boton2.grid(row=1, column=1, padx=5)
root.mainloop()
if __name__ == "__main__":
agregar()
Ten en cuenta que no necesitas usar lambda si la función no recibe argumentos, basta con pasar una referencia a la función en este caso (command=root.destroy
en vez de command=lambda:root.destroy()
):
Edición
Dado que la función agregar es lanzada al pulsar un botón en otra ventana debes tener en cuenta algunas cosas más:
Pasa siempre el padre a los widget, incluidas las variables (StringVar
, IntVar
, etc).
No debes usar en principio más de una sola instancia de tkinter.Tk
en una aplicación. Si necesitas más ventanas usa TopLevel
, mira esta pregunta relacionada:
Tu función debería quedar:
def agregar(root=None):
window = Toplevel(root)
pro = StringVar(window)
pre = StringVar(window)
can = StringVar(window)
var = StringVar(window)
frame1 = Frame(window)
frame1.pack()
entry1 = Entry(frame1, textvariable=pro)
entry1.grid(row=1, column=1, padx=10, pady=10)
entry2 = Entry(frame1, textvariable=pre)
entry2.grid(row=2, column=1, padx=10, pady=10)
entry3 = Entry(frame1, textvariable=can)
entry3.grid(row=3, column=1, padx=10, pady=10)
rb1 = Radiobutton(frame1, text="Producto", variable=var, value="Producto")
rb1.grid(row=4, column=1, sticky="w")
rb2 = Radiobutton(frame1, text="Servicio", variable=var, value="Servicio")
rb2.grid(row=5, column=1, sticky="w")
label1 = Label(frame1, text="Producto:")
label1.grid(row=1, column=0, padx=10, pady=10, sticky="w")
label1 = Label(frame1, text="Precio:")
label1.grid(row=2, column=0, padx=10, pady=10, sticky="w")
label1 = Label(frame1, text="Cantidad:")
label1.grid(row=3, column=0, padx=10, pady=10, sticky="w")
label1 = Label(frame1, text="Tipo:")
label1.grid(row=4, column=0, padx=10, pady=10, sticky="w")
frame2 = Frame(window)
frame2.pack()
boton1 = Button(frame2, text="Agregar", command=lambda: crear((pro.get(), pre.get(), can.get(), var.get())))
boton1.grid(row=1, column=0, padx=5)
boton2=Button(frame2, text="Cancelar", command=window.destroy)
boton2.grid(row=1, column=1, padx=5)
window.mainloop()
y debes llamarla mediante:
invMenu.add_command(label="Agregar Item", command=lambda: F.agregar(root))