java: web应用中不经意的内存泄露
前面有一篇讲解如何在spring mvc web应用中一启动就执行某些逻辑，今天无意发现如果使用不当，很容易引起内存泄露，测试代码如下：
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管理控制台上，一直处于部署中的状态，也没有任何输出，让人一头雾水，折腾半天才能定位错误，很浪费时间，如果是线上生产环境，是要粗事情的。