今天犯了一个大错误

前两天在回帖写一个多线程的程序的时候犯错误了，今天又犯了一个多线程方面的错误。记录在此引以为戒。

 

网站用了 Struts2。当用户下载时，要记录下载日志。因为用户下载是付了钱的，因此决不能因为记日志失败就中断下载。所以将计日志的代码放在 new Thread() 里。

 

这时候我忘了很重要的一点：新的线程里面是无法取得 ActionContext 的。于是很不幸的，新线程中那句 ActionContext.getContext().getSession() 抛出了 NullPointerException。

 

更不幸的是，这是个 RuntimeException，所以没有被 catch，直接抛出去了。虽然并未影响下载，但是后面的记日志就没有执行了。所以结果是：库表里一条下载日志记录都没有。

 

更更不幸的是，我只关注 log4j 的输出，而上面那个空指针异常因为没有 catch，所以 log4j 输出里没有。最后这个问题直到检查库表的时候才发现。

 

由此得到的教训是：新建线程的 run() 方法中一定要把所有的异常都 catch 起来。