如何使用泛型来处理 Action<IBuilder<T>> 中的特定类型配置？

Question

我正在尝试实现一个复杂的构建器来帮助我的测试环境。为此，我重构了我的代码以拥有一个方法：

public TestContext Add<T>(Action<IBuilder<T>> configurator) where T : class, new()
{
    IBuilder<T> builder = CreateBuilderOf<T>();
    configurator(builder);
    T item = builder.GetItem();
    RepositoryOf<T>().Insert(item);
    SetCurrent(item);
    return this;
}

当我需要在调用方法时指定配置时出现问题：

TestContext.Instance.Add<Person>(personBuilder => ((PersonBuilder)personBuilder).Name("SMITH"));

我需要能够使用配置器中的类型特定方法，这些方法由具体的构建器实现，例如：

public PersonBuilder : IBuilder<Person>
{
  private Person Item;

  public PersonBuilder() { Item = new Person(); }

  public Name(string mame) { Item.Name = name; }

  public Person GetItem() { return Item; }
}

显然，不允许将Action<PersonBuilder> 作为Action<IBuilder<Person>> 传递，即使PersonBuilder 实现了IBuilder<Person>，因此是演员表。

我非常愿意：

• 不需要在 lambda 中进行转换，而是在它的开头，例如
  (PersonBuilder personBuilder) => personBuilder.Name("SMITH")，但这归结为Action<PersonBuilder> 的一个实例，因此同样无效;
• 在 Add 的参数中使用诸如 BuildSimplePerson(PersonBuilder builder) 之类的函数：Add<Person>(BuildSimplePerson)

我想我可以通过两个BuildSimplePersons 实现进行类型转换，例如：

private void BuildSimplePerson(IBuilder<Person> builder)
{
  BuildSimplePerson(builder as PersonBuilder);
}
private void BuildSimplePerson(PersonBuilder builder)
{
  builder.Name("SMITH");
}

但这并不是一个真正优雅的解决方案。

我还意识到将Action<PersonBuilder> 作为Action<IBuilder<Person>> 传递是不正确的，因为我们不知道该函数的参数是否真的是PersonBuilder 或IBuilder<Person> 的任何其他实现。

我怎样才能做得更好？

Answer 1

正如我的评论已经指出的那样，问题在于您当前的代码假定 CreateBuilderOf<T> 返回一个 PersonBuilder 但它实际上可以返回任何实现 IBuilder<Person> 的东西，在这种情况下您的演员将失败。

您的代码看起来是通用的，但实际上并非如此。您总是希望在具体类 (PersonBuilder) 上工作，而不是在通用接口 IBuilder<Person> 上工作。

我的理解是，您需要一个通用的Add<T> 方法以避免在其中为每种类型重复该代码。

这是我的方法：

public TestContext Add<T>(IBuilder<T> builder) where T : class, new()
{
    T item = builder.GetItem();
    RepositoryOf<T>().Insert(item);
    SetCurrent(item);
    return this;
}

你可以这样称呼它：

TestContext.Instance.Add<Person>(CreatePersonBuilder().Name("SMITH"));

显然，您需要为每种要添加的类型提供CreateXBuilder 方法。但是，我认为您至少已经隐含地拥有了这一点，因为我认为您的 CreateBuilderOf<T> 方法无论如何都是一个巨大的 switch 语句。

如果您不想创建此类方法，另一种获取构建器的方法是通用方法，如下所示：

CreateBuilder<PersonBuilder>()

但实际上，这实际上只不过是一个new PersonBuilder()，所以你实际上可以简单地使用

TestContext.Instance.Add<Person>(new PersonBuilder().Name("SMITH"));

Configure 方法非常相似：

TestContext.Instance.Configure<Person>(id, p => new PersonBuilder(p).Name("SMITH"));

这将传递 ID，Configure 方法将使用它来查找对象，然后将其传递给回调。所以Configure 的第二个参数不是Action<IBuilder<T>>，而是Action<T>。

与您现有的代码相比，这种方法还有另一个优势：
您现有的代码不仅假定PersonBuilder 将是用于IBuilder<Person> 的实现。不，您的代码还假定它有一个没有参数的构造函数和一个接受Person 的构造函数。编译器无法验证这些假设。
使用我上面显示的代码，构建器实现可以毫无问题地采用其他参数，并且编译器将验证一切正常。