return 语句之前的局部变量，这有关系吗？

Question

对不起，如果这是一个新手问题，但我找不到答案。 这样做更好吗：

int result = number/number2;
return result;

或：

return number/number2;

我知道整数使用内存，所以我猜它会稍微降低性能？但另一方面，它使内容更清晰，尤其是当 int/string 是一个长计算时。

Answer 1

编辑：如果像我一样，您使用的 Kotlin 比 Java 还多，知道 IntelliJ 在 Kotlin 中也对此进行检查也很重要：

变量只在后面的返回中使用，应该被内联

此检查报告仅在下一个 return 语句中使用的局部变量或其他变量的精确副本。在这两种情况下，最好内联这样一个变量。

---

实际上有一个从 PMD 继承的名为 Unnecessary Local Before Return 的 SonarQube 规则讨论了这一点。它说：

避免不必要地创建局部变量。

这条规则是 SSLR 规则的later replacedVariables should not be declared and then immediately returned or thrown，它保持相同的位置：

声明一个变量只是为了立即返回或抛出它是一个错误 练习。一些开发人员认为这种做法改进了代码 可读性，因为它使他们能够明确地命名正在发生的事情 回来。但是，此变量是内部实现细节 不会暴露给方法的调用者。 方法名 应该足以让来电者确切知道会发生什么 返回。

我完全同意。

IntelliJ（或至少是 Android Studio）也有针对这种情况的警告：

变量只在后面的返回中使用，可以内联

此检查报告仅在下一次返回中使用的局部变量或其他变量的精确副本。在这两种情况下，最好内联这样一个变量。

---

我认为在这种情况下根本不需要担心性能问题。话虽如此，正如@Clashsoft 在他的评论中提到的那样，JIT 很可能会内联变量，无论哪种方式，您最终都会得到相同的结果。

Answer 2

选择您认为更具可读性的版本。

存在命名变量提高可读性的合理情况。例如

public String encrypt(String plainString)
{
    byte[] plainBytes      = plainString.getBytes(StandardCharsets.UTF_8);
    byte[] hashPlainBytes  = enhash( plainBytes, 4 );
    byte[] encryptedBytes  = doAes128(Cipher.ENCRYPT_MODE , hashPlainBytes );
    String encryptedBase64 = Base64.getEncoder().withoutPadding().encodeToString(encryptedBytes);
    return encryptedBase64;
}

public String decrypt(String encryptedBase64)
{
    byte[] encryptedBytes = Base64.getDecoder().decode(encryptedBase64);
    byte[] hashPlainBytes = doAes128(Cipher.DECRYPT_MODE , encryptedBytes );
    byte[] plainBytes     = dehash( hashPlainBytes, 4 );
    String plainString = new String(plainBytes, StandardCharsets.UTF_8);
    return plainString;
}

在某些情况下，我们需要与返回类型不同类型的变量。这会影响类型转换和推理，从而产生显着的语义差异。

Foo foo()            vs.        Foo foo()
{                               {
                                    Bar bar = expr;
    return expr;                    return bar;
}                               }

Answer 3

编译器通常足够聪明，可以适当地优化这类事情。请参阅维基百科上的data-flow optimizations。

在这种情况下，即使您自己没有指定一个，它也可能需要分配一个临时变量来存储结果。

编辑：Clashsoft 关于字节码编译器是正确的：

$ cat a.java
class a {
   public static int a(int x, int y) {
     return x / y;
   }

   public static int b(int x, int y) {
     int r = x/y;
     return r;
   }

   public static int c(int x, int y) {
     final int r = x/y;
     return r;
   }
}
$ javap -c a
Compiled from "a.java"
class a {
  a();
    Code:
       0: aload_0
       1: invokespecial #1                  // Method java/lang/Object."<init>":()V
       4: return

  public static int a(int, int);
    Code:
       0: iload_0
       1: iload_1
       2: idiv
       3: ireturn

  public static int b(int, int);
    Code:
       0: iload_0
       1: iload_1
       2: idiv
       3: istore_2
       4: iload_2
       5: ireturn

  public static int c(int, int);
    Code:
       0: iload_0
       1: iload_1
       2: idiv
       3: istore_2
       4: iload_2
       5: ireturn
}