从Timestamp中bug了解equals设计原则

读《Effective Java》有感。本文分析了equals方法在java编写中的一些原则,以及分析了jdk源码中存在的一个设计缺陷的案例

image.png

上面我们写一个单元测试,用于测试timestamp和date对象之间的equals方法的对称性。

根据equals的对称性原则,如果俩个非空对象a和b,如果存在a.equals(b)为true,那么一定存在b.equals(a)为true。但我们得到的运行结果却大不一样:date和ts equals结果为true,ts和date的equals的结果为false。

基于我们对equals对称性原则的认知,上述结果明显和原则不符,那么是什么原因导致这种现象的发生,为什么jdk又允许了这种现象的发生呢。

这里补充一个知识点:Timestamp的父类是java.util.Date,然后在timestamp中加了一个纳秒属性

分析问题之前,我们需要先了解equals的几大原则:

  1. 对称性(Symmetry):如果两个对象相互比较,比如 a.equals(b) ,则 b.equals(a) 也应该返回相同的结果。换句话说,对于任意的非空对象 a 和 b,如果 a.equals(b) 返回 true,则 b.equals(a) 也应该返回 true。

  2. 自反性(Reflexivity):一个对象通过调用其自身的 "equals" 方法应该返回 true。换句话说,对于任意的非空对象 a,a.equals(a) 应该返回 true。

  3. 传递性(Transitivity):如果对象 A 和对象 B 相等,并且对象 B 和对象 C 相>等,那么对象 A 和对象 C 也应该相等。换句话说,如果 a.equals(b) 返回 true,b.equals(c) 也返回 true,则 a.equals(c) 应该返回 true。

  4. 一致性(Consistency):对于任意的非空对象 a 和 b,如果对象的状态没有发生变化,那么多次调用 a.equals(b) 应该始终返回相同的结果。换句话说,如果 a 和 b 的内容没有发生变化,则多次调用 a.equals(b) 应该返回相同的结果。

  5. 非空性(Non-nullity):对于任意的非空对象 a,a.equals(null) 应该返回 false。换句话说,对象和 null 比较应该始终返回 false。

而我们上述的问题牵扯的就是第一个原则:对称性。

下面,我们看下Timestamp和Date在设计上为何违反了equals的对称性

    /**
     * Tests to see if this <code>Timestamp</code> object is
     * equal to the given object.
     *
     * This version of the method <code>equals</code> has been added
     * to fix the incorrect
     * signature of <code>Timestamp.equals(Timestamp)</code> and to preserve backward
     * compatibility with existing class files.
     *
     * Note: This method is not symmetric with respect to the
     * <code>equals(Object)</code> method in the base class.
     *
     * @param ts the <code>Object</code> value to compare with
     * @return <code>true</code> if the given <code>Object</code> is an instance
     *         of a <code>Timestamp</code> that
     *         is equal to this <code>Timestamp</code> object;
     *         <code>false</code> otherwise
     */
    public boolean equals(java.lang.Object ts) {
      if (ts instanceof Timestamp) {
        return this.equals((Timestamp)ts);
      } else {
        return false;
      }
    }

注意到方法注释上有一句话很重要:

This version of the method <code>equals</code> has been added to fix the incorrect signature of <code>Timestamp.equals(Timestamp)</code> and to preserve backward

所以Timestamp还提供了另外一个equals方法,专门用来比较Timestamp类型的,如下:

    /**
     * Tests to see if this <code>Timestamp</code> object is
     * equal to the given <code>Timestamp</code> object.
     *
     * @param ts the <code>Timestamp</code> value to compare with
     * @return <code>true</code> if the given <code>Timestamp</code>
     *         object is equal to this <code>Timestamp</code> object;
     *         <code>false</code> otherwise
     */
    public boolean equals(Timestamp ts) {
        if (super.equals(ts)) {
            if  (nanos == ts.nanos) {
                return true;
            } else {
                return false;
            }
        } else {
            return false;
        }
    }

综上我们可以进行简单总结下:

  1. 重写的Object的equals方法,如果传入对象ts是Timestamp则调用重载的equals方法进行比较,如果不是Timestamp类型,则直接返回false。
  2. 重载的equals方法主要是调用父类(java.util.Date)的equals方法,之后在比较timestamp的nanos

简单提供一个图助于理解:


未命名.drawio (1).png

为了方便比较,下面截取了Date类型的equals方法

    /**
     * Compares two dates for equality.
     * The result is <code>true</code> if and only if the argument is
     * not <code>null</code> and is a <code>Date</code> object that
     * represents the same point in time, to the millisecond, as this object.
     * <p>
     * Thus, two <code>Date</code> objects are equal if and only if the
     * <code>getTime</code> method returns the same <code>long</code>
     * value for both.
     *
     * @param   obj   the object to compare with.
     * @return  <code>true</code> if the objects are the same;
     *          <code>false</code> otherwise.
     * @see     java.util.Date#getTime()
     */
    public boolean equals(Object obj) {
        return obj instanceof Date && getTime() == ((Date) obj).getTime();
    }

Date类的equals的逻辑表示:

  1. 如果obj是Date类型或者obj的父类为Date类型
  2. 将obj强转为Date类型后的time和this.time相等则返回true。
    上述二者缺一不可。

我们通过分析Timestamp和Date的equals方法,可以发现,他们为什么会返回不一样的结果:

  1. Timestamp的equals方法不仅会先比较类型,还会比较time属性,还会比较另外一个nanos属性是否相等。
  2. Date的equals方法只会比较time属性。

同时正是因为timestamp和date类型之间存在者继承关系,所以导致在判断类型的时候timestamp instanceof date可以通过,而反之则不通过,这也是违反equals对称性的最大的一个原因。

以上的分析我们明白了在jdk中timestamp和date之间equals对称性之间的纠缠,但文章的研究还未结束,其实我们可以修改Timestamp源码来解决这个对称性问题。下面是修改后的源码:

public class MyTimestamp extends Timestamp {

    public MyTimestamp(long time) {
        super(time);
    }

    @Override
    public boolean equals(Object ts) {
        if (ts instanceof Timestamp) {
            return this.equals((Timestamp)ts);
        } else if (ts instanceof Date) {
            return this.getTime() == ((Date) ts).getTime();
        } else {
            return false;
        }
    }
}

我们来编写测试来测试我们自己构造的Timestamp:


image.png

可以看到测试是通过的,也就是说我们修改后的Timestamp的equals源码增加了对Date类型的支持后,就解决了对称性的问题。
但真的有这么简单吗?我们能想到的自然前辈们也能想到,那么这种写法会有什么问题呢?我们在看一个单元测试,如下:


image.png

单元测试中,我们声明了三个对象,date、myTimestamp1、myTimestamp2,通过我们对前一个单元测试的推断,date和myTimestamp1以及date和myTimestamp2是互相等价的,但是32行myTimestamp1和myTimestamp2却不想等。

文章开头我们提及过,equals一共有五个原则,我们上面分析了对称性,这里涉及到的就是传递性。如果o1和o2相等,o2和o3相等,那么o1等于o3,这里的o2可以带入为date,o1和o3分别为myTimestamp1和myTimestamp2

所以我们修改过的源码虽然能解决对称性,但是无法解决传递性,其实在这里,因为date和timestamp存在继承关系,必然无法满足equals的其中之一的准则。这是jdk设计上的一个缺陷。

关于此类问题,在《Effective Java》第二版一书中33页有提供相关解释。

事实上,这是面向对象语言中关于等价关系的一个基本问题。我们无法在拓展可实例化的类的同时,既增加新的值组件,同时又保留equals约定,除非愿意放弃面向对象的抽象所带来的优势。

同时还提供了相关的解决方案:

复合优先于继承

目前java相关组件和框架编写代码中,对于非业务(框架和公用jar包)的对象模型,一般都使用复合来替代继承。

到这里,本文通过一个简单的例子分析了jdk中一个坏的例子去了解equals在设计上需要满足的准则。让我们在后续的项目代码中保证合理的设计是研发必要的考虑。

下面是出自于《Effective Java》一书中对于实现高质量equals方法的诀窍

  1. 使用==操作符检查“参数是否为这个对象的引用”
  2. 使用instanceof操作符检查“参数是否为正确的类型”
  3. 把参数转换成正确的类型
  4. 对于该类中的每个“关键(significant)”域,检查参数中的域是否与该对象对应的域相匹配。
  5. 当你编写完成了equals方法之后,应该问自己三个问题:它是否是对称的、传递的、一致的?并且不要只是自问,还需要编写单元测试来检验这些特性
最后编辑于
©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 212,686评论 6 492
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 90,668评论 3 385
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 158,160评论 0 348
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 56,736评论 1 284
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 65,847评论 6 386
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 50,043评论 1 291
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 39,129评论 3 410
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 37,872评论 0 268
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 44,318评论 1 303
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 36,645评论 2 327
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 38,777评论 1 341
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 34,470评论 4 333
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 40,126评论 3 317
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 30,861评论 0 21
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 32,095评论 1 267
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 46,589评论 2 362
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 43,687评论 2 351

推荐阅读更多精彩内容