Which Is The Correct Way Of Opening A Connection?&>

Hi

I access a DB by as abvious creating a DB Connection and then sending  a querry to it. Finally creating an SQL Command and then exuting it.

 

But the thng is I am using a static function:

which opens a new connection and returns a data reader based on the querry. i.e

 

 

pubblic static SqlConnection OpenConnection()

{

try

{

string connectionString = "Data Source=mm\SQLEXPRESS;Initial Catalog=monsoonDB;Integrated Security=True";

SqlConnection conn = new SqlConnection(connectionString);

conn.Open();

return conn;

}

catch (Exception ex)

{

throw;

}

}

   

Now I have the execute selectquerry function.....

public static SqlDataReader ExecuteSelectQuerry(string querry)

{

try

{

SqlConnection conn = OpenConnection();

SqlCommand scomm = new SqlCommand(querry, conn);

SqlDataReader sdr = scomm.ExecuteReader();

return sdr;



}

catch (Exception e)

{

return null;

}

}
}

 

 

 

now my question is....the instance of conn is limited only to that function only, and not returned outside. Whereas only the SqlDataReader is returned outside. Does that have any abnormal affect on the application. Since my ASP Application is getting locked (not doing anything, nt even post back!) sometimes after a few DB operation.

 

Arent these connections and instances Managed (auto garabage collection)?? Can  memory leaks under such a situation occur.

Please help

 

Thanks