Powinieneś otworzyć połączenie i powinieneś również użyć parametrów sql. Mam nadzieję, że jest to poprawna składnia oracle, ponieważ nie mogę jej przetestować:
using(var con = new OracleConnection("ConnectionString Here"))
using(var cmd = new OracleCommand("ADD YOUR INSERT/UPDATE/DELETE", con))
{
con.Open();
cmd.ExecuteNonQuery();
using (var cm = new OracleCommand("select round(avg(rating),1)As AvgRating from rates where id_rec = @id", con))
{
cm.Parameters.AddWithValue("@id", id);
using (var reader = cm.ExecuteReader())
{
if (reader.Read())
{
textBox5.Text = reader.GetInt16(0).ToString();
}
}
}
}
Zauważ, że użyłem using
— oświadczenie, aby zapewnić, że wszystkie niezarządzane zasoby zostaną usunięte tak szybko, jak to możliwe. Zamyka również połączenia (nawet w przypadku błędu).
Edytuj :Ponieważ wybierasz tylko jedną wartość, sugeruję użycie ExecuteScalar
:
using (var cm = new OracleCommand("select round(avg(rating),1)As AvgRating from rates where id_rec = @id", con))
{
cm.Parameters.AddWithValue("@id", id);
object avgRating = cm.ExecuteScalar();
if (!(avgRating is DBNull))
{
textBox5.Text = avgRating.ToString();
}
}