逻辑应该在哪里来决定执行哪个 SQL 查询

Question

我有一个 DAO，其方法 CommitmentListDAO.getListByOwnerBadge 返回针对主管徽章的承诺项目数组列表（数据库字段 OWNED_BY）

    String SQL_VIEW_LIST_BY_SUPERVISOR = SELECT_QUERY + 
    " WHERE c.OWNED_BY = ? " +
    " ORDER BY p.PROGRAM_NAME";

现在，我想添加一个下拉菜单我的网络表单允许用户在 Owned By 或 Tasked To 之间进行选择 我需要在 DAO 中添加一个 WHERE c.TASKED_TO = ? 子句。

我是否执行在 DAO 中搜索哪个字段的逻辑 - 比如说传入的下拉参数（不是请求对象）并将方法重命名为 getListByBadge(String WhichField, String bad) 或我的 CommitmentListForm 类应该具有此逻辑，然后对 getListByOwnerBadge 或 getListByTaskeToBadge 进行适当的调用

Answer 1

我会在 DAO 上使用两种不同的方法来清楚地区分调用的作用。

DAO 的要点是隐藏 SQL 实现细节。您应该始终从“如果我切换到不同的持久性机制（例如 HBase）怎么办？”的角度来考虑这样的问题。 HBase 实现可能不会以简单通过字段名称区分的方式存储它。 DAO 应该能够隐藏该细节，从而隐藏不同的方法。

当然，这只是我的意见。 :)

Answer 2

我将确定控制器中的逻辑，并将两个 SQL 查询分成 getListByOwnerBadge 或 getListByTaskeToBadge。这样你就可以避免使用一种“包揽一切”的方法，并且很快就会失控。此外，选择使用“全部执行”方法的其他开发人员必须检查该方法的内部结构，以查看可以传入哪些有效字符串，而显式方法调用则可以清楚地了解该方法正在完成的任务。

Answer 3

我认为第二种解决方案更好。使 DAO 尽可能简单。没有逻辑，没有标志。创建 2 个简单的方法并以形式决定调用哪一个。或者甚至在表单和 DAO 之间创建另一个层来决定调用哪个 DAO 方法。

Answer 4

总结一下：

来自 @McStretch - 调用的逻辑位于您的控制器中。

来自 @rfreak - 查询方法本身位于 DAO 中

这是一个示例：

//Controller
CommitmentListAction {

updateForm(...){ 
  List<CommitmentItem> commitmentItems;
  if (formUsesOwnedBy){
    commitmentItems = CommitmentItemDAO.getListByOwnerBadge(...);
  } else {
    commitmentItems = CommitmentItemDAO.getListByTaskeToBadge(...);
  }
  // Do stuff with commitmentItems.
}
// DAO

getListByOwnerBadge(...){
 String SQL_VIEW_LIST_BY_SUPERVISOR = SELECT_QUERY + 
    " WHERE c.OWNED_BY = ? " +
    " ORDER BY p.PROGRAM_NAME"
  return doQuery(SQL_VIEW_LIST_BY_SUPERVISOR); // Performs the actual query
}

getListByTaskeToBadge(...){
String SQL_VIEW_LIST_BY_TASKED_TO = SELECT_QUERY + 
    " WHERE c.TASKED_TO = ? " +
    " ORDER BY p.PROGRAM_NAME"
  return doQuery(SQL_VIEW_LIST_BY_TASKED_TO); // Performs the actual query
}

如果您要拥有许多不同的 CommitmetItems 视图或许多不同的条件，请考虑将条件传递给 DAO。但只有当出现过多的 getListBy[blah] 方法污染 DAO 时才执行此操作。