领域驱动设计批评

Question

我想对下面的领域驱动设计提出一些建议和批评。我在下面包含了伪代码。真正的代码将具有封装的属性。

担忧

我唯一担心的是它似乎贫血。

步骤

1. 创建一个带有值的新请求
2. 请求是否已获得批准？
   1. 如果是，请显示值
   2. 如果否，请说明未获批准的原因

类

enum UnitedStatesState {
  ALABAMA,
  //...
  CALIFORNIA,
  //...
  MAINE,
  //...
  WASHINGTON
}

class License {
  int id;
  String name;

  //enum of state that the license is applicable in
  UnitedStatesState state; 
}

class LicenseRequest {
  //the name of the person making the request
  String name; 

  //enum of state to which the user is requesting a license in
  UnitedStatesState state; 

  LicenseResponse submit()
  {
     //TODO: move creation of the rules out of this class
     RuleGroup<LicenseRequest> ruleGroup = new RuleGroup<>();
     ruleGroup.add(new StateExclusionLicenseRequestRule(UnitedStatesState.MAINE));

     boolean approved = ruleGroup.execute(this);
     if(approved) {
       License license = createLiscense(request);
       return new ApprovedLicenseResponse(license);
     } else {
       DeniedLicenseResponse response = new DeniedLicenseResponse();
       response.rules = newArrayList(ruleGroup);
       return response;
     }
  }

  //TODO: move create license out of Request. maybe a factory class?
  private License createLicense()
  {
     License license = LicenseIdGenerator.generate(this.state);
     license.name = this.name;
     license.state = this.state;
     save(license);
     return license;
  }
}

//visitor for the rule
interface Rule<T> {
  public boolean execute(T o);
  public List<String> getMessages();
}

//rule that auto denies when the request is made in an excluded state
class StateExclusionLicenseRequestRule : Rule<LicenseRequest> {
  public List<String> getMessages();
  UnitedStatesState excludedState;
  public boolean execute(LicenseRequest request) {
     if(request.state == excludedState)
     {
       messages.add("No license for " + request.state + " is available at this time.");
       return false;
     }
     return true;
  }
}

//rule that groups all other rules
class RuleGroup<T> : Rule<T> {
  public void addRule(Rule<T> rule);
  public List<Rule<T>> getFailedRules();

  public List<String> getMessages() {
     List<String> messages = new ArrayList<>();
     for(Rule<T> rule : rules) {
       messages.addAll(rule.getMessages());
     }
     return messages;
  }

  public boolean execute(T o) {
     List<Rule<T>> failedRules = new ArrayList<>(rules.size());
     for(Rule<T> rule : rules) {
       boolean approve = rule.execute(o);
       if(!approve) {
         failedRules.add(rule);
       }
     }
     return !failedRules.isEmpty();
  }
}

interface LicenseResponse {
  boolean approved;
}

class ApprovedLicenseResponse : LicenseResponse {
  License license;
}
class DeniedLicenseResponse : LicenseResponse {
  private List<Rule<LicenseRequest>> rules;

  public List<String> getMessages()
  {
     List<String> messages = new ArrayList<>();
     for(Rule<LicenseRequest> rule : rules) {
       messages.addAll(rule.getMessages());
     }
     return messages;
  }
}

示例代码

request = new Request(name: 'Test', state: UnitedStatesState.CALIFORNIA)
response = request.submit()
if(response.approved)
{
  out('Your request is approved');
  out('license id = ' + reponse.id);
}
else
{
  out('Your request was denied');
  for(String message : response.messages)
  {
    out(message);
  }
}

更新 1：背景

这只是我想要实现的模拟。这是一个简单的系统，用户在表格中输入有关自己的信息，然后他们会被批准或拒绝许可。批准后，可打印证书。

举例来说，唯一的规则是拒绝缅因州的许可证请求。

更新 2：重构规则并删除处理程序

我对上面的示例进行了一些修改。删除了处理程序并将所有代码移至 LicenseRequest。我还将批准/拒绝规则移至实现访问者模式的类。

Answer 1

不幸的是，一些更相关的代码没有显示，但我会研究哪些代码可以被推送到 LicenseRequest 中。特别是，LicenseRequest 可能会创建 License 而不是处理程序（可能通过为其提供 ID）。如果 LicenseRequest 的属性仅用于创建批准的许可证，则尤其如此。那么这些就不必暴露在吸气剂中。

我还会让确定批准（可能使用其他名称）直接创建响应，而不是传递可写消息列表（仅在失败时使用）。

您应该寻找的气味是Feature Envy。特别是，应检查使用 License 或 LicenseRequest 中的数据进行的任何计算，以确定该计算是否应在这些类中完成。

数据对象（特别是不可变数据对象）有其用途，但您的担心是正确的。

Answer 2

为了可扩展性，状态应该是 ValueObjects。例如，当您需要将州的缩写与其名称等同时会发生什么？

看起来你的设计也没有真正的聚合根源。如果没有许可证可言，许可证请求就没有意义吗？在这种情况下，它应该通过许可证（或许可证服务）处理，而不是直接由调用代码（可能是 Web 服务器、控制台应用程序等）处理。

那么对于伪代码来说，类似的东西会

var license = licenseFactory.NewLicense();
response = license.Request(name, state)

更有意义吗？当您拥有多种类型的驾照（例如商业司机驾照）时会发生什么？

此外，对于规则，而不是命令模式（执行），您可以通过将规则作为规范来获得更清晰的解决方案 - https://en.wikipedia.org/wiki/Specification_pattern