Erreur avec OracleDataReader. Erreur: opération non Valide. La connexion est fermée

Lorsque j'essaie d'attribuer le lecteur C# déclenche une exception:

Invalid operation. The connection is closed

J'essaie d'obtenir un résultat à partir d'une requête qui renvoie une seule cellule avec une valeur moyenne à l'intérieur.
cmd est un oraclecomand que j'utilise pour insérer une ligne dans une table et c'est très bien. Je vois la boîte de message suivante et après que l'exception s'affiche.

          try
            {
                cmd.ExecuteNonQuery();
                MessageBox.Show("Recipe Rated");
                OracleCommand cm = new OracleCommand("select round(avg(rating),1) from rates where id_rec = "+id);
                OracleDataReader reader = cm.ExecuteReader();
                reader.Read();
                textBox5.Text =""+reader.GetInt16(0);
            }
InformationsquelleAutor Alek | 2013-06-12
  1. 1

    Lorsque vous utilisez `OracleCommand', vous devez associer un valide OracleConnection objet.

      using (OracleConnection connection = new OracleConnection(connectionString))
    {
                   MessageBox.Show("Recipe Rated");
                   OracleCommand cm = new OracleCommand("select round(avg(rating),1) from rates where id_rec = "+id);
                   try
                    {

                        cm.Connection = connection;
                        connection.Open(); //oracle connection object
                        OracleDataReader reader = cm.ExecuteReader();
                        reader.Read();
                        textBox5.Text =""+reader.GetInt16(0);
                    }
   }

    Espérons que cette aide.

    Grâce.

    InformationsquelleAutor FIre Panda
  2. 3

    Vous devez ouvrir la connexion et vous devez également utiliser sql-paramètres. J'espère que c'est la bonne oracle syntaxe parce que je ne peux pas le tester:

    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();
            }
        }
    }
}

    Noter que j'ai utilisé le using-déclaration pour s'assurer que toutes les ressources non managées sont éliminés dès que possible. Il ferme également les connexions (même par erreur).

    Modifier: Depuis que vous êtes de ne sélectionner qu'une seule valeur, je suggère l'utilisation de 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();
    }
}
    • +1 pour le rendre sûr et propre
    • Vous êtes de droite. J'ai oublié d'ajouter le lien dans le constructeur de la cm. Merci pour la réponse rapide
    InformationsquelleAutor Tim Schmelter