当需求以这种方式发展时，如何划分责任？

Question

首先，我的要求是

“我们可以创建一个帐户并存入资金，当我们购买商品时，我们会减少帐户”

所以我的 AccountController 看起来像

class AccountController
{
    private IAccountDataSource _accountDataSource;

    Create(Account anAccount)
    {
        _accountDataSource.Insert(anAccount);
         Render(anAccount.Id);
     }
}

但是有一个新的要求 “有些人可以拥有一个免费帐户（所有项目都是免费的），但如果我们创建一个真实帐户，那么我们会删除免费帐户”

所以我的控制器。创建变成了

Create(Account anAccount)
{
    _accountDataSource.Insert(anAccount);
    RemoveFreeAccount(anAccount.Customer);
    Render(anAccount.Id);
}

RemoveFreeAccount(Customer aCustomer)
{
    _accountDataSource.Remove(new AccountFilter() { Type='Free', CustomerId=aCustomer.Id });
}

但对我来说，感觉我应该把这个 在其他地方删除FreeAccount，但我不知道在哪里，因为IAccountDataSource 只是假设处理数据存储。

Answer 1

该问题表明您违反了 SRP。您的控制器不应包含业务逻辑。直接在控制器中使用存储库会迫使您将所有逻辑放入其中，从而承担两个职责（作为 MVC 中的 M + 处理业务逻辑之间的桥梁）。

第一个重构部分应该是将业务逻辑移至模型中（在 MVC 中，不要与实体模型或视图模型混淆），

这将为您的原始代码提供以下结构：

public class AccountService
{
    void CreateAccount(string accountName)
    {
       var account = new Account(accountName);
        _dataSource.Create(account);
        DomainEvents.Publish(new AccountCreated(account));
    }
}

public class AccountController
{
    private AccountService _service;

    Create(AccountViewModel model)
    {
        var account = _accountDataSource.Create(model.Name);
        Render(account.Id);
     }
}

可能看起来很小，但很重要：

1. 更改 控制器现在只有一个改变的理由（视图和模型之间的映射）
2. 业务需求的任何改变都不会强制UI层发生改变。
3. 要求的更改仅在一个地方进行

要添加对免费帐户的支持，我将使用事件驱动模型：

public class FreeAccountService : ISubscriberOf<UserCreated>, ISubscriberOf<AccountCreated>
{
    public FreeAccountService(AccountService)
    {
    }

    public void HandleEvent(UserCreated domainEvent)
    {
        accountService.Create(new FreeAccount());
    }

    public void HandleEvent(AccountCreated domainEvent)
    {
        var freeAccount = dbSource.GetFreeAccount();
        if (freeAccount != null)
            accountService.Delete(freeAccount)
    }
}

因为它不需要更改其他帐户服务。