在对象上调用getter而不是将其存储为局部变量（内存占用，性能）
在下面的代码中，我们调用listType.getDescription()两次：
for (ListType listType: this.listTypeManager.getSelectableListTypes()) { if (listType.getDescription() != null) { children.add(new SelectItem( listType.getId() , listType.getDescription())); } }
我倾向于重构代码以使用单个变量：
for (ListType listType: this.listTypeManager.getSelectableListTypes()) { String description = listType.getDescription(); if (description != null) { children.add(new SelectItem(listType.getId() ,description)); } }
我的理解是JVM以某种方式针对原始代码进行了优化，尤其是嵌套调用，如children.add(new SelectItem(listType.getId(), listType.getDescription())); 。
比较两个选项，哪一个是首选方法，为什么？ 这就是内存占用，性能，可读性/易用性以及其他我现在不想到的内容。
后一个代码片段何时变得比前者更有利，也就是说，当使用临时局部变量变得更合适时，是否有任何（近似）数量的listType.getDescription()调用，因为listType.getDescription()总是需要一些堆栈存储this对象的操作？
我几乎总是喜欢局部变量解决方案。
内存占用
单个局部变量需要4或8个字节。 它是一个参考，没有递归，所以让我们忽略它。
性能
如果这是一个简单的getter，JVM可以自己记忆，所以没有区别。 如果这是一个无法优化的昂贵电话，手动记忆会使其更快。
可读性
遵循DRY原则。 在你的情况下，它几乎不重要，因为局部变量名称在字符方面与方法调用一样长，但对于更复杂的东西，它是可读性的，因为你不必找到两个表达式之间的10个差异 。 如果您知道它们是相同的，那么使用局部变量清楚地表明它们。
正确性
想象一下，你的SelectItem不接受null ，你的程序是multithreading的。 listType.getDescription()的值可以在此期间发生变化并且您已经被烤了。
调试
拥有包含有趣值的局部变量是一个优点。
通过省略局部变量来获胜的唯一方法就是保存一行。 所以我只会在无关紧要的情况下这样做：
- 很短的表达
- 没有可能的并发修改
- 简单的私人终极吸气剂
我认为第二种方式肯定更好，因为它提高了代码的可读性和可维护性 ，这是最重要的事情。 除非你编写一个每毫秒都很重要的应用程序，否则这种微优化不会对你有任何帮助。
我不确定哪一个是首选。 我更喜欢的是清晰可读的代码，而不是性能代码，特别是当性能增益可以忽略不计时。 在这种情况下，我怀疑接下来没有明显的区别（特别是考虑到JVM的优化和代码重写function）
在命令式语言的上下文中，函数调用返回的值无法被记忆（请参阅http://en.m.wikipedia.org/wiki/Memoization ），因为无法保证该函数没有副作用。 因此，您的策略确实避免了函数调用，代价是分配临时变量来存储对函数调用返回的值的引用。 除了稍微高效一点（除非在循环中多次调用该函数，这并不重要），我会选择你的风格，因为更好的代码可读性。
只有当局部变量的名称是自我记录时，我才同意局部变量方法的可读性。 称它为“描述”是不够的（哪种描述？）。 将其称为“selectableListTypeDescription”将使其清楚。 我想抛出for循环中的递增变量应命名为“selectableListType”（特别是如果“listTypeManager”具有其他ListTypes的访问器）。
另一个原因是，如果不保证这是单线程或您的列表是不可变的。