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

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

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

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）所说：

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

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