架构问题：使用依赖注入导致垃圾 API

Question

我正在尝试创建一个类，该类执行各种与数据库相关的低级操作，但为 UI 层提供了一个非常简单的接口。

这个类代表了一堆数据，它们都在一个特定的聚合根中，由单个 ID int 检索。

构造函数有四个参数：

public AssetRegister(int caseNumber, ILawbaseAssetRepository lawbaseAssetRepository, IAssetChecklistKctcPartRepository assetChecklistKctcPartRepository, User user)
{
  _caseNumber = caseNumber;
  _lawbaseAssetRepository = lawbaseAssetRepository;
  _assetChecklistKctcPartRepository = assetChecklistKctcPartRepository;
  _user = user;
  LoadChecklists();
}

UI层通过接口IAssetRegister访问这个类。 Castle Windsor 可以自己提供 ILawbaseAssetRepository 和 IAssetChecklistKctcPartRepository 参数，但 UI 代码需要使用匿名类型提供另外两个，如下所示：

int caseNumber = 1000;
User user = GetUserFromPage();
IAssetRegister assetRegister = Moose.Application.WindsorContainer.Resolve<IAssetRegister>(new { caseNumber, user});

从 API 设计的角度来看，这是垃圾。 UI 层开发人员无法知道 IAssetRegister 需要一个整数和一个用户。他们需要知道类的实现才能使用它。

我知道我这里一定有某种设计问题。谁能指点一下？

Answer 1

尝试将消息与行为分开。创建一个包含操作数据的类，并创建一个包含该操作的业务逻辑的不同类。例如，创建这个命令：

public class RegisterAssetCommand
{
    [Required]
    public int CaseNumber { get; set; }

    [Required]
    public User Operator { get; set; }
}

现在定义一个处理业务命令的接口：

public interface ICommandHandler<TCommand>
{
    void Handle(TCommand command);
}

您的演示文稿代码现在将如下所示：

var command = new RegisterAssetCommand
{
    CaseNumber = 1000,
    Operator = GetUserFromPage(),
};

var commandHandler = WindsorContainer
    .Resolve<ICommandHandler<RegisterAssetCommand>);

commandHandler.Handle(command);

注意： 如果可能，将获取commandHandler 的职责移出表示类并将其注入该类的构造函数（再次构造函数注入）。

不，您可以像这样创建ICommandHandler<RegisterAssetCommand> 的实现：

public class RegisterAssetCommandHandler
    : ICommandHandler<RegisterAssetCommand>
{
    private ILawbaseAssetRepository lawbaseAssetRepository;
    private IAssetChecklistKctcPartRepository assetRepository;

    public RegisterAssetCommandHandler(
        ILawbaseAssetRepository lawbaseAssetRepository,
        IAssetChecklistKctcPartRepository assetRepository)
    {
        this.lawbaseAssetRepository = lawbaseAssetRepository;
        this.assetRepository = assetRepository;
    }

    public void Handle(RegisterAssetCommand command)
    {
        // Optionally validate the command

        // Execute the command
    }
}

或者，您甚至可以通过在RegisterAssetCommandHandler 中注入IUserProvider 将User 排除在RegisterAssetCommand 之外。 IUserProvider 接口可以有一个处理程序可以调用的GetUserForCurrentContext。

我希望这是有道理的。

Answer 2

正如 Morten 指出的，将不可注入的依赖项从构造函数调用移至实际需要使用它的方法，

如果您有无法（或难以）注入的构造函数参数，您将无法自动将 IAssetRegister 注入到任何需要它的类中。

当然，您总是可以创建一个 IUserProvider 接口，并按照以下思路进行具体实现：

public class UserProvider : IUserProvider 
{
    // interface method
    public User GetUser() 
    {
        // you obviously don't want a page dependency here but ok...
        return GetUserFromPage();
    }
}

因此创建了另一个没有的可注入依赖项。现在，您无需将用户传递给可能需要它的每个方法。