实践中的重构08_代码的可读性及If(null==a)和If(list.size()<1)

代码的可读性应该还是着重强调的。

如

if(null==a)

这样的代码，至少我读起来有点别扭。

当然，更厉害的是

if(list.size()<1)

我知道有些人喜欢写成

if(list.size()==0)

ok,这个我还可以想的通为什么这么写，但是<1这种写法真的让人无语。

if(list.isEmpty())

这样当然更进一步。

毕竟，代码是给人读的。

好吧，其实一般都是这么用的

if(list!=null && !list.isEmpty()){
// do something.
}

我们可以把对list的判null和判空抽出来，变成

if(!CollectionUtil.isEmpty(list)){
// do something.
}

更进一步。

if(CollectionUtil.isNotEmpty(list)){
// do something.
}

问题在于，null和empty在特定的上下文中真的是一样的吗。

null == a是c/c++年代遗留下来的写法，那时候IDE不会检查条件语句里面的赋值，为了避免写成if(a=null)，现在这种写法在Java中是没什么必要了。

java里面这种写法比较有意义的是和常量比较的时候，if(a.equals(CONST_STRING))就没有if(CONST_STRING.equals(a))来的好。后面不会产生空指针异常。如果实在不喜欢判空，建议使用NullObject模式。

同理，下面的list，考虑防御性原则，前面没有判空的话，也经常会写if(list==null || list.isempty())

遗留的写法的原因我是知道的，所以想着大家在java世界中不要这么搞了。

if(CONST_STRING.equals(a))这个是有一定的道理。



这个当然你是对的。但是我的题目是可读性，所以list和null的判断就没有写出来。
我补上了对判null的想法。

大多数时候关心是集合中 有元素,没有的话,可以不考虑!

list.size()<1
这么些我觉得没什么可读性差的。只是少了null的判断。

list 就不应该为null

对，除非是第3方库某个本该返回集合的方法返回了个null才需要这么处理（我认为那也是这个库的作者不专业，而且即使是第3方库也可以自己修改）。如果是自己写的方法，从语义上说，这个方法的返回值如果是一个集合，那在该方法运行结果是这个集合为空的情况下，怎么也不应该返回null。1是影响可读性，2是其它每个调用该方法的地方还要处理这个null。返回null是自己给自己找麻烦。



如果给自己一个“集合为空的情况下不返回null而是返回一个空集合”的原则，就不会有这个问题。

而且即使遇到别人的代码在返回空集合时用null替代，这时候是把别人的代码改简单些好呢？还是把自己的代码改复杂来适应别人的代码好？

这样的话，lz在http://www.iteye.com/topic/841196中的代码最后可以变成这样：

	private List<User> getUserList2(int size) {

		List<User> userList = new ArrayList<User>();

		userList.addAll(getUserListFromSource1(size - userList.size()));
		
		if (userList.size() >= size) return userList;

		userList.addAll(getUserListFromSource2(size - userList.size()));
		
		if (userList.size() >= size) return userList;

		userList.addAll(getUserListFromSource3(size - userList.size()));

		return userList;
	}

而且这代码还有改进的余地。但是没有上下文（对getUserList的调用；还有getUserListFromSourcex的定义），我只能写成这样。

lz在另一帖中说到“平铺直叙”，我想起有一种很简单实用的方法可以写出较高质量的代码（包括强可读性，绝对的DRY，足够的可扩展）：假设式脚本编写法（scripting by assumption）或称一厢情愿式编程法（programming by wishful thinking）。

楼上给解释一下

其实蛮简单的，和LZ所说的平铺直叙同一个意思，举个最近写的代码的例子：

首先是描述最直接，最表面的需求：

定义 类：节点服务器
  定义 方法： 启动节点服务器
    方法实现： 
    加载配置文件()
    IPQAM开始广播()
    连接上分发服务器()
    开始监听客户端连接和请求()
    每5秒检查并清除一次死掉的终端连接()
    记录系统启动日志()

然后会发现隐藏在更低一层的需求，发现需要定义加载配置文件，应该由一个专门配置系统的类来做：

定义 类：配置
   定义 方法： 加载
   方法实现：
   xxxxxxxx
   ......

然后是IPQAM的广播：

定义 类：IPQAM
   定义 方法： 广播
   方法实现：
   遍历所有频道
      频道.发送广播()
   遍历结束

然后又发现隐藏在IPQAM中的更底层的需求，需要定义一个频道要处理频道自身的逻辑：

定义 类：频道
   定义 方法： 发送广播
   方法实现：
   UDPSocket.new.connect(ip, port).send('boradcast message', 0)

只要把以上人类语言翻译成相应的代码即可。