关于使用哪些服务以及按什么顺序的决定是否构成 MVC 控制器方法中的逻辑？

Question

我正在开发的新 ASP.NET MVC 应用程序即将结束，并且我意识到我并没有 100% 明白我的控制器方法中应该发生什么。

操作方法是否更好地决定调用哪些服务/方法以及按什么顺序调用：

AccountService _accountService;
BillingService _billingService;
InvoiceService _invoiceService;

...    

public ActionResult UpgradeAccountPackage(PackageType packageType, int accountId)
{
    _accountService.UpgradeAccountPackage(packageType, accountId);
    _billingService.BillForAccountUpgrade(packageType, accountId);
    _invoiceService.CreateAccountUpgradeInvoice(packageType, accountId);
}

或者是否最好坚持对一个服务进行单一方法调用并允许该方法调用它需要的其他服务/支持方法？

public ActionResult UpgradeAccountPackage(PackageType packageType, int accountId)
{
    // account service upgrades account then calls the BillingService and InvoicService
    // methods called above within this method
    _accountService.UpgradeAccountPackage(packageType, accountId);
}

我倾向于在这里使用第二个示例，因为最初似乎第一个方法会以某种方式构成逻辑，并且意味着 acion 方法必须本质上了解帐户升级过程在我的应用程序中如何工作，这看起来像一件坏事。

然而，现在我的应用程序几乎完成了，它有一个很大的服务层，这种方法导致几乎每个服务都强烈依赖于许多其他服务，并且没有集中的地方来决定业务交易的流程，例如上面提到的那个如上所述，您必须深入研究服务方法才能发现流程。

我正在考虑重构以更类似于上面的第二种方法，或者在控制器和服务层之间引入一个新层来控制流程。

人们倾向于使用第一种方法还是第二种方法？民众的意见是什么？

Answer 1

我更喜欢第二种方法 - 更容易测试（使用模拟），并且逻辑可供重用。您最终会得到实际业务逻辑的外观，但这并不是坏事。

我不明白为什么你的服务层充满了具体的依赖关系......

要记住的是你希望类依赖于接口，而不是实现。 （然后使用依赖注入工具将其全部串在一起）

在 C# 中，我们可以让一个类实现多个接口。因此，您的服务实现可以实现许多接口，而调用者只需要知道他们需要的部分。

例如，您可能有一个实现 IDepositor 和 IWithdrawer 的 AccountTransactionService。如果您实现复式记账，那么这可能取决于 IDepositor 和 IWithdrawer，实际上，它们仅使用相同的 AccountTransactionService 实例>，但不是必须的，并且实现细节可以在之后更改。

一般来说，一个类对系统中其他类的了解越少越好。

Answer 2

我更密切地使用第一种方法。让控制器控制发生的事情。让服务来决定如何发生。

如果您添加第二层来控制流程流，您的 ActionMethods 是否只会进行一次调用？如果是这样，那么此时看来就没有必要了。

Answer 3

您可以拥有一个依赖于多个存储库（而不是其他服务）并定义业务操作的服务层：

public class MyService: IMyService
{
    private readonly IAccountRepository _accountRepository;
    private readonly IInvoiceRepository _invoiceRepository;
    private readonly IBillingRepository _billingRepository;
    public MyService(IAccountRepository accountRepository, IInvoiceRepository invoiceRepository, IBillingRepository billingRepository)
    {
        _accountRepository = accountRepository;
        _invoiceRepository = invoiceRepository;
        _billingRepository = billingRepository;
    }

    public void UpgradeAccountPackage(PackageType packageType, int accountId)
    {
        ...
    }

}

然后让您的控制器将 IMyService 作为依赖项：

public class HomeController: Controller
{
    private readonly IMyService _service;
    public HomeController(IMyService service)
    {
        _service = service;
    }

    public ActionResult UpgradeAccountPackage(PackageType packageType, int accountId)
    {
        _service.UpgradeAccountPackage(packageType, accountId);
        ...
    }
}

我使用存储库和服务将这些多个操作聚合到一个业务事务中。