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();
}
}
}
cn.Close()
adentro delforeach
.