Tener el método Conectar
como static
, no es necesariamente malo, aunque no entiendo por qué lo tienes como protected
, eso no me parece correcto.
Lo que definitivamente no está bien es que estés guardando el resultado en una variable estática global conexion
. Normalmente, este patrón se usa con la intención de compartir una conexión global para el app entero (aunque en tu caso, estás reasignando la variable a cada llamada, lo que es raro en sí). Compartir una conexión global nunca es buena idea, particularmente en apps multi usuarios (multi hilos), como debes ser tu caso, ya que se trata de una página web.
La razón por la que no es aconsejable es que, si más de un usuario trata de efectuar una acción al mismo tiempo y tratan de usar la misma conexión, varias cosas peligrosas pueden suceder. He aquí algunos ejemplos:
- Dependiendo de la base de datos, puede lanzarte excepciones por tratar de manejar varias result sets al mismo tiempo.
- Puedes corromper tus datos si las transacciones de los 2 usuarios se mezclan juntos.
Mas bien, permite que cada usuario (hilo) abra y cierre la conexión según la necesidad, idealmente usando un connection pool para minimizar el costo de "abrir" una conexión, aunque lo típico es que esto se haga automáticamente según el connection string que uses (no sé cómo funciona con Firebird).
En tu lugar, yo restructuraría la clase de esta forma (nota que el connection string sí queda cómo estático, eso no es un problema. Pero la variable conexion
en sí queda eliminida de la clase):
public static class ConexionFB
{
private static readonly string ConnectionString;
static ConexionFB()
{
var parametros_conexion = new FbConnectionStringBuilder();
parametros_conexion.DataSource = "localhost";
parametros_conexion.Port = 3050;
parametros_conexion.Database = @"C:\FirebirdDB\USUARIOS.FDB";
parametros_conexion.UserID = "SYSDBA";
parametros_conexion.Password = "masterkey";
parametros_conexion.ServerType = FbServerType.Default;
ConnectionString = parametros_conexion.ToString();
}
public static FbConnection Conectar()
{
return new FbConnection(ConnectionString);
}
}
Ejemplo de cómo usarlo:
using(var connexion = ConexionFB.Conectar())
{
// ... usar conexión aquí...
}
Pequeña nota adicional: aunque no conozco Firebird, sospecho que no sea buena idea de usar sysdba
para la conexión. Suena cómo que estás usando un usuario administrador con demasiados permisos.