IllegalStateException 当尝试从会话范围的 Bean 进行 .getSessionMap() 时

Question

我是Java和JSF的新手。我需要有关非法状态异常的帮助。场景如下：

在我当前的项目中，我为应用程序菜单提供了这个会话范围的 bean：

public final class MenuBean implements Serializable{  
    private MenuModel model;
    private FacesContext context = FacesContext.getCurrentInstance();
    public MenuModel getModel() {
        return model;
    }
    public MenuBean() {
        updateMenu();
    }
    public void updateMenu(){
        Map session = (Map<String,Object>) context.getExternalContext().getSessionMap();
        EUser user = (EUser) session.get(UserBean.USER_SESSION_KEY);
        ...
    }
    private MethodExpression createMethodExpression(String action) {  
        ...  
    }
}

在我的逻辑的某个时候，我需要更新菜单，所以我这样做：

ExternalContext extContext = context.getExternalContext();
Map sMap = (Map<String,Object>) extContext.getSessionMap();
MenuBean menu = (MenuBean) sMap.get("menuBean");
menu.updateMenu();

Bean 构造很好，但是当我尝试如上所示手动更新它时，我在更新方法的第一行得到 IllegalStateException updateMenu()

我不明白出了什么问题，因为我可以在第一次构建菜单时使用相同的调用来获取会话映射。

此外，使用 NetBeans 调试器，我可以看到 MenuBean 的实例已正确恢复。

你们能帮我吗？

Answer 1

FacesContext存储在 HTTP 请求线程中。您绝对不应该将其声明并分配为比HTTP请求寿命更长的实例的实例变量（最好也不要在它已经基于请求时 - 这是糟糕的设计）。FacesContext实例将在 HTTP 请求完成时释放并失效。在任何后续 HTTP 请求中，实例不再有效。有非法国家的手段。这解释了您所看到的IllegalStateException。

您需要删除以下行：

private FacesContext context = FacesContext.getCurrentInstance();

并修复您的代码以使其仅在方法块中本地线程：

Map<String, Object> sessionMap = FacesContext.getCurrentInstance().getExternalContext().getSessionMap();
// ...

您始终可以将其分配为变量，但这只能保持线程本地：

FacesContext context = FacesContext.getCurrentInstance();
Map<String, Object> sessionMap = context.getExternalContext().getSessionMap();
// ...

---

与具体问题无关，在这种特殊情况下使用@ManagedProperty更容易。

public final class MenuBean implements Serializable {  
    @ManagedProperty("#{user}")
    private EUser user;
    // ...
}

然后 JSF 将为您注入它。