`

.toString

    博客分类:
  • java
 
阅读更多

(转)近天从NullPoint异常说起,先看以下的代码

 

String defaultFollowGroup = ConfUtil.getValue("defaultFollowGroup").toString();

 

今天由于配置文件里的defaultFollowGroup这个属性没有写,造成抛出空异常,由于注册用户也用到了这段代码,

造成了用户注册过程也抛错,服务抛错,前台高速服务处理模块对于出现异常没有继续处理,造成出现异常时,没有打印出任何返回值。

 

通过统一规范,可以在一定程度上减少这类问题出现的几率,比如后台服务必须trycatch,把异常处理做好。

 

从代码的健壮性上讲,ConfUtil.getValue("defaultFollowGroup").toString();这句相当差,因为ConfUtil不处理空值情况,造成toString()

抛出异常。面向对象编程里,连续的“.”是非常危险的,除非每一个方法都做好了异常处理。在每一次“.”的时候,都要思考前边得到的对象

有没有可能是空的。

 

比较好的改进方法是,增强工具类,简单来说,由于读取配置文件大部分都是要得到字符串,可以增加一个默认的处理过程,比如空值时

返回""或者抛出明确的异常。或者增加一个重构方法,标明对于空值时的处理方式,isReturnNullWhenPropertyIsNull。

 

早些年由于此类问题出过一些事故,所以每次看到连续的“.”我都会神经过敏,感觉浑身不自在,算是有强迫症,也和很多人争论过。

对于这类编程习惯有各种流派,各有利弊,欢迎各位一起探讨,退役大神、首席架构师、各主程、编程小将多多参与。

分享到:
评论

相关推荐

Global site tag (gtag.js) - Google Analytics