有没有办法缩短这段代码？ 也许是三元表达式？

Question

    if (i <= 33) {
            variable = x;
        } else if (i > 33 && i <= 66) {
            variable = y;
        } else {
            variable = z;
        }

有什么办法可以缩短时间？ 在这种情况下，我找不到如何使用三元运算符。

Answer 1

如果要使用一些疯狂的嵌套三元组：

int variable = i <= 33 ? x : i > 33 && i <= 66 ? y : z

...但是不要这样做。 而是使您的代码具有可读性和可维护性！

另外，正如@EJP 在他的回答中提到的（您应该对此表示赞成）， i > 33条件是多余的，因此您可以将其缩短为：

int variable = i <= 33 ? x : i <= 66 ? y : z

我只是直接从if / else-s中获取条款。

Answer 2

if (i <= 33) {
        variable = x;

在这一点上， i <= 33.

    } else if (i > 33 && i <= 66) {

在这一点上，我们位于“ else”中，根据构造， i > 33 ，因此您无需再次对其进行测试。

        variable = y;
    } else {
        variable = z;
    }

因此，您可以将其减少为：

if (i <= 33) {
        variable = x;
    } else if (i <= 66) {
        variable = y;
    } else {
        variable = z;
    }

Answer 3

    if (i <= 33) variable = x;
    else if (i <= 66) variable = y;
    else variable = z;

由于每个子句只有一个语句，因此可以省略花括号并将这些语句置于该子句的同一行。

Answer 4

要授予可理解性，您可以先将其提取到方法中，然后：*使用第一个答案*如果您希望它更易于理解，则可以执行此操作

private Object valuePerState(int i){
    if (i<33) return x;
    if (i>66) return y;
    return z;
}

顺便说一句，缩短您的问题的第一步是对您做过的33项测试做完测试

Answer 5

您可以通过反转比较方向来简化操作而无需三元运算符：

if (i > 66) 
{
    variable = z;
}
else if (i > 33) 
{
    variable = y;
} 
else 
{
    variable = x;
}

但正如其他人指出的那样，您可以简化比较条件而无需反转方向。

Answer 6

您可以像这样使用ternery运算符：

if (i <= 33) {
        variable = x;
    } else {
        variable = (i <= 66) ? y : z;
    }

但是我认为这比您拥有的更糟糕。

请注意，可以将i > 33删除，因为else已暗示它

Answer 7

如果将x ， y和z存储在数组中，则可以使用除法访问数组的不同元素，这会更快，尤其是在您知道条件受0和100限制的情况下。

index = Math.min(Math.max((i-1) / 33, 0), 2);
variable = x[index];

如果i不能小于0或大于100，则变为：

index = (i-1) / 33;
variable = x[index];

Answer 8

我似乎晚会晚了，但是无论如何。 这是我使用链式三元运算符的方式。 我认为这是相当可读的。

variable = i <= 33 ? x
         : i <= 66 ? y
         : z ;

由于运算符和空格的位置，此方法也适用于版本控制：您在中间添加的任何新条件只是diff中的新行，而其他条件则保持不变。 另外，如果有那么多运算符使您感到紧张，您当然可以在条件周围加上括号。

Answer 9

由于使用的是else子句，因此不必对33进行两次检查：

if (i <= 33) {
    variable = x;
} else if (i <= 66) {
    variable = y;
} else {
    variable = z;
}