使用finally从setter返回旧值，不带临时变量

Question

我正在 Map.Entry 中实现 V setValue(V value) 方法，并完成了以下操作：

@Override public T setValue(T value) {
  try {
    return this.value;
  } finally {
    this.value = value;
  }
}

它按预期工作。

我知道有一些关于 finally 块之类的问题，但我找不到这个具体问题。您对这个特定用例有什么想法？这合法吗？应该避免吗？如果是，那么为什么或为什么不。谢谢！

Answer 1

更喜欢简单、明显的代码。即

@Override public T setValue(T value) {
    T old = this.value;
    this.value = value;
    return old;
}

此外，这可以扩展为[本地]线程安全（如 NIO attach）。

private final AtomicReference<T> valueRef;

@Override public T setValue(T value) {
    return valueRef.getAndSet(value);
}

Answer 2

它有效，但它并不那么明显（易于理解）：

@Override public T setValue(T value) {
  T oldValue = this.value;
  this.value = value;
  return oldValue;
}

为什么不使用这种更简单的形式？

Answer 3

嗯，我认为，目前来说，这是绝对合法的。通常，您使用finally 子句来指定在返回之前分配执行o 的值。但有时我们习惯将其放入返回值中，并引发警告。我认为你必须避免这种返回，但你可以使用finally来完成其余的事情。

否则，它就像一个过时的函数，无论您将这些赋值放在finally还是try/catch块之后，都没有关系。