不要在一个给UIData组件提供数据的函数中访问数据库！

不要在一个给UIData组件提供数据的函数中访问数据库！例如<h:dataTable>，我经常看到有人这样做。这是个坏建议，只会带来坏的实施。 不要这样做!

大多数人都没有意识到(或许他们根本就没有关注他们使用的ORM工具的 SQL 输出日志)在 JSF 生命周期中， 这些表达式不仅仅被计算一次-- 通常多于一次。当每次值表达式提供给UIData组件值的时候，都会访问下数据库。在这之上，根据你获取的内容和如何实现的方式可能会改变结果集。On top of that, the result set could change depending on what you are retrieving and how you are doing it.

我将给出一个简单的示例，然后讲解下如何在 Seam中解决这个问题。

假设，你使用下面的类来取得一个User实体的数据，UserService 示例用来代理查询数据库的操作。

package example;

public class UserListBean {
    private UserService userService;

    public List getUsers() {
        return userService.findAll();
    }
}

然后在JSF配置文件中配置该类为managed bean。UserService通过一个EL值表达式注入(或许使用一个 EL
resolver 从Spring 容器中得到一个示例). 顺便提一下，设置该bean的作用域为session是另外一个坏的实践
这里我不讨论这个问题。如果你打算使用Session作用域，建议你去看看Seam的会话作用域。

<manage-bean>
  <managed-bean-name>userList</managed-bean-name>
  <managed-bean-class>example.UserListBean</managed-bean-class>
  <managed-bean-scope>request</managed-bean-scope>
  <managed-property>
    <property-name>userService</property-name>
    <property-value>#{userService}</property-value>
  </managed-property>
</manage-bean>

使用UserListBean类的 getUsers()方法给一个UIData组件提供数据是一个坏的实践，如下示例：

<h:dataTable id="users" var="user" value="#{userList.users}">
  <h:column>
    <f:facet name="header">Name</f:facet>
    #{user.name}
  </h:column>
</h:dataTable>

在这种情况下,当渲染页面的时候，SQL日志记录会显示多个查询记录，假设在该页面中还有个UICommand组件，在JSF postback节段会有其他的查询记录，

正确的做法是，绑定该list到一个上下文变量中，可以使用Seam的工厂组件，这样不管这个变量没查询了多少次，这个用户的list就会只在每个页面执行一次。注意下面我们使用了Seam的组件来代替上面的类。使用page 作用域就避免了在JSF postback节段重新查询数据。

package example;

@Name("userList")
public class UserListBean {
    @In
    private UserService userService;

    @Factory(value = "users", scope = ScopeType.PAGE)
    public List getUsers() {
        return userService.findAll();
    }
}

现在，我们就可以在JSF view中使用这个users 上下文变量了。

<h:dataTable id="users" var="user" value="#{users}">
  <h:column>
    <f:facet name="header">Name</f:facet>
    #{user.name}
  </h:column>
</h:dataTable>

请不要再一次进入前面提及的陷阱!