可恶的NullPointerException与JVM的失职

由于程序的不严密而出现NullPointerException异常的代码太常见了，
虽然可以在使用对象前检查是否为空，但过多的这种检查会使代码臃肿。
出现这样的异常的时候常常没有明确的信息，给调试带来不便，
即使知道抛出异常的代码行号，但或许因为你手头没有代码，或许因为这行代码有多次方法调用，
而很难得到错误的原因或是判断究竟什么对象是空。

我觉得这是JVM的失职，它只是抛出了没有任何信息的NullPointerException，
为什么不追加“调用XXX方法时，发现对象不可用”的信息呢

评论

64 楼 long_biti 2008-11-27  

对于obj.a().b().c()的情况，可以分成以下两类来看
1.方法返回的是this,比如common-tuils包下的
new EqualsBuilder()
    .append(a.p1,a.p2)
    .append(a.s1,a.s2)
    .isEqual();
这只是一种风格，个人觉得没什么问题。

2.不同的方法返回的是不同的对象，比如
public void deleteRegion(){
    this.getDepartment().getRegion().delete();
}
这种情况理论上是不建议使用的。因为从OO的角度来看，该方法涉及到了太多的东西造成一定的耦合性。

对于空指针的异常，一般系统会有一些基于约定的编程规范，比如所有返回集合的方法不允许直接抛null而应该返回一个空的集合对象。这样可以避免调用者做这种null的判断，从而让我们的领域层只关注业务逻辑的实现。

至于null模式，感觉有点小题大作了。

63 楼 xixix2004 2008-11-25  

jeff.chuh 写道

javaeyebird 写道

jeff.chuh 写道

由于程序的不严密而出现NullPointerException异常的代码太常见了，
虽然可以在使用对象前检查是否为空，但过多的这种检查会使代码臃肿。
出现这样的异常的时候常常没有明确的信息，给调试带来不便，
即使知道抛出异常的代码行号，但或许因为你手头没有代码，或许因为这行代码有多次方法调用，
而很难得到错误的原因或是判断究竟什么对象是空。

我觉得这是JVM的失职，它只是抛出了没有任何信息的
NullPointerException，
为什么不追加“调用XXX方法时，发现对象不可用”的信息呢

为什么必须要单独给NullPointerException加上“调用XXX时”呢？其他的Exception不需要加么？
因为从stack trace中就能得到方法调用的信息
try { ... } catch (Exception e) {
  log.debug(e); // 或者简单地e.printStackTrace()
}

只能得到某个方法执行到第几行的时候出现了NullPointerException，
而不知道那行代码究竟是什么，比如执行XX对象的XX方法这样的信息没有。

难道一定要依赖这样明确的信息你才有能力对自己编写的代码进行调试吗？这是最基本最基本的基本功了吧。

62 楼 chenkan2000 2008-11-25  

NullPointerException, ClassCastException，IndexOutOfBoundsException,这些RuntimeException都会经常发生的，但这些不可怕，关键是：
1. 养成良好的编程习惯，
2. 好的logger机制
3. 好的exception处理机制。

最讨厌有人写程序时把所有异常都捕获，简单打一句错误信息，异常不再抛出，搞得外部调用者不知道错在哪里。

61 楼 leadyu 2008-11-24  

真不明白，你要的这些，在异常栈信息里面不是都有？

而且，nullpoint是一个vm指令级别的错误，和一般我们包装好的，包含文字信息的错误可不同。

另外，‘带回变量名’，这更没办法了，经过编译，什么变量，都变成指针偏移量和常量了。

60 楼 jeff.chuh 2008-11-21  

java9981 写道

kaka2008 写道

datuo 写道

也与编程习惯有关。
比如经常字符串比较值的时侯
object.equals("")
为了避免空指针异常可以写成为：
"".equals(object)

这个。。。
这个还用说？

不喜欢这种写法，当然，这纯属个人爱好。
原因是：表义不明
面向对象的程序就应该用对象的角度写代码，
我们要测试的对象是object不是""，
我宁愿写成object!=null && object.equals("")

对于String来说一般写成object!=null && object.length > 0
这样效率比equals好，length不需要计算。

59 楼 java9981 2008-11-21  

kaka2008 写道

datuo 写道

也与编程习惯有关。
比如经常字符串比较值的时侯
object.equals("")
为了避免空指针异常可以写成为：
"".equals(object)

这个。。。
这个还用说？

不喜欢这种写法，当然，这纯属个人爱好。
原因是：表义不明
面向对象的程序就应该用对象的角度写代码，
我们要测试的对象是object不是""，
我宁愿写成object!=null && object.equals("")

58 楼 java9981 2008-11-21  

支持lz，虽然我觉得问题没那么严重，如果出现很多NullPointerException的话，
那应该考虑增加参数的检查，一般参数都是跟业务有关的，应该返回跟业务相关的错误代码，
而不应该要求jvm帮你处理了，业务相关的东西还是手工写吧。
jvm给的出错信息的确很不足，遇到NullPointerException的话jvm肯定是知道要调用哪个类的哪个方法，
完全可以把这些信息打印出来的，这根本就不会有速度上的影响。
而且使用的那个变量，如果有变量名的话完全可以给出变量名的。

57 楼 javaeyebird 2008-11-21  

jeff.chuh 写道

javaeyebird 写道

为什么必须要单独给NullPointerException加上“调用XXX时”呢？其他的Exception不需要加么？
因为从stack trace中就能得到方法调用的信息
try { ... } catch (Exception e) {
  log.debug(e); // 或者简单地e.printStackTrace()
}

只能得到某个方法执行到第几行的时候出现了NullPointerException，
而不知道那行代码究竟是什么，比如执行XX对象的XX方法这样的信息没有。

你是要得到运行时的某个xx对象么？怎么表示这个对象？
Foo@2342342么？照样不知道这个对象是哪个。。。
如果是说变量名，那从stack trace里看对应行的代码，就能看到诸如 a.foo(...)，这就知道是a变量存放的对象执行Foo方法

如果要得到详细的stack运行时信息，那得利用jvm的debug接口，这可以用于各个异常，不是专门针对NullPointerException

56 楼 icewubin 2008-11-21  

建议用apache的类判断文本：

StringUtils.isEmpty(str);
StringUtils.isNotEmpty(str);

利用Spring的Assert简化判断：

Assert.notNull(classA);
Assert.notNull(classB, "classB不能为空");

55 楼 amonlei 2008-11-21  

怎么会没有呢？耐心的把异常报告看完，肯定会看到自己程序部分的行号

54 楼 bloodrate 2008-11-21  

在未来的JDK7版本中,增加新特性a..b两个点引用如果a是null则不抛出空指针异常

53 楼 spiritfrog 2008-11-20  

我只觉得如果jvm连NullpointerException都不放过的话， 实在会引起很多开发时候的麻烦。还是就这样好，至少简单。

52 楼 kaka2008 2008-11-20  

datuo 写道

也与编程习惯有关。
比如经常字符串比较值的时侯
object.equals("")
为了避免空指针异常可以写成为：
"".equals(object)

这个。。。
这个还用说？

51 楼 niyanshi 2008-11-20  

jeff.chuh 写道

你这样写的代码会很健壮，我现在也是尽量这么做的，但是大项目中往往控制不了别人怎么写代码。

这个是项目组的best practice.事实上在做类设计的时候，我们把每个方法抛什么Exception也写进去了(Javadoc)。coder必须得按设计写代码，要抛的Exception必须抛出来，不然任务就没完成。

50 楼 datuo 2008-11-20  

也与编程习惯有关。
比如经常字符串比较值的时侯
object.equals("")
为了避免空指针异常可以写成为：
"".equals(object)

49 楼 rianychan 2008-11-20  

刚开始的时候遇到这种问题挺郁闷的

48 楼 jeff.chuh 2008-11-20  

抛出异常的爱 写道

jeff.chuh 写道

由于程序的不严密而出现NullPointerException异常的代码太常见了，
虽然可以在使用对象前检查是否为空，但过多的这种检查会使代码臃肿。
出现这样的异常的时候常常没有明确的信息，给调试带来不便，
即使知道抛出异常的代码行号，但或许因为你手头没有代码，或许因为这行代码有多次方法调用，
而很难得到错误的原因或是判断究竟什么对象是空。

我觉得这是JVM的失职，它只是抛出了没有任何信息的NullPointerException，
为什么不追加“调用XXX方法时，发现对象不可用”的信息呢

log4j.properte
info=debug
日志全开
再把你看着没问题的模块日志关闭
对于

obj.getaaa()
    .getbbbb()
      .getcccc();

用以上写法....

的确是一种方法，可以作为编码规范的一部分。

47 楼 jeff.chuh 2008-11-20  

harryzyp 写道

dennis_zane 写道

对象是null，没有带类型信息，jvm恐怕也没办法给出是什么对象。
在很多需要判断null地方，最好还是采用Null Object模式

Null Object模式是身？？？

google一下就有了，好像就是用不做任何事的对象代替null

46 楼 xiaojiit 2008-11-20  

自己感觉，还是这样的好，在JVM上加上信息话，虚拟机会又会慢很多，为了报错，每个调用都要检查，保存，会更慢的。
http://www.sharejava.cn/

45 楼 抛出异常的爱 2008-11-20  

jeff.chuh 写道

由于程序的不严密而出现NullPointerException异常的代码太常见了，
虽然可以在使用对象前检查是否为空，但过多的这种检查会使代码臃肿。
出现这样的异常的时候常常没有明确的信息，给调试带来不便，
即使知道抛出异常的代码行号，但或许因为你手头没有代码，或许因为这行代码有多次方法调用，
而很难得到错误的原因或是判断究竟什么对象是空。

我觉得这是JVM的失职，它只是抛出了没有任何信息的NullPointerException，
为什么不追加“调用XXX方法时，发现对象不可用”的信息呢

log4j.properte
info=debug
日志全开
再把你看着没问题的模块日志关闭
对于

obj.getaaa()
    .getbbbb()
      .getcccc();

用以上写法....