????????NullPointerException是Java开发中最常见的异常,它产生的原因很简单,即:当我们尝试访问一个null对象的方法或属性时就会抛出空指针异常,一个对象可能来自于用户输入的值,其它方法或接口的返回值、数据库查询的数据等等,这些不确定的对象中很可能就藏着一个null。
? ? ? ? 对付NullPointerException最简单的方法便是捕获异常,但是下面这段看似简单平常的异常处理代码却实在地造成了生产事故:
public class Demo extends Thread{
@Override
public void run() {
BatchJobLog jobLog = new BatchJobLog();//任务日志
while (true){
try{
System.out.println("业务逻辑处理...");
}catch (Exception e){
e.printStackTrace();
String message = e.getMessage();
jobLog.setStatus("E");
jobLog.setErrorMsg(message.length()>100?message.substring(0,100):message)
}finally {
insertBatchJobLog(jobLog);//将异常插入日志表
}
}
}
????????这段代码大致逻辑是:用一个线程处理某种业务,处理失败就将错误消息记录在错误日志表,接着继续处理业务。问题就出在这句message.length()>100,空指针异常的e.getMessage()返回值也是null,访问它的length()方法时再次抛出NullPointerException,最坑的是接下来就是finally,因为不管jobLog.setErrorMsg(message.length()>100?message.substring(0,100):message)这句有没有抛出异常,接下来执行的都是finally块,不同的是没异常finally块执行完回到while,有异常finally块执行完线程死掉,这让人很难发现这里抛出了异常,给人一种执行完finally块线程就莫名其妙死掉的错觉。
????????我在开发中不止一次看到这种代码,也许开发者的本意是防止错误消息太长导致数据库插入失败,也许这个异常处理本就不是给NullPointerException准备的,但NullPointerException就是无处不在。这次事故至少造成了五位数的损失,还好业务规模不是很大并且及时止损,在此记录和分享这次案例,同时也告诫自己,再简单的代码也不能马虎。