Cfrtjryk

Am I closing my SQL connection correctly, by placing it inside a "using" block?

This is how I grab a new connection, execute a query, and return the results:

using (SqlConnection objCS = DB.LMSAdminConn())

{

    objCS.Open();

    SqlCommand objCommand = new SqlCommand("SELECT TOP 1 * FROM users WHERE userid = @userid", objCS);

    objCommand.Parameters.Add("@revisionid", SqlDbType.Int).Value = userid;

    SqlDataReader reader = objCommand.ExecuteReader();

    while (reader.Read())

    {

        //do something

    }

    reader.Close();

}

The connection itself comes from this call:

public static SqlConnection LMSAdminConn()

{

    return new SqlConnection(ConfigurationManager.ConnectionStrings["lmsadmin"].ToString());

}

I am opening the connection inside a "using" block, and I thought that the connection would be closed as well, because it is opened inside the "using" block. But since the "new SqlConnection" object is actually generated from an outside class, is my connection still getting appropriately closed? SQL Server shows the connection as still being open, but I'm not sure if that is ADO.NET connection pool recycling / sharing, or if the connection is truly being held open.