Является ли это хорошим или плохим способом использования цепочки конструкторов? (... для тестирования)

Question

Мотивация для цепочки конструкторов классов здесь заключается в том, что у меня есть стандартный конструктор для основного использования моим приложением, а второй, который позволяет мне вводить макет и заглушку.

Это просто кажется уродливым «новым» в «: this (...)» вызовом и встречным интуитивным вызовом параметризованного конструктора из конструктора по умолчанию, я задавался вопросом, что здесь будут делать другие люди?

(FYI ->SystemWrapper)

using SystemWrapper; 

public class MyDirectoryWorker{ 

    // SystemWrapper interface allows for stub of sealed .Net class. 
    private IDirectoryInfoWrap dirInf; 

    private FileSystemWatcher watcher; 

    public MyDirectoryWorker() 
     : this(
     new DirectoryInfoWrap(new DirectoryInfo(MyDirPath)), 
     new FileSystemWatcher()) { } 


    public MyDirectoryWorker(IDirectoryInfoWrap dirInf, FileSystemWatcher watcher) 
    { 
     this.dirInf = dirInf; 
     if(!dirInf.Exists){ 
      dirInf.Create(); 
     } 

     this.watcher = watcher; 

     watcher.Path = dirInf.FullName; 

     watcher.NotifyFilter = NotifyFilters.FileName; 
     watcher.Created += new FileSystemEventHandler(watcher_Created); 
     watcher.Deleted += new FileSystemEventHandler(watcher_Deleted); 
     watcher.Renamed += new RenamedEventHandler(watcher_Renamed); 
     watcher.EnableRaisingEvents = true; 
    } 

    public static string MyDirPath{get{return Settings.Default.MyDefaultDirPath;}} 

    // etc... 
} 

Answer 1

Включение конструктора по умолчанию - это запах кода, поскольку теперь класс связан с конкретной реализацией IDirectoryInfoWrap. Чтобы сделать вашу жизнь проще, используйте контейнер IOC, внешний для класса, для ввода различных зависимостей в зависимости от того, используете ли вы тестовый код или основное приложение.

Answer 2

Это довольно много, как я это делаю.

class MyUnitTestableClass 
{ 
    public MyUnitTestableClass(IMockable foo) 
    { 
     // do stuff 
    } 

    public MyUnitTestableClass() 
     : this(new DefaultImplementation()) 
    { 
    } 
}