Java返回分配与分配然后返回。 （例如，在单例中）

Question

基本上，您可以拥有这样的代码：

baeldung.com

public static synchronized ClassSingleton getInstance() {
    if(instance == null) {
        instance = new ClassSingleton();
    }
    
    return instance;
}

您可以拥有像代码一样此：

我的版本：

public static synchronized ClassSingleton getInstance() {
    return instance = (instance == null) ? new ClassSingleton() : instance;
}

下面的代码以任何方式都更干净，但是Sonarlints Rule Java：S1121认为这是不合规的（主要的，代码味），

所以Sonarlint在谈论的可读性之外，还有更多吗？

我有一种奇怪的感觉，我的版本在返回之前一直在做任务，这是否可以表现不佳？

很高兴听到你们要说的话。

Answer 1

是的，它可能具有[可能可以忽略的]性能影响。可以通过尝试编译并查看出来的内容来测试这一点。这是Javac 16.0.1的字节码：

Compiled from "ClassSingleton.java"
public class ClassSingleton {
  public ClassSingleton();
    Code:
       0: aload_0
       1: invokespecial #1                  // Method java/lang/Object."<init>":()V
       4: return

  public static synchronized ClassSingleton getInstance();
    Code:
       0: getstatic     #7                  // Field instance:LClassSingleton;
       3: ifnonnull     16
       6: new           #8                  // class ClassSingleton
       9: dup
      10: invokespecial #13                 // Method "<init>":()V
      13: putstatic     #7                  // Field instance:LClassSingleton;
      16: getstatic     #7                  // Field instance:LClassSingleton;
      19: areturn

  public static synchronized ClassSingleton getInstanceShort();
    Code:
       0: getstatic     #7                  // Field instance:LClassSingleton;
       3: ifnonnull     16
       6: new           #8                  // class ClassSingleton
       9: dup
      10: invokespecial #13                 // Method "<init>":()V
      13: goto          19
      16: getstatic     #7                  // Field instance:LClassSingleton;
      19: dup
      20: putstatic     #7                  // Field instance:LClassSingleton;
      23: areturn
}

通过long = bad的老年度量，第二版显然更糟。尽管如此，在所有情况下，它都只能执行2个额外说明。第一个dup重复堆栈上的最后一项，因此它可以将该额外项目使用到putstatic将实例分配给堆栈上的最高值。我们可以推测，如果另一个线程在classSingleton上不同步，并且实例具有正确的属性，那么从理论上讲，我们可能会得到一些奇怪的行为，而实例可能无法正确设置。但是，考虑到同步为我们处理大多数，这似乎很不可能。

最后，JIT编译器可能会通过使用寄存器来消除对dup的需求，并且它有很大的机会弄清楚tat，它可以摆脱额外的putstatic也是如此。但是，我在JIT编译器上的经验不足，无法猜测它的行为。

话虽如此，只需使用第一个版本即可。它更容易读取并生成较短的字节码。