dois-je utiliser une classe statique DbUtil pour le web, est-ce dangereux?

Question

est-il autorisé à utiliser quelque chose comme ça dans le web: (ma demande est sur mvc de asp.net)

public static class DbUtil 
{ 
     public static int Insert(object o, string cs) 
     { 
      using (var conn = new SqlConnection(cs)) 
      using (var cmd = conn.CreateCommand()) 
      { 
       ... 

       conn.Open(); 
       return Convert.ToInt32(cmd.ExecuteScalar()); 
      } 
     } 
} 

utilisation:

public class Repository<T> : IRepository<T> 
{ 
     public virtual int Insert(T o) 
     { 
      return DbUtil.Insert(o, Cs); 
     } 
} 

et après l'injection du constructeur dans le service ou le contrôleur

public MyController(
     IRepository<Organization> organizationRepository) 
    { 
     this.organizationRepository = organizationRepository; 
    } 

Answer 1

il est tout à fait correct d'utiliser cette classe statique aussi longtemps qu'il est caché et enveloppé derrière le référentiel tel qu'il est dans votre cas . Cela permet un couplage plus faible entre la logique du contrôleur utilisant le référentiel et l'accès aux données. La méthode statique que vous avez montrée semble parfaitement reentrant ce qui le rend sûr pour les threads.

Answer 2

Comme indiqué, votre méthode statique n'a aucun problème de concurrence, car elle ne fonctionne sur aucune donnée partagée et chaque appel instancie ses propres objets de connexion et de commande locaux.

Cependant, il ne semble pas que vous gagniez beaucoup à rendre la méthode statique. En général, vous préférerez les méthodes d'instance plutôt que les méthodes statiques, car les appels de méthodes statiques ne peuvent pas être modifiés lors des tests.