java – 访问器方法性能和优化

通常,我会遇到代码,其中重复使用/滥用Getter方法来获取某些值或将其作为方法参数传递,例如：

public class Test {
   public void someMethod() {
      if(person.getName() != null && person.getName().equalsIgnoreCase("Einstein")) {
           method1(person.getName());
      }
      method2(person.getName());
      method3(person.getName());
      method4(person.getName());
   }
}

我通常编码,如下所示：

public class Test {
   public void someMethod() {
      String name = person.getName();
      if(name != null && name.equalsIgnoreCase("Einstein")) {
           method1(name);
      }
      method2(name);
      method3(name);
      method4(name);
   }

在我看来,将getter分配给变量并使用它有很大的内存/性能优势,因为Getters是Java方法并使用堆栈帧.编码方式真的有相当大的优势吗？
}

解决方法

您最近是否了解过您的意见.

性能：

这可能是一个微观优化,在1999-2001之前的1.2 JVM中需要关注的事情,即使这样,我也会质疑它,除非一些严肃的数字显示不同.

现代JIT实施会告诉您,今天您的意见不合适.

现代编译器实现会进行各种优化,这使得在这样的事情上思考会浪费Java时间. JIT只是让人更加担心.

逻辑：

在并发情况下,如果要查看更改,则两个代码块在逻辑上不等同,使本地副本可以防止这种情况发生.根据您的想法,一种或另一种方法可能会产生非常微妙的非确定性错误,这些错误在更复杂的代码中很难确定.

特别是如果返回的内容是可变的,而不像String是不可变的.然后,即使是本地副本也可能会发生变化,除非你做了深度克隆,并且很快就会很容易出错.

关注自己正确地做,然后测量然后优化重要的东西,只要它不会使代码不易维护.

如果除了返回this.name之外的方法调用中没有任何内容,JVM将内联对最终实例成员的任何调用并删除方法调用;它知道访问器方法中没有逻辑,并且它知道引用是最终的,因此它知道它可以内联该值,因为它不会改变.

为此目的

person.getName() != null && person.getName().equalsIgnoreCase("Einstein")

被更正确地表达为

person != null && "Einstein".equalsIgnoreCase(person.getName())

因为没有机会出现NullPointerException

重构：

现代IDE重构工具删除了有关必须在一堆地方更改代码的任何争论.