获取数据库条目的正确代码模式是什么？

Question

在另外两个问题中（此处和此处）BalusC 做出了直接声明：

获取器仅用于访问 bean 属性，而不是执行某些业务逻辑。那里有 bean 构造函数、初始化块或事件方法。所有这些在 bean 的生命周期中只执行一次，而这正是您想要的。

哎呀——这让我已经编写的无数行代码失效了。好吧，那么，实现填充数据表的支持 bean 的正确方法是什么？我理解他的观点和概念，但不理解实践。我的问题是双重的：

1. 为什么我做事的方式是错误的？
2. 我该如何修复它？

我经常使用 PrimeFaces p:dataTable，它的 value 属性解析为一个集合。由于我不在这里讨论的原因，我不使用PrimeFaces的延迟表加载功能。相反，我实现了自己的过滤/排序控件，但它们会触发 AJAX 事件，从而导致表中填充从数据库获取的记录。

该表的标记如下：

 <p:panel id="mqTable">

   <h:outputText value="Sort/Filter: #{maintCategory.tableQueryParameters}" />

   <p:dataTable
      id="mqDataTable"
      rows="#{maintCategory.pageSize}"
      value="#{maintCategory.dataModel}"
      selection="#{maintCategory.selected}"
      var="cat"
      selectionMode="single"
      emptyMessage="No Categories Found">

现在 dataModel 的极其糟糕的 UN-JSFish（或者我刚刚发现的） getter 如下所示：

public ATMDataModel getDataModel() {
    TableQueryParameters p = getTableQueryParameters();
    if (p.isChangePending()) clearDataModel();
    p.setChangePending(false);
    if (dataModel != null) return dataModel;
    List<ET> list = getDAO().runQuery(p);
    if (p.isNeedResultSize()) p.setResultSize(getDAO().runQueryCount(p));
    dataModel = new ATMDataModel(list);
    return dataModel;
}

一些解释。

1. 这是来自一个抽象超类，其中 ET 是“实体类型”。我所有的 CRUD 都使用相同的例程。
2. ATMDataModel 类是实现 SelectableListModel 的列表的包装器。 PrimeFaces 中的行选择逻辑需要这样做。 （这是 PF 3 中出现的一个问题，但它使行选择工作更加可靠。）
3. TableQueryParameters 类是我编写的，用于封装用户屏幕上表的当前状态。它包括什么排序参数、什么过滤器参数、我们在哪个页面等。因为需要保留这些，所以支持 bean 是 ViewAccesScoped （通过 MyFaces CODI）和 TableQueryParameters strong> 是其中的一个属性。
4. TableQueryParameters 通过 AJAX 事件进行响应更新，这也会更新表单，从而导致调用 getDataModel。当发生任何变化时，isChangePending 方法将变为 true。因此，getDataModel 方法使用此方法在更改之间仅生成一次从 DAO 获取的数据，无论调用多少次。

但是如果TableQueryParameters确实发生变化，我必须使用这些参数调用runQuery来获取新的记录集用户想看。如果我不在 getDataModel 中调用它，我应该在哪里调用它？

请指教。

Answer 1

您基本上是在 getter 中延迟加载数据。您不会在同一请求（或视图）范围内的每个 getter 调用中访问数据库。这是实惠的。我不使用 CODI，但我想 getTableQueryParameters() 调用也特别便宜，无需担心。

至于具体问题，您通常会在附加到 UICommand 组件和/或 ajax 事件标记的操作（侦听器）方法中执行数据库/业务工作。

例如（也可以用作 ）

<p:ajax listener="#{bean.deleteSelectedRow}" />
...
<p:ajax listener="#{bean.saveRowDetail}" />

根据

public void deleteSelectedRow() {
    someService.delete(selectedRow);
    dataModel = loadDataModel();
}

public void saveRowDetail() {
    someService.save(selectedRow);
    dataModel = loadDataModel();
}

p.isChangePending() 的含义，我认为您也可以摆脱它这样，看起来您是在操作（侦听器）方法中设置它。