构造函数中有许多参数的最佳实践?

时间:2015-12-03 10:38:14

标签: c#

如果我有一个需要访问5个不同存储库的视图,我现在正在这样做:

why would this be dropped or missing

这种方法的问题是构造函数越来越大,如果构造函数中的某些内容发生了变化,我必须在整个系统中编辑更改。如何避免这个问题?

我考虑过创建一个构建器类,它将负责创建存储库并将它们解析为视图。或者这是一个糟糕的方法?

2 个答案:

答案 0 :(得分:-3)

您可以使所有存储库接口派生自基接口IRepositoryBase,然后将IEnumerable<IRepositoryBase>传递给构造函数。

此外,由于您的存储库最终由MyClass以某种方式处理,因此它们可能具有一些共同的功能;您还可以在IRepositoryBase中为此常用功能添加方法,然后MyClass可以完成其工作,而无需了解其正在处理的存储库的实际类型。

示例代码:

public class MyClass
{
    private readonly IRepositoryBase[] repos; 

    public MyClass (IEnumerable<IRepositoryBase> r)
    {
        repos = r.ToArray();
    }

    // ...
}

答案 1 :(得分:-5)

您可以使用构建器模式初始化MyClass实例,而不是使用依赖注入(引入反射,使代码更慢,不安全且可读性更低!):

<强> MyClassBuilder.cs

public class MyClassBuilder {
    IRepo1 rep1;
    IRepo2 rep2;
    IRepo3 rep3;
    IRepo4 rep4;
    // ...

    public MyClassBuilder() {

    }

    public MyClassBuilder setRepo1(IRepo1 rep1) {
        this.rep1 = rep1;
        return this;
    }

    public MyClassBuilder setRepo2(IRepo2 rep1) {
        this.rep2 = rep2;
        return this;
    }

    public MyClassBuilder setRepo3(IRepo3 rep3) {
        this.rep3 = rep3;
        return this;
    }

    public MyClassBuilder setRepo4(IRepo4 rep4) {
        this.rep4 = rep4;
        return this;
    }
    // ...

    public MyClass build() {
        return new MyClass(
            rep1 ?? DEFAULT_REP1,
            rep2 ?? DEFAULT_REP2,
            rep3 ?? DEFAULT_REP3,
            rep4 ?? DEFAULT_REP4
            // ...
        );
    }
}

使用示例

MyClass myclass = new MyClassBuilder()
    .setRepo1(new Repo1(/*...*/))
    .setRepo4(new Repo4(/*...*/))
    .build();