Porque fetchone no me hace lo que quiero

[poeta3d]

TENGO ESTA FUNCION Y ME PASA DE UN REGISTRO A OTRO PERO CUANDO VUALVO A EJECUTAR LA FUNCION NO ME MUESTRA EL SEGUNDO REGISTRO PROXIMO. LES AGRADECERIA UNA AYUITA. QUIERO QUE ME PASE DE UN REGISTRO A OTRO Y DESPUES AL OTRO ETC

def onsiguiente(self,event):
conexion=sqlite3.connect("MOTOS.S3DB")
cursor=conexion.cursor()
cursor.execute("SELECT CODIGO,PRECIO,DETALLE FROM MOTOS")
cursor.fetchone()
for row in cursor:
self.textocodigo.SetValue(row[2])
self.textocodigo2.SetValue(row[0])
self.textocodigo3.SetValue(row[1])

cursor.close()
conexion.close()

[AlvaroG]

Estás usando mal la función, ya que de esa forma siempre estás obteniendo el primer registro (ya que siempre estás ejecutando la consulta)

No sé cuál será tu necesidad de usar una función para obtener cada registro, pero por un lado podés usar LIMIT (aunque es de MySQL, no sé si en SQLite se puede hacer igual) y por otro quizás te sirva usar yield.

Aunque mi pregunta real es: ¿por qué tu aplicación requiere que traigas un registro cada vez? Creo que tu forma de atacar el problema no es correcta.


Saludos.

[poeta3d]

mi idea es hacer un programa contable que busque registros escibiendolos en un textctrl y pulsando el boton me salga en otro texctrl mercaderia precio etc eso lo logre ya con sqlite y sqliteadmin, pero he visto algunos programas que dan la posibilidad de navegar registros sin tener que escribirlo, por eso mi pregunta y espero entiendas lo que quiero hacer.gracias por la respuesta

[AlvaroG]

Comprendo, pero el problema es que al hacer la consulta en cada ejecución de la función, siempre estás accediendo al primer registro. Deberías hacer la consulta primero, y luego iterar sobre el cursor. Una forma podría ser:

Código Python:

def consulta(self):
    conexion=sqlite3.connect("MOTOS.S3DB")
    self.cursor=conexion.cursor()
    self.cursor.execute("SELECT CODIGO,PRECIO,DETALLE FROM MOTOS")
 
def onsiguiente(self,event, cursor):
    row = self.cursor.fetchone()
    self.textocodigo.SetValue(row[2])
    self.textocodigo2.SetValue(row[0])
    self.textocodigo3.SetValue(row[1])
 

Otra forma mejor es aprovechar las capacidades de SQLite:

Código Python:

def onsiguiente(self,event, offset):
    conexion=sqlite3.connect("MOTOS.S3DB")
    cursor=conexion.cursor()
    cursor.execute("SELECT CODIGO,PRECIO,DETALLE FROM MOTOS LIMIT 1 OFFSET " + offset)
    row = cursor.fetchone()
    self.textocodigo.SetValue(row[2])
    self.textocodigo2.SetValue(row[0])
    self.textocodigo3.SetValue(row[1])
 

offset es una variable que iniciás en cero e incrementás en 1 cada vez que se llame al método. Con eso irás iterando sobre los registros de la base, sin obtener todos a la vez.

También deberías pensar en no abrir la base cada vez, a menos que la cierres tan pronto como dejes de usarla. Abrir y cerrar la base en cada llamada a la función puede ser detrimental para el rendimiento, quizás sea mejor que conserves un objeto connection y lo reuses.


Saludos.