通常,我会遇到代码,其中重复使用/滥用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
重构：
