0

En mi proyecto de mysql con python, al tratar de modifcar un campo de la tabla me genera error la secuencia sql y no entiendo por qué. Me acepta el cambio de nombre pero cuando digito el nuevo precio surge el error. El elemento Cod es la llave primaria de mi tabla.

Este es mi código:

sql="update productos set Nombre='"+nombre+"'Precio='"+precio+"' where Cod="+cod    
mysql.connection.query(self, query)

Y el error que lanza:

_mysql_exceptions.ProgrammingError: (1064, "You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'Precio='120' where Cod=1' at line 1")

5
  • intentastes dar un espacio en Precio campo de la tabla el 5 jul. 2017 a las 17:59
  • Creo que tienes un lío con las comillas simples, revisa esto y en especial, precio es un dato numérico? no debería ir sin comillas? el 5 jul. 2017 a las 18:01
  • ok, voy a evaluar. el 5 jul. 2017 a las 18:05
  • hice el cambio de las comillas, ya me acepta el cambio sin error, pero me concatena nombre y precio en un solo campo? el 5 jul. 2017 a las 18:35
  • si amigo ese era el problema comillas y coma que estaba afectando. gracias nuevamente. el 5 jul. 2017 a las 18:48

2 respuestas 2

0

Debes separar cada campo con una coma, en tu caso quieres actualizar Nombre y Precio, es decir, entre estos dos va una coma.

NOTA: En el caso de que tu Precio sea un tipo de dato entero, este no debe llevar comillas simples.

sql="update productos set Nombre='"+nombre+"', Precio="+precio+" where Cod="+cod+"
1
  • correcto asi es. sin discusion, gracias por tu orientacion. el 5 jul. 2017 a las 18:49
0

Una sugerencia, la forma en la que estás ejecutando tu query es insegura. Imagina que tu variable nombre contenga una sentencia para eliminar una tabla usando un DROP TABLE. A esto se le conoce como SQL injection.

En vez de ejecutar tu query de esa forma te recomiendo hacerlo usando cursores y parámetros. No estoy seguro de la librería para MySQL que estás usando pero básicamente sería algo como esto:

connection = mysql.connector.connect(...)
cursor = connection.cursor()
query = '''
    UPDATE productos 
    SET Nombre=%s, Precio=%s 
    WHERE Cod=%s
'''
params = (
    nombre,
    precio,
    cod
)
cursor.execute(query, params)

No sólo es más seguro, también es más legible.

1
  • 1
    entendido gracias el 8 jul. 2017 a las 17:32

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.