Méthodes statiques et le localisateur de service

Question

J'ai une méthode d'aide statique dans mon application ASP.NET MVC. Voici un exemple de code qui illustre le problème (le code réel est un peu plus complexe). S'il vous plaît se concentrer sur le modèle. Le code réel n'interagit pas avec une base de données.

Le code ci-dessous est extrait pour permettre les tests unitaires. Le problème que j'ai est d'obtenir l'implémentation concrète pour IDatabaseManager. L'utilisation d'un localisateur de service est-elle mauvaise? Des recommandations sur les bonnes pratiques? J'ai lu que les localisateurs de service sont l'anti-pattern. S'il vous plaît donnez votre avis. Tous les autres endroits dans le code utilisent l'injection de dépendance (le problème ici est évidemment la méthode statique DataWriter).

public static class DataWriter { 
    public static void WriteDate() { 
     var datetime = DateTime.Now.ToString(); 

     //Is this service locator a bad pattern? 
     var databaseManager = StructureMap.ObjectFactory.Container.GetInstance<IDatabaseManager>(); 
     var dataRepository = new DataRepository(databaseManager); 

     dataRepository.Write(datetime); 
    } 
} 

public class DatabaseManager : IDatabaseManager { 
    public void Write(string s) { 
     //Write to database 
    } 
    //Other methods 
    ... 
    ... 
} 

public interface IDatabaseManager { 
    void Write(string s); 
    //Other signatures 
    ... 
    ... 
} 

public class DataRepository : IDataRepository { 
    private IDatabaseManager _databaseManager; 
    public DataRepository(IDatabaseManager databaseManager) { 
     _databaseManager = databaseManager; 
    } 

    public void Write(string data) { 
     _databaseManager.Write(data); 
    } 
} 

public interface IDataRepository { 
    void Write(string data); 
} 

Édition selon les recommandations d'Alex. Modification à nouveau pour supprimer les informations sur cshtml (comme ce n'est pas l'objet de la question)

Answer 1

J'ai lu que localisateurs de service sont les anti-modèle

Peut-être pas à 100%, mais ils devrait être découragé. Mark Seemann explique que Service Locator is an Anti-Pattern. Je pense que vous devriez regarder Dependency Injection.

Le problème que vous rencontrez est que la résolution de dépendance est intrinsèquement difficile avec les méthodes statiques.

Je vous suggérerais de vous intéresser à l'implémentation de méthodes non statiques et d'injection de dépendances provenant d'autres applications de votre application. Comme Alexei a souligné, appelant les choses d'un Voir n'est pas idéal.

ASP.NET MVC 4 Dependency Injection