今天无意发现如果使用不当,很容易引起内存泄露,测试代码如下:

1、定义一个类App

package com.cnblogs.yjmyzz.web.controller;

import java.util.Date;

public class App {

boolean isRun = false;

public App() {

isRun = true;

}

public void start() {

while (isRun) {

System.out.println("=======> I AM ALIVE =>" + new Date());

try {

Thread.sleep(1000);

} catch (InterruptedException e) {

e.printStackTrace();

}

}

}

public void stop() {

isRun = false;

}

}

代码里面的内容不是重点,只是示意一下,我打算在spring mvc 应用一启动时,就让这个类实例化,执行其中的start方法,即:每隔一秒输出一句话。

2、定义一个Listener


import com.cnblogs.yjmyzz.web.controller.App;

import org.springframework.context.ApplicationListener;

import org.springframework.context.event.ContextRefreshedEvent;

import org.springframework.stereotype.Component;

@Component

public class StartupListener implements ApplicationListener<ContextRefreshedEvent> {

App app;

@Override

public void onApplicationEvent(ContextRefreshedEvent evt) {

if (evt.getApplicationContext().getParent() == null) {

new Thread(new Runnable() {

@Override

public void run() {

app = new App();

app.start();

}

}).start();

}

}

}

代码也很简单,应用一启动,就开一个线程,实例化App,然后调用app.start()方法,运行一下,也跟预期的一样,每隔一秒输出类似下面的内容:

=======> I AM ALIVE =>Wed Sep 16 21:55:42 CST 2015

正式部署到jboss上以后,问题来了,在jboss管理控制台上,把这个应用给disable甚至remove后,日志里仍然不断有上面的类似输出,即app的实例仍然活着,其start方法也始终在运行,换句话说,app并没有被销毁。

简单分析一下:jboss的每个server启动后,会伴随启动一个jvm实例,而部署在该server上的web应用,里面创建的各种资源也在这个jvm实例中,就算把应用给停掉甚至删除,由于代码中没有任何清除app或停止start方法的处理,所以这个实例一直存在,不会被销毁,除非server重启。


另一个问题:如果把上面这段代码中,创建线程的部分去掉,改成直接 app = new App(); app.start(); 部署时会发现另一个现象,日志里仍然不断有输出,即代码在执行,但是该应用在jboss中的状态始终是isdeploying,部署一直无法结束,始终处于『部署中』的状态。

原因:start方法中的Thread.sleep()方法会阻塞线程,导致部署无法执行完毕。

解决办法:

import org.springframework.stereotype.Component;

import javax.annotation.PostConstruct;

import javax.annotation.PreDestroy;

import java.util.Date;

@Component

public class App {

boolean isRun = false;

@PostConstruct

public void init() {

System.out.println("init ==> " + new Date());

isRun = true;

}

public void start() {

while (isRun) {

System.out.println("=======> I AM ALIVE =>" + new Date());

try {

Thread.sleep(1000);

} catch (InterruptedException e) {

e.printStackTrace();

}

}

}

public void stop() {

isRun = false;

}

@PreDestroy

public void destroy() {

System.out.println("destroy ==> " + new Date());

stop();

}

}

这里做了几处改进:

a) 加上@Component后,App的实例将由Spring容器自动创建,即由容器来管理

b) 加上了@PreDestroy,Bean的生命周期由Spring容器来管理后,凡是Bean里加上该注解的方法,会在Bean销毁前被执行,通常该方法用于清理资源

c) 将初始化的工作,移到了init方法中,并通过@PostConstruct注解告诉Spring,在调用完Bean的默认构造方法后,自动来调用该方法(当然这一步是可选的,并非必须)

@Component

public class StartupListener implements ApplicationListener<ContextRefreshedEvent> {

@Autowired

App app;

@Override

public void onApplicationEvent(ContextRefreshedEvent evt) {

if (evt.getApplicationContext().getParent() == null) {

new Thread(new Runnable() {

@Override

public void run() {

app.start();

}

}).start();

}

}

}

Listener中就简单多了,直接@Autowired注入app实例就行了。

个人建议:

a) 如果要在web 应用一启动时,就执行某些操作,特别是对资源类的长连接实例创建(比如:加载数据到缓存中预热、连接到Zookeeper监控节点变化、连接到Ftp准备取数据),最好交给Spring容器来自动创建,且务必记得在Destroy前,清理资源(即:断开连接)

b) 在启动的执行逻辑中,不要使用阻塞线程的操作(比如:Thread.sleep之类的方法),否则部署时,实际上代码已经在后台执行了,jboss管理控制台上,一直处于部署中的状态,也没有任何输出,让人一头雾水,折腾半天才能定位错误,很浪费时间,如果是线上生产环境,是要粗事情的。