与if语句相比,如何正确构造Java的三元运算符?

时间:2019-04-03 04:22:34

标签: java ternary-operator ternary

我有一个简单的问题,关于同一代码的两个版本,一个使用三元运算符,一个使用if语句,以及为什么它们的答案不同。

有关该问题的背景信息。我正在尝试轻松地对整数数组进行排序并获取数组的范围(最大和最小)

给出的数组:[6,2,3,8]

如果语句版本:

public int range(int[] num_list) {
    int[] min_max = new int[2];
    for (int i : statues) {
         if (min_max[0] == 0) {
            min_max[0] = i;
        } else {
            if (i < min_max[0]) min_max[0] = i;
        }
         if (min_max[1] == 0) {
            min_max[1] = i;
        } else {
            if (i > min_max[1]) min_max[1] = i;
        }
  System.out.println(String.valueOf(min_max[0] + " | " + min_max[1]));
    }
}

控制台日志:

6 | 6
2 | 6
2 | 6
2 | 8

三元运算符版本:

public int range(int[] num_list) {
    int[] min_max = new int[2];
    for (int i : statues) {
        min_max[0] = min_max[0] == 0 ? i 
               : (min_max[0] = i < min_max[0] ? i : i);
        min_max[1] = min_max[1] == 0 ? i 
               : (min_max[1] = i > min_max[1] ? i : i);
  System.out.println(String.valueOf(min_max[0] + " | " + min_max[1]));
    }
}

控制台日志:

6 | 6
2 | 2
3 | 3
8 | 8

为什么三元运算符版本不正确?据我所知,它应该是if语句版本的三元副本吗?

2 个答案:

答案 0 :(得分:1)

可以使用if条件将整个内容重写为:

public int range(int[] num_list) {
    int[] min_max = new int[2];
    for (int i : statues) {
        if (min_max[0] == 0 || i < min_max[0]) {
            min_max[0] = i;
        }

        if (min_max[1] == 0 || i > min_max[1]) {
            min_max[1] = i;
        }

        System.out.println(String.valueOf(min_max[0] + " | " + min_max[1]));
    }
}

如果您真的要使用三元表达式:

public int range(int[] num_list) {
    int[] min_max = new int[2];
    for (int i : statues) {
        min_max[0] = (min_max[0] == 0 || i < min_max[0]) ? i : min_max[0];
        min_max[1] = (min_max[1] == 0 || i > min_max[1]) ? i : min_max[1];

        System.out.println(String.valueOf(min_max[0] + " | " + min_max[1]));
    }
}

答案 1 :(得分:1)

您有两个错误。这个:

    if (min_max[0] == 0) {
        min_max[0] = i;
    } else {
        if (i < min_max[0]) min_max[0] = i;
    }

等效于此:

    min_max[0] = min_max[0] == 0 ? i :
        (i < min_max[0] ? i : min_max[0]);

注意,在三元运算符中,“ else”必须始终存在。因此,您需要在其中添加一些内容-在这种情况下,分配旧值类似于不执行任何操作。

第二个if语句的故事类似。