Duda tomar datos de wx.TextCtrl y actualizar registros en la BD sin agregar nuevos

[tecafede]

Buenas amigos.

Siento que en esto de Python lo que me está trancando son las herramientas accesorias más que el propio lenguaje. O sea, con Python en el sentido clásico de lenguaje de programación me revuelvo bien pero me está complicando wxPyton y la parte de base de datos.

En el módulo que vengo armando y en el cual me han dado una mano (razpeitia sobre todo) tengo una base de datos con 5 registros, cada registro tiene número, nombre y una letra S o N para determinar si es el usuario activo o no. (Solo uno es e el activo).

Estoy desarrollando el módulo que lee esos datos, los presenta en pantalla, permite cambiar los nombres de los 5 usuarios y también cambiar cuál es el usuario activo.

Hasta ahí todo bien, lo que no estoy logrando es grabar los cambios una vez que salgo del formulario y confirmo que quiero guardar.

Línea 58.

1) no me doy cuenta como "tomar" los nuevos valores de los 5 wx.TextCtrl que en teoría en la líneas 99 a 103 son asignadas a variables independientes. Si uso usuario1 = text1 me da que la variable no es global. Probé con text1.GetValue() y tampoco

2) en el mismo def def GuardoUsuarios(): no me doy cuenta como grabar las modificaciones sin agregar nuevos registros en la base de datos. Es decir, la base de datos sigue teniendo solo 5 registros pero con datos actualizados.
En otros programas lo podía resolver con un índice o puntero, pero en este caso no me doy cuenta como lograrlo...

Agradezco cualquier idea porque la verdad quedé trancado

Código Python:

#!/usr/bin/python
# -*- coding: utf-8 -*-
 
import sqlite3 as lite
import sys
import wx
 
 
def LeoUsuarios():
    # Necesario para asignar valores
    global usuario1
    global usuario2
    global usuario3
    global usuario4
    global usuario5
    global usuarioactivo, usuarioactivonumero
 
    con = lite.connect('usuariosbd')
    listausuarios = []
    
    with con:
    
        cur = con.cursor()    
        cur.execute("SELECT * FROM Usuarios")
 
 
        while True:
      
            row = cur.fetchone()
            
            
        
            if row == None:
                break
 
            #Tomo el valor del usuario activo
            if row[2] == "S":
                print "El usuario activo es :", row[1]
                usuarioactivo = row[1]
                usuarioactivonumero = row[0]
 
           
            #agrego a la lista el nombre del usuario
            listausuarios.append(row[1])
 
 
    #asigno cada item de la lista a una variable independiente para conservar los valores una vez que salgo del DEF
    usuario1 = listausuarios[0]
    usuario2 = listausuarios[1]
    usuario3 = listausuarios[2]
    usuario4 = listausuarios[3]
    usuario5 = listausuarios[4]
 
 
    #retorno los datos de usuarios leídos
    return (usuario1, usuario2, usuario3, usuario4, usuario5)
 
def GuardoUsuarios():
    wx.MessageBox('ACEPTAR entonces guardo los cambios', 'Info', wx.OK | wx.ICON_INFORMATION)
    #Asigno los nuevos nombres de Usuarios a guardar
    usuario1 = text1
    print "para verificar si estoy tomando los nuevos nombres de usuarios correctamente"
    print "usuario1 es"
    print wxFrame.text1.GetValue()
 
    #Guardo las actualizaciones de los nombres de Usuarios
    
 
 
 
class RadioButtonFrame(wx.Frame):
    def __init__(self):
 
        print "YA DENTRO DE LA FUNCION TENGO LO SIGUIENTE"
        print usuarioactivo
        print usuarioactivonumero
        wx.Frame.__init__(self, None, -1, 'Radio Example', 
                size=(200, 200))
 
        self.Bind(wx.EVT_CLOSE, self.OnClose)
        
        panel = wx.Panel(self, -1)
        radio1 = wx.RadioButton(panel, -1, "1", pos=(20, 50), style=wx.RB_GROUP)
        radio2 = wx.RadioButton(panel, -1, "2", pos=(20, 80))
        radio3 = wx.RadioButton(panel, -1, "3", pos=(20, 110))
        radio4 = wx.RadioButton(panel, -1, "4", pos=(20, 140))
        radio5 = wx.RadioButton(panel, -1, "5", pos=(20, 170))
 
        #Selecciono el Radio Button activo de acuerdo al Usuario activo leído previamente
        if usuarioactivonumero == 1 : radio1.SetValue(1)
        if usuarioactivonumero == 2 : radio2.SetValue(1)
        if usuarioactivonumero == 3 : radio3.SetValue(1)
        if usuarioactivonumero == 4 : radio4.SetValue(1)
        if usuarioactivonumero == 5 : radio5.SetValue(1)
        
        
        #radio3.SetValue(1)
 
        text1 = wx.TextCtrl(panel, -1, usuario1, pos=(80, 50))
        text2 = wx.TextCtrl(panel, -1, usuario2, pos=(80, 80))
        text3 = wx.TextCtrl(panel, -1, usuario3, pos=(80, 110))
        text4 = wx.TextCtrl(panel, -1, usuario4, pos=(80, 140))
        text5 = wx.TextCtrl(panel, -1, usuario5, pos=(80, 170))
        self.texts = {"1": text1, "2": text2, "3": text3, "4": text4, "5": text5}
        for eachText in [text2, text3, text4, text5]:
            eachText.Enable(False)
        for eachRadio in [radio1, radio2, radio3, radio4, radio5]:
            self.Bind(wx.EVT_RADIOBUTTON, self.OnRadio, eachRadio)
        self.selectedText = text1
 
 
    def OnRadio(self, event):
        if self.selectedText:
            self.selectedText.Enable(False)
        radioSelected = event.GetEventObject()
        text = self.texts[radioSelected.GetLabel()]
        text.Enable(True)
        self.selectedText = text
 
    def OnClose(self, event):
        dlg = wx.MessageDialog(self,
            "Guardar los cambios realizados?",
            "Confirm Exit", wx.OK|wx.CANCEL|wx.ICON_QUESTION)
        result = dlg.ShowModal()
        dlg.Destroy()
        if result == wx.ID_OK:
            GuardoUsuarios()
            self.Destroy()
        else:
            wx.MessageBox('CANCELAR entonces NO guardo los cambios', 'Info', wx.OK | wx.ICON_INFORMATION)
            self.Destroy()
 
 
if __name__ == '__main__':
    app = wx.PySimpleApp()
    LeoUsuarios()
    RadioButtonFrame().Show()
    app.MainLoop()
 

[carbon]

No entiendo como accedes al valor de text1, si nunca has declarado un tal objeto wxFrame. Debes crear un objeto el cual va a ser la instancia de la clase RadioButtonFrame:

Código Python:

frm = RadioButtonFrame()
frm.Show()
 

Así podrás acceder a:

Código Python:

frm.text1.GetValue()
 

El comando SQL para actualizar un campo es el siguiente (si esa es tu inquietud):

Código SQL:

UPDATE Usuarios SET campo=nuevo_valor WHERE campo=algun_valor
 

Espero que te sirva!

[razpeitia]

1.- No haces el insert o update.
2.- En caso que lo hagas recuerda hacer el commit.
3.- No uses variables globales para algo tan simple.
4.- No enumeres tus variables, variable1, variable2, variable3, etc... es una mala practica.
5.- Cuida los espacios veo espacios (lineas en blanco) en lugares innecesarios.
6.- Trata de usar if ... elif y no varios ifs separados.
7.- No hagas un "select * from table", especifica el nombre de los campos.


Veo que tienes muchas malas practicas, ya no hablo de la separación de conceptos con mvc.

Mira, creo que esto es muy claro, el programar de manera profesional solo lo vas a obtener de una forma, con experiencia. Y la experiencia la obtienes con el tiempo y a base de prueba y error. Entonces no te desesperes, toma tu tiempo y recuerda mejorar tu código de otra forma nunca vas a tener mas experiencia.

PD: También sirve mucho leer código de otros.

[tecafede]

Iniciado por carbon

No entiendo como accedes al valor de text1, si nunca has declarado un tal objeto wxFrame. Debes crear un objeto el cual va a ser la instancia de la clase RadioButtonFrame:

Código Python:

frm = RadioButtonFrame()
frm.Show()
 

Espero que te sirva!

y no es lo mismo lo que tengo en la línea 99??

text1 = wx.TextCtrl(panel, -1, usuario1, pos=(80, 50))

me sirve y mucho, gracias!

[tecafede]

Iniciado por razpeitia

1.- No haces el insert o update.
2.- En caso que lo hagas recuerda hacer el commit.
3.- No uses variables globales para algo tan simple.
4.- No enumeres tus variables, variable1, variable2, variable3, etc... es una mala practica.
5.- Cuida los espacios veo espacios (lineas en blanco) en lugares innecesarios.
6.- Trata de usar if ... elif y no varios ifs separados.
7.- No hagas un "select * from table", especifica el nombre de los campos.


Veo que tienes muchas malas practicas, ya no hablo de la separación de conceptos con mvc.

Mira, creo que esto es muy claro, el programar de manera profesional solo lo vas a obtener de una forma, con experiencia. Y la experiencia la obtienes con el tiempo y a base de prueba y error. Entonces no te desesperes, toma tu tiempo y recuerda mejorar tu código de otra forma nunca vas a tener mas experiencia.

PD: También sirve mucho leer código de otros.

1) no, porque justamente no sabía como actualizar los datos sin grabar nuevos registros

2) ok, gracias!

3) idem!

4) es solo para este caso puntual que preciso 5 variables del mismo tipo y las identifico por 1 al 5 porque así es la estructura del programa. El "canal 1" es un usuario de nombre X. El nombre no importa, puede cambiar lo que no cambia es el número 1 en ese caso

5) El código internamente tenía cantidad de PRINT para ir viendo como evolucionaban las variables. Los eliminé antes de pegarlo para que el código quedara más claro pero olvidé borrar las líneas en blanco.

6) Gracias!

7) "No hagas un "select * from table", especifica el nombre de los campos." Ok tomo nota e investigo al respecto.


Sin dudas que tengo malos hábitos porque como mencioné antes no había programado con objetos, si con programas de lenguajes estructurados.

Estoy de acuerdo con tus anotaciones, pero de momento no pretendo programar de forma profesional, preciso que el programa funcione, luego ya lo podré mejorar para hacerlo más sencillo, más "pythoniano", etc.. pero voy aprendiendo sobre la marcha.

"También sirve mucho leer código de otros"

Así vengo aprendiendo, (y leyendo este foro aunque no pueda aportar mucho de momento) pero me mata que la información está muy dispersa.

En los "viejos tiempos" te comprabas un buen manual y ahí tenías todo... a la mano... como hacer un if, un for, como mostrar información en pantalla en forma de texto o gráfica, como leer y escribir en archivos, etc...

Ahora te lees un libro de Python y resulta que cuando terminas no sabes ni siquiera poner una ventana con un texto.. entonces tienes que comenzar a estudiar un tema aparte que son las interfases gráficas... y cuando tienes eso y quieres guardar datos en una base de datos, pues a empezar de nuevo con otra "materia" de estudio... je je je..

De todas formas vengo aprendiendo mucho en poco tiempo, al menos desde mi punto de vista.

Saludos!

[tecafede]

Hola a todos, avancé bastante y pude solucionar el tema de los wxTextCtrl y también simplifiqué el uso de variables de tipo
usuario1, usuario2, usuario3 por una lista

Sin embargo estoy en un error del cual no puedo salir.

1) El programa lee los nombres de usuarios de la base de datos y los despliega asignando cada wxTextCtrl a un RadioButton

2) Permite cambiar los valores de los wxTextCtrl y también el usuario activo

3) Al cerrar el programa si confirmo que quiero guardar los cambios debo lograr llegar a esa parte del programa con los nuevos valores de los usuarios (los tengo guardados en una lista así que funciona) y también con el usuario activo, o sea el último RadioButton seleccionado y acá tengo el problema porque no me reconoce la variable donde almaceno ese valor

Estoy hablando de las línea 126 (donde almaceno el último RadioButton pulsado y efectivamente funciona) y la línea 54 donde imprimo esa misma variable para ver si llegó hasta allí con el valor correcto.

No importa que la defina como global, o que la ponga con un return en la línea 27, igual ESA variable no me reconoce, pero SI me reconoce a los wxTextCtrl... la verdad me tiene desorientado ese problema y llevo dos días sin poder resolverlo.

Código Python:

#!/usr/bin/python
# -*- coding: utf-8 -*-
 
import sqlite3 as lite
import sys
import wx
global usuarioactivonumeronuevo
 
 
 
def LeoUsuarios():
    # Necesario para asignar valores
    global listausuarios
    global usuarioactivo, usuarioactivonumero
 
    con = lite.connect('db.usuarios')
    listausuarios = []
    
    with con:
        cur = con.cursor()    
        cur.execute("SELECT * FROM Usuarios")
 
        while True:
            row = cur.fetchone()
            
            if row == None:
                break
 
            #Tomo el valor del usuario activo
            if row[2] == "S":
                print "El usuario activo es :", row[1]
                usuarioactivo = row[1]
                usuarioactivonumero = row[0]
 
            #print "IMPRIMO LOS TRES CAMPOS PARA VERIFICAR QUE FUERON LEIDOS"
            #print row[0], row[1], row[2]
            
            #agrego a la lista el nombre del usuario
            listausuarios.append(row[1])
 
        print "LISTA DE USUARIOS"
        print listausuarios
 
    #retorno los datos de usuarios leídos
    return (listausuarios)
 
def GuardoUsuarios():
    wx.MessageBox('ACEPTAR entonces guardo los cambios', 'Info', wx.OK | wx.ICON_INFORMATION)
    #Asigno nuevo usuario activo y los nuevos nombres de Usuarios a guardar
    print "==========="
    print "VERIFICANDO"
    print "==========="
    print "U S U A R I O   A C T I V O - "
    print usuarioactivonumeronuevo
    print "EL PRIMER USUARIO ES ", text1.GetValue()
    print "EL SEGUNDO USUARIO ES ", text2.GetValue()
    print "EL TERCER USUARIO ES ", text3.GetValue()
    print "EL CUARTO USUARIO ES ", text4.GetValue()
    print "EL QUINTO USUARIO ES ", text5.GetValue()
 
class RadioButtonFrame(wx.Frame):
    def __init__(self):
        #print "ESTOY DENTRO DE LA SEGUNDA FUNCION Y USUARIO 1 ES"
        #print usuario1
        print "YA DENTRO DE LA FUNCION TENGO LO SIGUIENTE"
        print usuarioactivo
        print usuarioactivonumero
        wx.Frame.__init__(self, None, -1, 'Radio Example', 
                size=(200, 200))
 
        self.Bind(wx.EVT_CLOSE, self.OnClose)
        
        panel = wx.Panel(self, -1)
        radio1 = wx.RadioButton(panel, -1, label="1", pos=(20, 50), style=wx.RB_GROUP)
        radio2 = wx.RadioButton(panel, -1, label="2", pos=(20, 80))
        radio3 = wx.RadioButton(panel, -1, label="3", pos=(20, 110))
        radio4 = wx.RadioButton(panel, -1, label="4", pos=(20, 140))
        radio5 = wx.RadioButton(panel, -1, label="5", pos=(20, 170))
 
        #Selecciono el Radio Button activo de acuerdo al Usuario activo leído previamente
        if usuarioactivonumero == 1 : radio1.SetValue(1)
        if usuarioactivonumero == 2 : radio2.SetValue(1)
        if usuarioactivonumero == 3 : radio3.SetValue(1)
        if usuarioactivonumero == 4 : radio4.SetValue(1)
        if usuarioactivonumero == 5 : radio5.SetValue(1)
        
        #radio3.SetValue(1)
        global text1, text2, text3, text4, text5
 
        text1 = wx.TextCtrl(panel, -1, listausuarios[0], pos=(80, 50))
        text2 = wx.TextCtrl(panel, -1, listausuarios[1], pos=(80, 80))
        text3 = wx.TextCtrl(panel, -1, listausuarios[2], pos=(80, 110))
        text4 = wx.TextCtrl(panel, -1, listausuarios[3], pos=(80, 140))
        text5 = wx.TextCtrl(panel, -1, listausuarios[4], pos=(80, 170))
        self.texts = {"1": text1, "2": text2, "3": text3, "4": text4, "5": text5}
        for eachText in [text1, text2, text3, text4, text5]:
            eachText.Enable(False)
        for eachRadio in [radio1, radio2, radio3, radio4, radio5]:
            self.Bind(wx.EVT_RADIOBUTTON, self.OnRadio, eachRadio)
 
        #Selecciono el Text activo de acuerdo al Usuario activo leído previamente
        self.selectedText = text1
        #print "Estoy verificando y el usuario activo es -->", usuarioactivonumero
        
        if usuarioactivonumero == 1 : text1.Enable(True)
        if usuarioactivonumero == 2 : text2.Enable(True)
        if usuarioactivonumero == 3 : text3.Enable(True)
        if usuarioactivonumero == 4 : text4.Enable(True)
        if usuarioactivonumero == 5 : text5.Enable(True)
 
        
 
    def OnRadio(self, event):
        for eachtext in [text1, text2, text3, text4, text5]:
            eachtext.Enable(False)
        if self.selectedText:
            self.selectedText.Enable(False)
        radioSelected = event.GetEventObject()
        
        text = self.texts[radioSelected.GetLabel()]
        text.Enable(True)
        self.selectedText = text
 
        btn = event.GetEventObject()
        usuarioactivonumeronuevo = btn.GetLabel()
        print "EL BOTON ACTIVO ES", usuarioactivonumeronuevo
        return usuarioactivonumeronuevo
 
 
    def OnClose(self, event):
        dlg = wx.MessageDialog(self,
            "Guardar los cambios realizados?",
            "Confirm Exit", wx.OK|wx.CANCEL|wx.ICON_QUESTION)
        result = dlg.ShowModal()
        dlg.Destroy()
        if result == wx.ID_OK:
            GuardoUsuarios()
            self.Destroy()
        else:
            wx.MessageBox('CANCELAR entonces NO guardo los cambios', 'Info', wx.OK | wx.ICON_INFORMATION)
            self.Destroy()
 
if __name__ == '__main__':
    app = wx.PySimpleApp()
    LeoUsuarios()
    #print "PERO APENAS SALGO EL VALOR DE USUARIO1 ES"
    #print usuario1
    frm = RadioButtonFrame()
    frm.Show()
    #RadioButtonFrame().Show()
    app.MainLoop()
 

[razpeitia]

Código MySQL:

create table if not exists Usuarios (
    id integer primary key,
    nombre varchar(150),
    activo varchar(1)
);
 
insert into Usuarios values (1, 'Usuario 1', 'S');
insert into Usuarios values (2, 'Usuario 2', 'N');
insert into Usuarios values (3, 'Usuario 3', 'N');
insert into Usuarios values (4, 'Usuario 4', 'N');
insert into Usuarios values (5, 'Usuario 5', 'N');
 

Código Python:

#!/usr/bin/python
# -*- coding: utf-8 -*-
 
import sqlite3 as lite
import sys
import wx
 
 
class User(object):
    def __init__(self, _id, name, active):
        self.id = _id
        self.name = name
        self.active = active
 
    def is_active(self):
        return self.active == 'S'
 
    def set_active(self, active):
        if active:
            self.active = 'S'
        else:
            self.active = 'N'
 
    def __str__(self):
        return self.name
 
class Users(object):
    def __init__(self):
        self.user_list = []
 
    def add(self, user):
        self.user_list.append(user)
 
    def __iter__(self):
        return iter(self.user_list)
 
def get_users():
    db_connection = lite.connect('db.usuarios')
    users = Users()
    with db_connection:
        cursor = db_connection.cursor()
        cursor.execute("SELECT id, nombre, activo FROM Usuarios")
 
        for _id, name, active in cursor:
            user = User(_id, name, active)
            users.add(user)
 
    return users
 
def save_users(users):
    db_connection = lite.connect('db.usuarios')
    with db_connection:
        cursor = db_connection.cursor()
        for user in users:
            query = "UPDATE Usuarios SET activo=?, nombre=? WHERE id=?"
            params = (user.active, user.name, user.id)
            cursor.execute(query, params)
        db_connection.commit()
 
class RadioButtonFrame(wx.Frame):
    def __init__(self):
        wx.Frame.__init__(self, None, -1, 'User Admin', size=(200, 200))
 
        self.Bind(wx.EVT_CLOSE, self.OnClose)
        
        panel = wx.Panel(self, -1)
        self.widgets = []
        self.users = get_users()
        for i, user in enumerate(self.users):
            radio = wx.RadioButton(panel, -1, label=str(i+1), pos=(20, 50+(i*30)))
            text = wx.TextCtrl(panel, -1, user.name, pos=(80, 50+(i*30)))
            text.Enable(False)
 
            if user.is_active():
                radio.SetValue(True)
                text.Enable(True)
 
            def OnButton(event, radio_index=i):
                self.OnRadio(event, radio_index)
            radio.Bind(wx.EVT_RADIOBUTTON, OnButton)
 
            widget = {'radio': radio, 'text': text}
            self.widgets.append(widget)
 
    def OnRadio(self, event, index):
        for widget in self.widgets:
            widget['text'].Enable(False)
        widget = self.widgets[index]
        widget['text'].Enable(True)
 
    def OnClose(self, event):
        dlg = wx.MessageDialog(self, "Guardar los cambios realizados?",  "Confirmar Salida", wx.OK|wx.CANCEL|wx.ICON_QUESTION)
        result = dlg.ShowModal()
        dlg.Destroy()
        if result == wx.ID_OK:
            for user, widget in zip(self.users, self.widgets):
                user.set_active(widget['radio'].GetValue())
                user.name = widget['text'].GetValue()
            save_users(self.users)
        self.Destroy()
 
if __name__ == '__main__':
    app = wx.PySimpleApp()
    frm = RadioButtonFrame()
    frm.Show()
    app.MainLoop()
 

Mi intento de hacer esta pequeña app. No esta bien estructurada y hay SQL generado por un humano de por medio, pero hay te dejo la prueba.

[carbon]

No entendí muy bien, pero si lo que quieres es modificar "usuarioactivonumeronuevo" debes hacer:

Código Python:

...
def OnRadio(self, event):
    global usuarioactivonumeronuevo
    ...
 

De lo contrario estás creando otra variable que será visible únicamente dentro de esa función, y no para todo el programa.



Edito: No vi tu post razpeitia!

[razpeitia]

@carbon no te preocupes, los comentarios u observaciones de la comunidad nunca esta sobra.

[tecafede]

Iniciado por razpeitia

Mi intento de hacer esta pequeña app. No esta bien estructurada y hay SQL generado por un humano de por medio, pero hay te dejo la prueba.

:o jotes:

Realmente asombroso, el que sabe sabe, y el que no hace programas kilométricos que no funcionan del todo

Te agradezco, funciona perfecto, es lo que precisaba. De todas formas como rehiciste el código para simplificarlo lo voy a estudiar bien para ver las soluciones que implementaste.

Gracias de nuevo.

P.D.: que el código SQL genere los usuarios automáticamente está bien, porque el programa una vez terminado simplemente entrega una base de datos con usuario1, usuario2, usuario3.... etc y la primera vez que se abre el programa te da la posibilidad de cambiar los nombres.

[tecafede]

Iniciado por carbon

No entendí muy bien, pero si lo que quieres es modificar "usuarioactivonumeronuevo" debes hacer:

Código Python:

...
def OnRadio(self, event):
    global usuarioactivonumeronuevo
    ...
 

De lo contrario estás creando otra variable que será visible únicamente dentro de esa función, y no para todo el programa.



Edito: No vi tu post razpeitia!

No puedo creer que lo que estaba haciendo era definir la variable global en el lugar inadecuado!!!

Con ese sencillo cambio mi código funcionó perfecto, muchas gracias, me sirve todo para aprender.!