[4.0-C#] Problema apertura/chiusura connessione

[fermat]

ho questa classe:

    public class Decoder
    {
        private SqlConnection conn = new SqlConnection("Data Source=10.0.5.131;Initial Catalog=TELENEWS;Persist Security Info=True;User ID=sa;Password=sa");

        public DataSet ddlNetwork()
        {
            SqlDataAdapter adapter = new SqlDataAdapter("select_soluzioni", conn);
            try
            {
                conn.Open();
                DataSet ds = new DataSet();
                adapter.Fill(ds);
                return ds;
            }
            catch (Exception ex)
            {
                throw new Exception(ex.Message);
            }
            finally
            {
                conn.Close();
            }
        }

        public SqlDataReader ddlGruppi(int soluzione)
        {
            SqlDataReader reader = null;
            try
            {
                SqlCommand cmd = new SqlCommand("select_gruppi_soluzione", conn);
                cmd.CommandType = CommandType.StoredProcedure;
                cmd.Parameters.AddWithValue("@soluzione#", soluzione);
                cmd.Connection = conn;
                conn.Open();
                reader = cmd.ExecuteReader();
                return reader;
            }
            catch (Exception ex)
            {
                throw new Exception(ex.Message);
            }
            finally
            {
                if (reader != null)
                {
                    reader.Close();
                }
                conn.Close();
            }
        }
    }

ottengo sempre questo errore:
Invalid attempt to call FieldCount when reader is closed.

ovviamente se levo il primo metodo nn lo ottengo.
ma nn capisco il motivo.

[GabbOne]

sei sicuro di non avere il problema sul metodo ddlGruppi?

[fermat]

in origine era così:

    public class Decoder
    {
        private SqlConnection conn = new SqlConnection("Data Source=10.0.5.131;Initial Catalog=TELENEWS;Persist Security Info=True;User ID=sa;Password=sa");

        public DataSet ddlNetwork()
        {
            SqlDataAdapter adapter = new SqlDataAdapter("select_soluzioni", conn);
            try
            {
                DataSet ds = new DataSet();
                adapter.Fill(ds);
                return ds;
            }
            catch (Exception ex)
            {
                throw new Exception(ex.Message);
            }
        }


        public DataSet ddlGruppi()
        {
            SqlDataAdapter adapter = new SqlDataAdapter("select_gruppi", conn);
            try
            {
                DataSet ds = new DataSet();
                adapter.Fill(ds);
                return ds;
            }
            catch (Exception ex)
            {
                throw new Exception(ex.Message);
            }
        }

    }

così funziona, ma la SP di ddlGruppi dovrebbe essere un'altra, che richiede un parametro.
ho cambiato DataSet in DataReader nel metodo dllGruppi proprio perchè quella SP richiede un parametro.
cambiando come prima nn funziona più.

[fermat]

mo funziona:

        private SqlConnection conn = new SqlConnection("Data Source=10.0.5.131;Initial Catalog=TELENEWS;Persist Security Info=True;User ID=sa;Password=sa");

        public DataSet ddlNetwork()
        {
            SqlDataAdapter adapter = new SqlDataAdapter("select_soluzioni", conn);
            try
            {
                DataSet ds = new DataSet();
                adapter.Fill(ds);
                return ds;
            }
            catch (Exception ex)
            {
                throw new Exception(ex.Message);
            }
        }

        public SqlDataReader ddlGruppi(int soluzione)
        {
            SqlDataReader reader = null;
            SqlCommand cmd = null;
            try
            {
                conn.Open();
                cmd = new SqlCommand("select_gruppi");
                cmd.Connection = conn;
                cmd.Parameters.AddWithValue("@soluzione#", soluzione);
                reader = cmd.ExecuteReader();
                return reader;
            }
            catch (Exception ex)
            {
                throw new Exception(ex.Message);
            }
        }

nn capisco proprio......

[fermat]

per la cronaca il problema è il finally commentato:

        private SqlConnection conn = new SqlConnection("Data Source=10.0.5.131;Initial Catalog=TELENEWS;Persist Security Info=True;User ID=sa;Password=sa");

        public DataSet ddlNetwork()
        {
            SqlDataAdapter adapter = null;
            DataSet ds = null;
            try
            {
                conn.Open();
                adapter = new SqlDataAdapter("select_soluzioni", conn);
                ds = new DataSet();
                adapter.Fill(ds);
                return ds;
            }
            catch (Exception ex)
            {
                throw new Exception(ex.Message);
            }
            finally
            {
                ds.Dispose();
                adapter.Dispose();
                conn.Close();
            }
        }

        public SqlDataReader ddlGruppi(int soluzione)
        {
            SqlDataReader reader = null;
            SqlCommand cmd = null;
            try
            {
                conn.Open();
                cmd = new SqlCommand("select_gruppi_soluzione", conn);
                cmd.CommandType = CommandType.StoredProcedure;
                cmd.Parameters.AddWithValue("@soluzione#", soluzione);
                reader = cmd.ExecuteReader();
                return reader;
            }
            catch (Exception ex)
            {
                throw new Exception(ex.Message);
            }
            //finally
            //{
            //    conn.Close();
            //}
        }

senza commento da errore.

[djciko]

NON funziona:

     SqlCommand cmd = new SqlCommand("select_gruppi_soluzione", conn);

Funziona:

      cmd = new SqlCommand("select_gruppi");

mi sembrano due stored diverse...non ho capito.

PS: il Datareader non è un oggetto DISCONNESSO come il Dataset

[fermat]

Originariamente inviato da djciko
NON funziona:

     SqlCommand cmd = new SqlCommand("select_gruppi_soluzione", conn);

Funziona:

      cmd = new SqlCommand("select_gruppi");

mi sembrano due stored diverse...non ho capito.

PS: il Datareader non è un oggetto DISCONNESSO come il Dataset

entrambe funzionano se il finally è commentato.
entrambe nn funzionano se il finally non è commentato.
sono due SP diverse, ma simili quindi andrebbero bene entrambe.
il problema è quel finally che volevo usare per chiudere la connessione.

[djciko]

secondo me non è il finally, ma la chiusura della connessione che c'e' dentro.
Ti ripeto, l'SQLDataReader è un oggetto connesso, non in memoria come il Dataset

Questo è il primo metodo che usavi, chiudevi sia il reader che la connessione.
Il finally viene eseguito sempre, è un problema di flusso e logica, l'errore che ottenevi infatti te lo diceva: "Invalid attempt to call FieldCount when reader is closed".
Inoltre non restituisci il reader nel punto giusto secondo me, andrebbe restituito alla fine...

public SqlDataReader ddlGruppi(int soluzione)
        {
            SqlDataReader reader = null;
            try
            {
                SqlCommand cmd = new SqlCommand("select_gruppi_soluzione", conn);
                cmd.CommandType = CommandType.StoredProcedure;
                cmd.Parameters.AddWithValue("@soluzione#", soluzione);
                cmd.Connection = conn;
                conn.Open();
                reader = cmd.ExecuteReader();
                return reader;
            }
            catch (Exception ex)
            {
                throw new Exception(ex.Message);
            }
            finally
            {
                if (reader != null)
                {
                    reader.Close();
                }
                conn.Close();
            }
        }

[fermat]

ho provato più volte anche così ma da sempre lo stesso risultato.
forse è dove restituisco il reader come dici te.

[djciko]

il metodo che ho scritto nel post precedente e' quello errato, l'ho riportato per evidenziare l'errore.

ok, cmq hai risolto