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;@Overridepublic void onApplicationEvent(ContextRefreshedEvent evt) {if (evt.getApplicationContext().getParent() == null) {new Thread(new Runnable() {@Overridepublic 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或

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

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

解决办法：

停止start方法的处理，所以这个实例一直存在，不会被销毁，除非server重启。

import org.springframework.stereotype.Component;
import javax.annotation.PostConstruct;
import javax.annotation.PreDestroy;
import java.util.Date;@Component
public class App {boolean isRun = false;@PostConstructpublic 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;}@PreDestroypublic 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> {@AutowiredApp app;@Overridepublic void onApplicationEvent(ContextRefreshedEvent evt) {if (evt.getApplicationContext().getParent() == null) {new Thread(new Runnable() {@Overridepublic void run() {app.start();}}).start();}}

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

个人建议：

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

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