当需求以这种方式演变时，如何划分职责？

Question

首先我的要求是

“我们可以创建一个账户并在上面存钱，当我们购买物品时，我们会减少账户”

所以我的 AccountController 看起来像

class AccountController
{
    private IAccountDataSource _accountDataSource;

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

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

所以我的 controller.Create 变成了

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

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

但对我来说，我觉得我应该把这个 RemoveFreeAccount 放在其他地方，但我不知道在哪里，因为 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)
    }
}

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