Tengo mi lógica empresarial implementada en clases estáticas simples con métodos estáticos. Cada uno de estos métodos abre / cierra la conexión SQL cuando se llama:
public static void DoSomething(string something)
{
using (SqlConnection connection = new SqlConnection("..."))
{
connection.Open();
// ...
connection.Close();
}
}Pero creo que evitar abrir y cerrar una conexión ahorra rendimiento . Hice algunas pruebas hace mucho tiempo con la clase OleDbConnection (no estoy seguro sobre SqlConnection), y definitivamente ayudó a trabajar así (por lo que recuerdo):
//pass the connection object into the method
public static void DoSomething(string something, SqlConnection connection)
{
bool openConn = (connection.State == ConnectionState.Open);
if (!openConn)
{
connection.Open();
}
// ....
if (openConn)
{
connection.Close();
}
}Entonces la pregunta es: ¿debo elegir el método (a) o el método (b)? Leí en otra pregunta de stackoverflow que la agrupación de conexiones me ahorró rendimiento, no tengo que molestarme en absoluto ...
PD. Es una aplicación ASP.NET: las conexiones solo existen durante una solicitud web. No es una aplicación o un servicio para ganar.
Un detalle que falta es cómo este método es parte de una solicitud de página. ¿Es el único método llamado o es, como asumí en mi respuesta, uno de los muchos métodos que se llama en una página solicitada, afecta qué respuesta es correcta?)
—
David Mårtensson
David - un montón de métodos de este tipo están siendo llamados :)
—
Alex
El caso A muestra una falta de fe en Dispose: consulte stackoverflow.com/questions/1195829/… y el ejemplo en MSDN msdn.microsoft.com/en-us/library/…
—
user2864740
DbConnection.StateChangeevento para monitorear los cambios en el cambio de estado de la conexión (y puede almacenarse localmente) en lugar de verificar laDbConnection.Statepropiedad directamente. Le ahorrará costes de rendimiento.