相同的多个向下倾斜;他们会优化吗?

时间:2015-10-01 11:22:43

标签: java equals downcast

我倾向于在Java中将我的equals方法写成一个班轮......

class Test {
   private String a = "";
   private Integer b = Integer.MIN_VALUE;
   private Long c = Long.MIN_VALUE;

   public Test(final String a, final Integer b, final Long c) {
       this.a = a;
       this.b = b;
       this.c = c;
   }

   @Override
   public boolean equals(final Object obj) {
        return obj instanceof Test && ((Test) obj).a.equals(this.a)
          && ((Test) obj).b.equals(this.b)
          && ((Test) obj).c.equals(this.c);
   }
}

正如您所看到的,在这种方法中,我将Object实例向下转换为Test实例多次。我的问题是,它是否会被编译器优化,所以如果我像这样编写我的equals方法,那么将会有一个向下转换而不是三个向下转换?

 public boolean equals(final Object obj) {
     if (obj instanceof Test) {
         final Test test = (Test) obj;
         return test.a.equals(this.a) && test.b.equals(this.b)
           && test.c.equals(this.c);
     } else {
         return false;
     }
 }

它不是评论中question的副本,因为我感兴趣的是简洁方法的实现(6行对3),但不是以性能下降为代价。在另一个问题中,差异是一行。

1 个答案:

答案 0 :(得分:2)

当我对类进行反编译时,我发现Eclipse编译器和javac确实在checkcast方法中生成了三条equals指令,因此不会进行优化。

当通过Hotspot进行优化时,热点可能会发现一次投射就足够了。

反编译的字节码:

public boolean equals(java.lang.Object);
   Code:
      0: aload_1
      1: instanceof    #13                 // class sov/Test
      4: ifeq          62
      7: aload_1
      8: checkcast     #13                 // class sov/Test
     11: getfield      #3                  // Field a:Ljava/lang/String;
     14: aload_0
     15: getfield      #3                  // Field a:Ljava/lang/String;
     18: invokevirtual #14                 // Method java/lang/String.equals:(Ljava/lang/Object;)Z
     21: ifeq          62
     24: aload_1
     25: checkcast     #13                 // class sov/Test
     28: getfield      #7                  // Field b:Ljava/lang/Integer;
     31: aload_0
     32: getfield      #7                  // Field b:Ljava/lang/Integer;
     35: invokevirtual #15                 // Method java/lang/Integer.equals:(Ljava/lang/Object;)Z
     38: ifeq          62
     41: aload_1
     42: checkcast     #13                 // class sov/Test
     45: getfield      #12                 // Field c:Ljava/lang/Long;
     48: aload_0
     49: getfield      #12                 // Field c:Ljava/lang/Long;
     52: invokevirtual #16                 // Method java/lang/Long.equals:(Ljava/lang/Object;)Z
     55: ifeq          62
     58: iconst_1
     59: goto          63
     62: iconst_0
     63: ireturn