关于 Java switch 语句 - 在每种情况下使用 return 并省略中断

Question

鉴于这种方法，这是否代表了一些令人震惊的文体或语义失礼：

private double translateSlider(int sliderVal) {
    switch (sliderVal) {
        case 0:
            return 1.0;
        case 1:
            return .9;
        case 2:
            return .8;
        case 3:
            return .7;
        case 4:
            return .6;
        default:
            return 1.0;
    }
}  

显然不符合 Java 教程here。

但是，它清晰、简洁，到目前为止已经完全满足了我的需求。是否有一个令人信服的、务实的理由来创建一个局部变量，在每个案例中为其分配一个值，为每个案例添加一个中断并在方法结束时返回值？

Answer 1

虽然这个问题已经足够老了，但现在仍然可以参考。

从语义上讲，这正是 Java 12 引入的 (https://openjdk.java.net/jeps/325)，因此，在这个简单的示例中，只要我看不到任何问题或缺点。

Answer 2

人类逻辑到计算机生成字节码的最佳情况是使用如下代码：

private double translateSlider(int sliderVal) {
  float retval = 1.0;

  switch (sliderVal) {
    case 1: retval = 0.9; break;
    case 2: retval = 0.8; break;
    case 3: retval = 0.7; break;
    case 4: retval = 0.6; break;
    case 0:
    default: break;
  }
  return retval;
}

因此消除了方法的多个出口并在逻辑上使用该语言。 （即当sliderVal 是1-4 的整数范围时，改变float 值，否则如果sliderVal 为0 和所有其他值，retval 保持相同的float 值1.0）

然而，像这样的情况，sliderVal 的每个整数值都是 (n-(n/10)) 真的可以只做一个 lambda 并获得更快的结果：

private double translateSlider = (int sliderVal) -> (1.0-(siderVal/10));

编辑： 模数为 4 可能是为了保持逻辑（即(n-(n/10))%4)）

Answer 3

从人类智能的角度来看，您的代码很好。从静态代码分析工具来看，有多个返回，这使得调试变得更加困难。例如，您不能在返回之前立即设置一个且唯一的断点。

此外，您不会在专业应用程序中对 4 个滑块步骤进行硬编码。 通过使用 max - min 等计算值，或者 在数组中查找它们：

public static final double[] SLIDER_VALUES = {1.0, 0.9, 0.8, 0.7, 0.6};
public static final double SLIDER_DEFAULT = 1.0;


private double translateSlider(int sliderValue) {
  double result = SLIDER_DEFAULT;
  if (sliderValue >= 0 && sliderValue < SLIDER_VALUES.length) {
      ret = SLIDER_VALUES[sliderValue];
  }

  return result;
}

Answer 4

我认为你写的很好。我也没有看到有多个返回语句的任何可读性问题。

当我知道要返回时，我总是更喜欢从代码中的点返回，这将避免在返回下方运行逻辑。

可以有一个参数用于调试和日志记录的单个返回点。 但是，在您的代码中，如果我们使用它，则不存在调试和日志记录的问题。就像你写的那样简单易读。

Answer 5

将值分配给局部变量，然后在最后返回该值被认为是一种很好的做法。具有multiple exits are harder to debug 的方法可能难以阅读。

也就是说，这是这个范式剩下的唯一优点。它起源于只有低级程序语言出现时。在那个时候它更有意义。

当我们讨论这个主题时，您必须check this out。读起来很有趣。

Answer 6

为什么不直接

private double translateSlider(int sliderval) {
if(sliderval > 4 || sliderval < 0)
    return 1.0d;
return (1.0d - ((double)sliderval/10.0d));
}

还是类似的？

Answer 7

是的，这很好。 教程并不总是简洁而整洁。 不仅如此，创建局部变量既浪费空间又低效

Answer 8

如果您要拥有一个只运行开关然后返回一些值的方法，那么请确保这种方法有效。但是，如果您想在方法中使用其他内容进行切换，则不能使用 return，否则方法中的其余代码将不会执行。请注意教程中的代码后如何打印？你的将无法做到这一点。

Answer 9

我建议你不要使用文字。

除了样式本身看起来不错。

Answer 10

不，你所拥有的一切都很好。您也可以将其作为公式 (sliderVal < 5 ? (1.0 - 0.1 * sliderVal) : 1.0) 或使用 Map<Integer,Double>，但您所拥有的就可以了。