存取方法性能和优化

Question

通常,我会遇到代码,其中重复使用/滥用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方法并使用堆栈帧.编码方式真的有相当大的优势吗？}

Answer 1

你有没有异型您的意见最近.

性能:

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

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

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

逻辑:

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

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

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

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

为此目的

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

被更正确地表达为

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

因为没有机会 NullPointerException

重构:

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

Answer 2

没有性能差异。如果有的话，它很小。

更重要的是，这两个代码示例在存在多个线程的情况下实际上完全不同。

说到一半，有人打电话setName并更改名称。现在，您的代码正在经历一条完全意外的路径！实际上，这实际上是几个历史（甚至是内核级）安全漏洞的根本原因。

请注意，我并不是在盲目地将您得到的所有结果复制到局部变量中。我只是指出一个潜在的陷阱。