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

我正在尝试创建一个类，它执行各种与数据库相关的低级操作，但为 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 需要一个整数和一个 User。他们需要了解该类的实现才能使用它。

我知道我这里一定有某种设计问题。有人能给我一些指点吗？

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

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);

Note: 如果可能的话，转移获得许可的责任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
    }
}

或者，您甚至可以离开User出于RegisterAssetCommand通过注入IUserProvider in the RegisterAssetCommandHandler. The IUserProvider接口可以有一个GetUserForCurrentContext处理程序可以调用。

我希望这是有道理的。