[SOLUCIONADO] Error convirtiendo

[yonerflash]

Hola tengo un valor en un textbox y lo quiero guardar en la base de datos como entero, trato de convertirlo usando Convert.ToInt32 pero me sale error diciéndome que Convert no existe. Este es el código:
string connStr = "server=localhost;user=root;database=bd_personas;p ort=3306;password=1234;";
MySqlConnection conn = new MySqlConnection(connStr);
string query = "INSERT INTO tbl_personas (id, nombres, apellidos) VALUES (Convert.ToInt32(texCedula.Text), 'Jhon', 'Hernandez')";

[alexg88]

No puedes usar métodos de C# dentro de consultas SQL. Lo que tienes que hacer es sacar la función de conversión fuera.

Con String.Format:

Código C:

string query = String.Format("INSERT INTO tbl_personas (id, nombres, apellidos) VALUES ({0},'{1}','{2}')", Convert.ToInt32(texCedula.Text), "Jhon", "Hernandez");
 

Como tú querías hacerlo, pero bien:

Código C:

string query = "INSERT INTO tbl_personas (id, nombres, apellidos) VALUES (" +  Convert.ToInt32(texCedula.Text) + "'Jhon', 'Hernandez')";
 

[yonerflash]

Muchas gracias, pero me dice ahora que hay un error en la sintaxis de sql, en esa línea

Lo arreglé poniéndole comillas sencillas
'" + Convert.ToInt32(texCedula.Text) + "'

[gnzsoloyo]

Dudo que lo arreglaras bien. Si te fijas con cuidado, el problema es que se generan dos datos y estás intentando insertar tres columnas.
Falta una coma de separación entre el primer dato (convertido) y el segundo (el nombre):

Código vb:

string query = "INSERT INTO tbl_personas (id, nombres, apellidos) 
VALUES (" +  Convert.ToInt32(texCedula.Text) + ", 'Jhon', 'Hernandez')";
 

[Malenko]

Ninguna de las soluciones es "buena" xD

Veamos, tenemos una cadena de texto (un textbox) que contiene un número. Y quieres pasarlo a número para luego pasarlo a cadena ya que lo estas concatenando en la query... No tiene mucho sentido.

Soluciones que yo usaría:

Código C:

string query = "INSERT INTO tbl_personas (id, nombres, apellidos) VALUES (" +  texCedula.Text + ", 'Jhon', 'Hernandez')";
 

Código C:

string query = String.Format("INSERT INTO tbl_personas (id, nombres, apellidos) VALUES ({0}, 'Jhon', 'Hernandez')", texCedula.Text);
 

E incluso usando un StringBuilder, pero hacer la conversión a entero en este caso no tiene ningún sentido.

[gnzsoloyo]

Personalmente, prefiero usar parametrizaciones de las consultas.
¿no sería algo mejor?

[Malenko]

Iniciado por gnzsoloyo

Personalmente, prefiero usar parametrizaciones de las consultas.
¿no sería algo mejor?

Lo ideal efectivamente sería usar un store procedure y pasarle los parametros.

De todas formas si analizamos lo que hace el insert no tiene mucho sentido. El ID tendría que ser PK y autoincremental, almenos yo lo haría así, de forma que solo "insertas" nombre y apellidos.

[gnzsoloyo]

No hace falta exactamente un SP para usar parametrizaciones en una query. Basta con poner los parámetros con que se reemplazará.
Supongamos este ejemplo:

Código vb:

string query = "INSERT INTO tbl_personas (id, nombres, apellidos) VALUES (@cedula, @nom, @ape)";
MySqlCommand cmd = new MySqlCommand(query, conn);
cmd.Parameters.AddWithValue("@cedula", Convert.ToInt32(texCedula.Text));
cmd.Parameters.AddWithValue("@nom", "John");
cmd.Parameters.AddWithValue("@ape", "Hernandez");
 

En esencia, la parametrización es aplicable a cualquier objeto MysqlCommand. El uso de los mismos en el caso de los SP es habitual, pero no es exclusivo.
Esto implica mayor codificación, pero también más seguridad y control de tipos, así como detectar mejor las excepciones que se produzcan por errores de dato.

MySQL Reference Manual:: MysqlCommand

[Malenko]

Lo del SP lo decía porque, aunque en este caso no sería aplicable, el gestor de BBDD compila el plan de ejecución la primera vez y las siguientes llamadas (en teoría) son más optimas. Además, no se en MySQL pero en SQL Server puedes usar el comando "Optimize for" para que cree un plan de ejecución optimo para unos valores en especial (los que más se consulten, por ejemplo).

[gnzsoloyo]

Lo pensé, pero al mirar qué conector usaba vi que era MySQL, entonces la cosa cambia.
En MySQL si bien se "compila" los SP, estos quedan en realidad almacenados en la tabla correspondiente, pero una cosa no elimina la otra. El conector es el que hace la conversión de datos, y está preparado para parametrizar tanto para SP como para consultas comunes.
Una diferencia esencial entre ambos DBMS es que en los SP los parámetros no se indican en el prototipo del SP como variables con "@". En MySQL van sin nada, porque la "@" está reservada a un tipo especial de variables, que no son sinónimo de las locales del SP.
Por supuesto que para mayor seguridad, es mejor usar SP, pero eso es una decisión de diseño.

Veremos que nos dice.