前两天在回帖写一个多线程的程序的时候犯错误了,今天又犯了一个多线程方面的错误。记录在此引以为戒。
网站用了 Struts2。当用户下载时,要记录下载日志。因为用户下载是付了钱的,因此决不能因为记日志失败就中断下载。所以将计日志的代码放在 new Thread() 里。
这时候我忘了很重要的一点:新的线程里面是无法取得 ActionContext 的。于是很不幸的,新线程中那句 ActionContext.getContext().getSession() 抛出了 NullPointerException。
更不幸的是,这是个 RuntimeException,所以没有被 catch,直接抛出去了。虽然并未影响下载,但是后面的记日志就没有执行了。所以结果是:库表里一条下载日志记录都没有。
更更不幸的是,我只关注 log4j 的输出,而上面那个空指针异常因为没有 catch,所以 log4j 输出里没有。最后这个问题直到检查库表的时候才发现。
由此得到的教训是:新建线程的 run() 方法中一定要把所有的异常都 catch 起来。