apache commons-lang.jar源码问题

Question

在EqualsBuilder中append（Object lhs,Object rhs）是不是应该把 
     if (lhs == null || rhs == null) {
         this.setEquals(false);
         return this;
     }
放在：
     if (lhs == rhs) {
         return this;
     }
前面？？？？
因为lhs和rhs都为null的时候，if(lhs == rhs)为true，已经return了.if (lhs == null || rhs == null)将不会起到作用！

Answer 1

if(lhs == rhs)成立时，表示两者为同一对象，或者都为null，总之是相等的

Answer 2

那还要if (lhs == null || rhs == null)判断做什么？？？那不是多此一举么！！！

Answer 3

不太清楚这里要实现什么样的逻辑功能，但是看样子类似equals()，建议楼主参照一下其他equals()方法...按照楼主的说法，似乎应该将：
if (lhs == rhs) {
    return this;
}
改为：
if(lhs != null && lhs == rhs) {
    return this;
}