这种依赖注入的设计容易理解吗?

我在一个有大量现有代码库的项目中工作,我的任务是对其中的一小部分进行重新设计。现有的代码库没有太多的单元测试,但我希望至少我正在研究的部分有一整套具有良好代码覆盖率的单元测试。

1. 我提取了我的“协调器”类所依赖的所有非平凡类的接口。其他开发人员勉强同意了这一点,只是有些小小的抱怨。

2. 我修改了coordinator类,以便只引用其操作的接口。

.

public sealed class Coordinator
{
    private IFactory _factory;

    public Coordinator(int param)
        : this(param, new StandardFactory())
    {            
    }

    public Coordinator(int param, IFactory factory)
    {
        _factory = factory;
        //Factory used here and elsewhere...
    }

    public interface IFactory
    {
        IClassA BuildClassA();

        IClassB BuildClassB(string param);

        IClassC BuildClassC();
    }

    private sealed class StandardFactory : IFactory
    {
        public IClassA BuildClassA()
        {
            return new ClassA();   
        }

        public IClassB BuildClassB(string param)
        {
            return new ClassB(param);
        }

        public IClassC BuildClassC()
        {
            return new ClassC();
        }
    }
}

这允许注入stubbed工厂来返回模拟的依赖项以进行单元测试。我担心的是,这也意味着这个类的任何用户都可以提供自己的工厂来改变这个类的操作方式,这并不是真正的意图。这个构造函数纯粹是用于单元测试的,但是它的副作用是意味着需要额外的扩展性。在我使用的其他类中,我通常看不到这种模式,这让我怀疑是否过于复杂,是否有更好的方法来实现所需的隔离。

注意:代码格式有点奇怪,所以我为上面糟糕的格式道歉。不知道为什么它不能让我正确格式化它。