ExecuteReader

Några tips för optimering av följande kod?

        public static SqlDataReader ExecuteReader(string Sql)
        {
            SqlCommand Command = new SqlCommand(Sql, Connection);

            try
            {
                if (Command.Connection.State != ConnectionState.Open)
                {
                    Command.Connection.Open();
                }
            }
            catch (Exception)
            {
                throw;
            }

            return Command.ExecuteReader(CommandBehavior.CloseConnection);
        }

kolla på

 using

i msdn.

P skrev:

kolla på

 using

i msdn.

Håller inte med, using skulle inte hjälpa ett skvatt i detta fall. Du kan på din höjd förkorta koden lite.

        public static SqlDataReader ExecuteReader(string Sql)
        {
            SqlCommand Command = new SqlCommand(Sql, Connection);

            try{
                if (Command.Connection.State != ConnectionState.Open){
                    Command.Connection.Open();
                    return Command.ExecuteReader(CommandBehavior.CloseConnection);
                }
            }catch (Exception){
                throw;
            }
        }

Frågan är var Connection kommer ifrån..

emission skrev:

Frågan är var Connection kommer ifrån..

Statisk klassvariabel?