使用变量存储值还是直接从对象获取值？

Question

场景是我需要至少两次访问一个值。 即我正在使用记录器来跟踪应用程序中正在发生的事情。 我想记录对象的名称，该函数正在运行，以后再使用相同的名称进行操作（即检查它是否包含一些字符串或将其放入数组中）。

将名称存储在变量中：

foo(Bar bar){
    String name = bar.getName();
    logger.info("I am working with "+name);
    this.doSomethingWith(name);
}

或两次调用getName（）：

foo(Bar bar){
    logger.info("I am working with "+bar.getName());
    this.doSomethingWith(bar.getName());
}

我了解，在第一种情况下，我将创建一个新的String，为其分配一个值，然后两次检索该值。 这样，我正在使用更多的内存资源，对吗？

在第二种情况下，我是否两次访问对象栏，然后两次访问其名称。 我想这不是DRY方法。 但另一方面，我不是在记忆中重复自己，对吗？

哪种方法更好？

Answer 1

在第一个示例中，您没有使用更多的内存，因为String是一个不可变的对象。 这样，对String的任何引用都只是指向内存中同一对象的指针。

后一个选项还存在一些线程安全性问题，其中getName()的结果可能在两次调用之间发生变化。 尽管可能不太可能，但是您可能需要考虑这一点。

考虑到这一点，我将建议第一个选项，即使它更“健谈”。

注意： getName()也有可能是通过计算生成的，在这种情况下，您实际上会以比第一种方法更多的内存使用第二种方法。

Answer 2

您的Bar应该是一成不变的阶级（在大多数情况下）。 对于不可变的类，这些方法是相同的，因此您可以随意选择。

仅当Bar可变时，才会发生任何实际问题，因此bar.name值可以在两次读取之间更改。 但是这种情况将使用Bar作为域对象（似乎是这样）进行渲染是毫无意义的。 您可以通过在foo()顶部创建bar本地副本来解决这种情况。 然后，再次获得原始bar本地副本后，您可以选择自己喜欢的任何方式。

因此，这是口味问题。 是的，在第一种情况下，您可能会浪费一点内存用于本地引用，但是最有可能的是，JVM将对其进行优化，使其看起来像字节码级别的第二种方式。

Answer 3

就个人而言，我更喜欢第二种方法。 确实，我通常仅在必要时创建临时变量。

Martin Fowler（ http://en.wikipedia.org/wiki/Martin_Fowler ）也遵循此准则。 他在我读过的书中提到了它：

http://www.amazon.fr/Refactoring-Improving-Design-Existing-Code/dp/0201485672

有关此主题的书的免费摘录在这里：

http://sourcemaking.com/refactoring/replace-temp-with-query

有人认为删除临时变量可能会导致性能问题。

正如马丁·福勒（Martin Fowler）所说：

在这种情况下，您可能会担心性能。 与其他性能问题一样，让它暂时滑动。 十分之九的时间无所谓。 当这很重要时，您将在优化过程中解决问题。 更好地分解代码后，您通常会发现更强大的优化，而无需重构就可能会错过这些优化。 如果情况变得更糟，很容易将温度降低。

但是无论如何，这是一个品味问题。 有些人发现第一种方法更具可读性，其他人则发现第二种方法。 我真的更喜欢第二种方法，因为我讨厌为临时变量添加没有实际值的行:)