Проблема архитектуры: использование внедрения зависимостей приводит к мусорному API

Как указывает Мортен, переместите неинъекционные зависимости из вызова конструктора в метод (ы), которые действительно должны его использовать,

Если у вас есть параметры конструктора, которые нельзя (или сложно) внедрить, вы также не сможете автоматически внедрить IAssetRegister в любой класс, который в этом нуждается.

Конечно, вы всегда можете создать IUserProvider интерфейс с конкретной реализацией в следующих направлениях:

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

Таким образом создается еще одна инъекционная зависимость там, где ее не было. Теперь вы избавляетесь от необходимости передавать пользователя каждому методу, который может в нем нуждаться.

Попробуйте отделить сообщение от поведения. Создайте класс, содержащий данные для операции, и создайте другой класс, содержащий бизнес-логику для этой операции. Например, создайте эту команду:

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
    }
}

При желании вы могли бы даже оставить User вне RegisterAssetCommand, введя IUserProvider в RegisterAssetCommandHandler. Интерфейс IUserProvider может иметь GetUserForCurrentContext, который может вызвать обработчик.

Я надеюсь это имеет смысл.