残酷的非空if判断

经常我们在开发过程中会出现很多if（xx==null）等这样的判断，而且基本上会占据大量的这样非空的这样的判断，对于编码来说很反感，但是在代码中如果不进行这样的判断又会造成程序的bug，  大家有什么好的想法能解决这个问题，大家平时的编码习惯是怎么样的 一起讨论

评论

15 楼 jansel 2011-05-18  

Assert
NullObject

14 楼 uhlan 2011-05-18  

非空检查是不可避免的
因为很多基础接口会抛出空指针，比如正则：Matcher m = pattern.matcher();可能会抛出空指针 ，如果方法需要调用pattern.matcher，就必须预先做非空检查

非空检查一般分为两种
1. 空值为合法输入，有很多方便的工具类能简化非空检查
    String 可以用 org.apache.commons.lang.StringUtils.isEmpty(str)
    Collection和Map可以用 org.springframework.util.CollectionUtils.isEmpty()
    如果自定义对象的话，更可以使用Null Object模式减少这种判断
2. 空值不合法
   可以使用Assert做：org.springframework.util.Assert.notNull(obj,message);

13 楼 steafler 2011-05-18  

不要返回空指针

12 楼 trjianjianjiao 2011-05-17  

一般不好避免。如果总是对同一个类判空，可以考虑设计一个对应的NullObject。

11 楼 watanoeye 2011-05-17  

freemarker中:
<#if name??>${name}</#if>
或者直接 ${name!"ooxx"}
还可以: ${name???string(name, 'ooxx')}

10 楼 duqiangcise 2011-05-17  

if(null!=name) {  
} 
/////
if(StringUtils.isNotBlank(name)) {  
} 

9 楼 幻无极 2011-05-17  

还是js的写法轻松点

8 楼 sam.ds.chen 2011-05-17  

纠正两个笔误

ray_linn 写道

一个东西叫design by contact

contact应为contract

night_stalker 写道

...
如果用 groovy，很多情况就能用 .? 运算符简化，读和写都舒服很多。
...

.?应为?.

7 楼 frogfool 2011-05-17  

chenchuan 写道

经常我们在开发过程中会出现很多if（xx==null）等这样的判断，而且基本上会占据大量的这样非空的这样的判断，对于编码来说很反感，但是在代码中如果不进行这样的判断又会造成程序的bug，  大家有什么好的想法能解决这个问题，大家平时的编码习惯是怎么样的 一起讨论

if(null==xx)  {
}

//建议常量前置

if(null!=name && name.equals("ITEYE")) {
}

有时可能少写了判断

if("ITEYE".equals(name)) {
}

6 楼 xieye 2011-05-17  

如果php

if (!$xx) {
  .....
}

5 楼 yangyi 2011-05-17  

http://en.wikipedia.org/wiki/Null_Object_pattern

4 楼 night_stalker 2011-05-17  

语言缺陷，基本无解。

某些情况可以用 @NotNull 和 @Nullable 使代码清晰些 ⋯⋯ 但是不会有太大帮助。
guava 的 Strings.isNullOrEmpty() 也有些用。

如果用 groovy，很多情况就能用 .? 运算符简化，读和写都舒服很多。
如果用 scala，用 option 传参数可以强制实现的人做空判断，可以减少一些疏忽或者偷懒造成的问题⋯⋯
最完美的解决方法就是用一种没有 null / nil 值的语言，如 haskell，不过那些靠写 if(null != xx) 吃饭的人肯定不同意⋯⋯

3 楼 抛出异常的爱 2011-05-17  

NullObject 对于有出空可能的
对于不应该出空的地方加断言

2 楼 ray_linn 2011-05-17  

一个东西叫design by contact

1 楼 treemap 2011-05-17  

你要觉得不爽 专门 封装 一个 方法好 了

数据有效性判断是 必须的 

就跟 非运行异常 需要处理一样