0

Hola el problema es que la conexión esta bien solo cuando estoy utilizando el ODBC, pero cuando lo cambio al proveedor de PostgreSQL Npgsql, ya no me deja conectar me dice que error de sintaxis, yo creo que el error es en la linea de conexion:

try {

-->  NpgsqlConnection cn = new NpgsqlConnection("SERVER=localhost; UID=postgres;PWD=damaris;DATABASE=ProyectoAsistenciaCCB;");

    cn.Open();

    NpgsqlCommand cmd = new  NpgsqlCommand("INSERT INTO reportes VALUES(?, ?, ?, ?, ?)",cn);

    foreach (DataGridViewRow row in dataGridView1.Rows)
    {
        cmd.Parameters.Clear();

        cmd.Parameters.Add("@p1", NpgsqlDbType.Numeric).Value = Convert.ToInt32(row.Cells[0].Value);

        cmd.ExecuteNonQuery();
        cn.Close();
    }
}
3
  • 2
    Sería apreciado que formatees tu pregunta para que sea legible para futuros lectores también. En este momento, el código no se puede leer correctamente. Y siempre es bueno que seas específico en cuanto al error que tuvistes, y en qué línea. Nuevamente, se trata de beneficiar a los futuros lectores también.
    – sstan
    Commented el 22 oct. 2016 a las 19:26
  • ok gracias lo tendre en cuenta
    – karol
    Commented el 24 oct. 2016 a las 15:18
  • 1
    Mucho mejor. Pero aun falta que especifiques cuál línea produce el error. Y de paso, te aviso que debe ser un error el que tengas cn.Close() adentro del foreach.
    – sstan
    Commented el 24 oct. 2016 a las 15:26

2 respuestas 2

1

Esta pregunta es un buen ejemplo del por qué es sumamente importante que incluyas toda la información pertinente en tu pregunta, incluyendo el mensaje de error exacto, y también el stack trace.

De haberlo hecho, hubiera sido claro desde el principio que el error no tiene nada que ver con la conexión de base de datos. Mas bien, tu error sucede al ejecutar:

cmd.ExecuteNonQuery(); // verifica que de aquí viene tu error.

Y la razón del error es que no le gusta el SQL, particularmente el símbolo ? que usas para los parámetros:

INSERT INTO reportes VALUES(?, ?, ?, ?, ?)

Con Npgsql, los parámetros deben pasarse usando la sintaxis siguiente (los nombres que escojas no tienen mayor importancia, siempre y cuando uses @ en frente):

INSERT INTO reportes VALUES(@p1, @p2, @p3, @p4, @p5)

El siguiente problema que tienes es que solo le pasas un parámetro a la sentencia (@p1). Pero ¿qué hay de los 4 otros valores que el INSERT necesita?

Y finalmente, no tiene sentido que ejecutes cn.Close() dentro del foreach. Piénsalo bien, y verás por qué. De hecho, no hay necesidad de ejecutar cn.Close() explícitamente si usas using para instanciar la conexión a la base de datos. Esto es mejor práctica, y te protege de situaciones de errores que pueden resultar en que la conexión nunca se cierre.

Te dejo un modelo a lo que debería parecerse tu código para que funcione correctamente:

using(var cn = new NpgsqlConnection("SERVER=localhost; UID=postgres;PWD=damaris;DATABASE=ProyectoAsistenciaCCB;"))
{
    cn.Open();

    using(var cmd = new NpgsqlCommand("INSERT INTO reportes VALUES(@p1, @p2, @p3, @p4, @p5)", cn))
    {
        foreach (DataGridViewRow row in dataGridView1.Rows)
        {
            cmd.Parameters.Clear();

            cmd.Parameters.Add("@p1", NpgsqlDbType.Numeric).Value = Convert.ToInt32(row.Cells[0].Value);

            // Solo un ejemplo, pero necesitas pasar los 4 siguientes parámetros también...
            cmd.Parameters.Add("@p2", NpgsqlDbType.Numeric).Value = ???;
            cmd.Parameters.Add("@p3", NpgsqlDbType.Numeric).Value = ???;
            cmd.Parameters.Add("@p4", NpgsqlDbType.Numeric).Value = ???;
            cmd.Parameters.Add("@p5", NpgsqlDbType.Numeric).Value = ???;

            cmd.ExecuteNonQuery();
        }
    }
}
4
  • ok perdon no supe como plasmar mi error entonces ya no es necesario poner un try{ }catch{ ???
    – karol
    Commented el 24 oct. 2016 a las 18:07
  • Lo que puse es solo un modelo. En el código de tu pregunta, no hay catch, así que no puedo juzgar si es necesario o no. Nuevamente, la intención de mi respuesta es de ayudarte a identificar tu problema y darte ideas para mejorar el código. Pero tu tienes que investigar el asunto a fondo para saber lo que es apropiado en tu caso.
    – sstan
    Commented el 24 oct. 2016 a las 18:10
  • ok pues muchas gracias mi es de mucho apoyo tu comentario gracias
    – karol
    Commented el 24 oct. 2016 a las 18:20
  • ¿Qué debo hacer cuando alguien contesta mi pregunta?
    – sstan
    Commented el 24 oct. 2016 a las 19:07
0

Según la documentación de Npgsl el formato es el siguiente:

NpgsqlConnection("Host=myserver;User Id=mylogin;Password=mypass;Database=mydatabase"))

Prueba a reemplazar UID y PWD por Username y Password.

http://www.npgsql.org/doc/connection-string-parameters.html

14
  • ok deja cambiarlo
    – karol
    Commented el 22 oct. 2016 a las 18:04
  • @karol: Si esta respuesta te ayudó a resolver tu problema, sería una muestra de aprecio que le des un voto a favor, y que además la marques como aceptada (haz click en el "check" a la izquierda de la respuesta). Gracias.
    – sstan
    Commented el 22 oct. 2016 a las 19:27
  • hola mmm no aun sigue el error me dice que error de sintaxis
    – karol
    Commented el 24 oct. 2016 a las 15:19
  • @karolprincess puedes hacer un paste de la sentencia?
    – McNets
    Commented el 24 oct. 2016 a las 15:21
  • de cual? .......
    – karol
    Commented el 24 oct. 2016 a las 15:22

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.