下面一段《JSF第一步》书中的代码:
public void departmentChanged(ValueChangeEvent event) {
//取得faces上下文
FacesContext facesContext= FacesContext.getCurrentInstance();
try{
//获得选取的部门
Department selectedDep=new Department();
Integer departmentId=(Integer)event.getNewValue();
selectedDep.setDepartmentId(departmentId);
this.userList1=userService.depUsers(selectedDep);
//准备一个用户id和姓名的选择项目列表,
//为在诸如下拉列表之类的UI中显示它们做准备
this.userList=new ArrayList();
for (int i=0;i<userService.depUsers(selectedDep).size();i++){
Person person=(Person)userService
.depUsers(selectedDep).get(i);
this.userList.add(new SelectItem(person.getPersonId(),
person.getPersonName()));
}
//准备一个角色id和角色名称的选择项目列表,
//为在诸如下拉列表之类的UI中显示它们做准备
this.roleList=new ArrayList();
for (int i=0; i<userService.Roles().size(); i++) {
Role role=(Role)userService.Roles().get(i);
this.roleList.add(new SelectItem(role.getRoleId(),
role.getRoleName()));
}
}
catch(DbException de){
//捕获DbException异常
FacesMessage msg=MessageFactory.getMessage
(facesContext,"error_database_error","");
msg.setSeverity(FacesMessage.SEVERITY_WARN);
facesContext.addMessage(null,msg);
}
catch(Exception e){
//捕获其他异常
FacesMessage msg=MessageFactory.getMessage
(facesContext,"error_unexpected","");
msg.setSeverity(FacesMessage.SEVERITY_ERROR);
facesContext.addMessage(null,msg);
}
}
这段代码主要是//用于在分配用户角色时用户选择一个部门后,列出该部门的人员列表和角色列表
本来这段代码是没有什么问题的,但持久化方面用的是hibernate做的。我们都知道hibernate是延迟加载的,即要用到数据库的东西才最后加载。
this.userList=new ArrayList();
for (int i=0;i<userService.depUsers(selectedDep).size();i++){
Person person=(Person)userService
.depUsers(selectedDep).get(i);
this.userList.add(new SelectItem(person.getPersonId(),
person.getPersonName()));
}
在上面的代码中由于没有完全从数据库里取出某个部门的人员列表,但在for循环时才要用到一条条语句,这样就造成了循环一次就执行一次SQL查询语句。同样角色列表也是这个问题,但人员比较多和角色也有10多个时这样就要执行好多条查询语句,这样效率就大打折扣。