¿Es esta una buena o mala manera de usar el encadenamiento de constructores? (... para permitir la prueba)

Question

Mi motivación para encadenar los constructores de mi clase aquí es que tengo un constructor por defecto para el uso general de mi aplicación y un segundo que me permite inyectar un simulacro y un talón.

Parece un poco feo 'nuevo' -ing cosas en el ": esto (...)" llamada y contra-intuitivo llamando a un constructor parametrizado desde un constructor predeterminado, me pregunté qué harían otras personas aquí?

(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

La inclusión de un constructor por defecto es un olor código como ahora la clase se acopla a la aplicación concreta de la IDirectoryInfoWrap. Para hacerte la vida más fácil, utiliza un contenedor IOC externo a la clase para inyectar dependencias diferentes dependiendo de si estás ejecutando el código de prueba o la aplicación convencional.

Answer 2

Eso es más o menos como lo hago.

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

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