CDI 会话范围的 bean 未被销毁导致内存泄漏

Question

我对会话范围的 CDI bean 的生命周期有疑问。
据我了解，会话范围的 CDI bean 由容器在会话开始时构造，并在会话结束时销毁。在 bean 被销毁之前，@PreDestroy 方法被调用，如此处 https://docs.oracle.com/javaee/6/tutorial/doc/gmgkd.html 所述。还说在这个方法里释放资源

在我构建的 JSF 应用程序中，我遇到了内存泄漏，因为 bean 似乎没有被销毁，因此没有调用 @PreDestroy 方法来释放一些引用垃圾收集器。所以我构建了一个简单的应用程序来测试行为。我的经验是，会话结束时会话 bean 不会被销毁，而且当需要内存 space 时它甚至不会被销毁。我不敢相信我是第一个遇到这种情况的人，但我没有找到有关此行为的任何信息..

所以我的问题是：CDI bean 是否应该被销毁 - 因此 @PreDestroy 方法被调用 - 在其上下文过期后立即被调用？如果不应该至少在需要 space 时将其销毁？

我的测试申请：

不允许我post一张图，但是大纲是eclipse生成的非常基础的jsf webapp。我还有 beans.xml 文件。

Test.java:

package com.test;

import java.io.Serializable;
import java.util.ArrayList;

import javax.annotation.PostConstruct;
import javax.annotation.PreDestroy;
import javax.enterprise.context.SessionScoped;
import javax.inject.Named;

@SessionScoped
@Named
public class Test implements Serializable {

    /**
     * 
     */
    private static final long serialVersionUID = 1L;
    private String test;
    private ArrayList<ComplexType> cps;
    private ArrayList<ComplexType> cps_2;

    @PostConstruct
    public void init() {
        System.out.println("test postconstruct..");
        test = "Cdi Test";
    }

    @PreDestroy
    public void cleanUp() {
        cps = null;
        cps_2 = null;
        System.out.println("test cleanUp....");
    }

    public void data_1() {

        cps = new ArrayList<ComplexType>();

        for(int i = 0; i < 800; i++) {
            String[] s = new String[100000];
            ComplexType cp = new ComplexType(i, s);
            cps.add(cp);
            System.out.println(i);
        }
        System.out.println("data_1");
    }

    public void free_1() {
        cps = null;
        System.out.println("free_1");
    }

    public void data_2() {

        cps_2 = new ArrayList<ComplexType>();

        for(int i = 0; i < 800; i++) {
            String[] s = new String[100000];
            ComplexType cp = new ComplexType(i, s);
            cps_2.add(cp);
            System.out.println(i);
        }
        System.out.println("data_1");
    }

    public void free_2() {
        cps_2 = null;
        System.out.println("free_1");
    }

    public String getTest() {
        return test;
    }

    public void setTest(String test) {
        this.test = test;
    }   
}

ComplexType.java:

package com.test;

public class ComplexType {

    private int id;
    private String[] name;

    public ComplexType(int id, String[] name) {

        this.id = id;
        this.name = name;
    }
    public int getId() {
        return id;
    }
    public void setId(int id) {
        this.id = id;
    }
    public String[] getName() {
        return name;
    }
    public void setName(String[] name) {
        this.name = name;
    }
}

index.xhtml:

<html xmlns="http://www.w3.org/1999/xhtml"
xmlns:h="http://java.sun.com/jsf/html"
xmlns:f="http://java.sun.com/jsf/core"
>

<h:head>
    <title>Cdi test </title>
</h:head>

<h:body>

    <h:outputText value="#{test.test}"></h:outputText>

    <h:form>
        <h:commandButton value="cp_1 data" actionListener="#{test.data_1}">
            <f:ajax></f:ajax>
        </h:commandButton>
        <h:commandButton value="cp_1 Free" actionListener="#{test.free_1}">
            <f:ajax></f:ajax>
        </h:commandButton>

        <br></br>
        <h:commandButton value="cp_2 data" actionListener="#{test.data_2}">
            <f:ajax></f:ajax>
        </h:commandButton>
        <h:commandButton value="cp_2 Free" actionListener="#{test.free_2}">
            <f:ajax></f:ajax>
        </h:commandButton>
    </h:form>

</h:body>
</html>

我打开 index.xhtml 页面，@PostConstruct 方法按预期被调用。当我调用 data_1 和 data_2 时，堆 space 被超出，两者之间没有释放。当我释放其中一个资源或连续两次调用一个方法时，堆 space 就足够了，因为垃圾收集器会释放内存。这正如我所期望的那样有效。

但是当我调用一个数据函数时，关闭浏览器并因此关闭会话，打开一个新浏览器并再次调用其中一个数据函数，然后应用程序停止工作（我猜）超出内存space。重点是：第一个会话 bean 没有被销毁，它的 @PreDestroy 方法没有被调用，因此 ArrayList 仍在内存中。

有人可以向我解释一下这是怎么回事吗？ CDI bean 不应该在其上下文过期后立即被容器销毁，以便可以将引用设置为 null 并且垃圾收集器可以释放资源吗？
我正在使用 JBoss AS 7.1.1 及其默认实现 JSF Mojarra 2.1.

Answer 1

会话 bean（不管是 CDI 还是 JSF 管理）保持活动状态，直到超过某个会话超时（通常默认为 30 分钟，取决于应用程序服务器），您可以在 web.xml 中指定。只是关闭浏览器不会使会话无效，它会在超时到期后等待被 servlet 容器销毁。所以，我的假设是，这样的行为很好，稍后将调用@PreDestroy 方法。

Answer 2

@olexd的回答基本说明了我的错误，非常感谢！但是在确定的时间段后使会话无效不是一种选择，所以我也不得不使用@geert3 的评论，谢谢你！我正在回答我自己的问题，以展示我是如何在这里详细解决我的特定问题的。

我错了：我认为浏览器一关闭会话就会过期。这是错误的，也是有道理的。人们可能想要关闭浏览器并再次打开它以在与以前相同的会话中工作。
对我来说，这种行为是不合适的，因为我想在浏览器关闭后立即释放资源。所以答案是像这样手动使会话无效：

FacesContext.getCurrentInstance().getExternalContext().invalidateSession();

这个方法一被调用，@PreDestroy方法就被调用了，正合我意。现在我必须确定何时调用此函数。我搜索了一种方法来收听类似 browserclose 事件的内容。有 onbeforeunload 和 onunload 事件。 onunload 在 Chrome 中似乎对我不起作用，但 onbeforeunload 可以。另请参阅此答案：

所以我写了一个隐藏按钮，javascript 在 beforeunload 上点击它并调用适当的 backingbean 方法。这正如我所期望的那样有效。我在 Chrome 43.0.2357.65 和 IE 11 上测试过，目前我对它很满意。然而，它不适用于 onunload，但我现在不关心这个。

所以我的最终代码是这样的：

index.xhtml

<html xmlns="http://www.w3.org/1999/xhtml"
    xmlns:h="http://java.sun.com/jsf/html"
    xmlns:f="http://java.sun.com/jsf/core">

<h:head>
    <title>Cdi test</title>
    <h:outputScript library="default" name="js/jquery-1.11.3.min.js"
        target="head"></h:outputScript>
</h:head>

<h:body>

    <h:outputText value="#{test.test}"></h:outputText>

    <h:form id="overall">
        <h:commandButton value="cp_1 data" actionListener="#{test.data_1}">
            <f:ajax></f:ajax>
        </h:commandButton>
        <h:commandButton value="cp_1 Free" actionListener="#{test.free_1}">
            <f:ajax></f:ajax>
        </h:commandButton>

        <br></br>
        <h:commandButton value="cp_2 data" actionListener="#{test.data_2}">
            <f:ajax></f:ajax>
        </h:commandButton>
        <h:commandButton value="cp_2 Free" actionListener="#{test.free_2}">
            <f:ajax></f:ajax>
        </h:commandButton>

        <br></br>

        <h:commandButton id="b" style="display:none"
            actionListener="#{test.invalidate}"></h:commandButton>

    </h:form>

    <script type="text/javascript">
        $(window).on('beforeunload', function() {
            $('#overall\:b').click();
        });
    </script>
</h:body>
</html>

Test.java

package com.test;

import java.io.Serializable;
import java.util.ArrayList;

import javax.annotation.PostConstruct;
import javax.annotation.PreDestroy;
import javax.enterprise.context.SessionScoped;
import javax.faces.context.FacesContext;
import javax.inject.Named;

@SessionScoped
@Named
public class Test implements Serializable {

    /**
     * 
     */
    private static final long serialVersionUID = 1L;
    private String test;
    private ArrayList<ComplexType> cps;
    private ArrayList<ComplexType> cps_2;

    @PostConstruct
    public void init() {
        System.out.println("test postconstruct..");
        test = "Cdi Test";
    }

    @PreDestroy
    public void cleanUp() {
        cps = null;
        cps_2 = null;
        System.out.println("test cleanUp....");
    }

    public void data_1() {

        cps = new ArrayList<ComplexType>();

        for (int i = 0; i < 800; i++) {
            String[] s = new String[100000];
            ComplexType cp = new ComplexType(i, s);
            cps.add(cp);
            System.out.println(i);
        }
        System.out.println("data_1");
    }

    public void free_1() {
        cps = null;
        System.out.println("free_1");
    }

    public void data_2() {

        cps_2 = new ArrayList<ComplexType>();

        for (int i = 0; i < 800; i++) {
            String[] s = new String[100000];
            ComplexType cp = new ComplexType(i, s);
            cps_2.add(cp);
            System.out.println(i);
        }
        System.out.println("data_2");
    }

    public void free_2() {
        cps_2 = null;
        System.out.println("free_2");
    }

    public void invalidate() {
        FacesContext.getCurrentInstance().getExternalContext().invalidateSession();
        System.out.println("invalidate");
    }

    public String getTest() {
        return test;
    }

    public void setTest(String test) {
        this.test = test;
    }

}

注意我用的是JQuery。这适用于 JBoss AS 7.1.1 和默认的 Weld 实现。
要添加的一件事：不必手动将所有引用设置为 null。这也有道理，因为它会很乏味..

CDI 会话范围的 bean 未被销毁导致内存泄漏

CDI session scoped bean not destroyed results in memory leakage

jsf

session-scope

destroy

cdi

weld