验证二维数组的元素

Question

我用于确保元素不超过 100 或小于 0 的代码是：

boolean test = true;
for (int i = 0; i < inArray.length; i++) {
    for (int j = 0; j < inArray[0].length; j++) {
        if (inArray[i][j] <= 1 && inArray[i][j] >= 100) {
            test = false;
        }
    }
}
return test;

无论输入是什么，答案总是true

我的输入 array 是

{12, 1331, 23, 5, 1, 3},
{22, 23, 231, 21312, 1, 3},
{23, 31, 343, 3432, 1, 3},
{42, 12, 5454, 1212, 3, 9}

如果我输入这个 no，测试应该 = false？

学院规则意味着我不能在子模块末尾以外的任何地方使用break; 或return。

Answer 1

这里有三个问题，其中两个很关键。首先，如果要遍历每个子数组，请测试 its 长度，而不是每次都测试 array[0] 的长度。其次，整数不能小于 2 并且 超过 99，因此您的条件 (inArray[i][j] <= 1 && inArray[i][j] >= 100) 永远不会触发。

但是，这里还有更多的作用：如果您的测试纯粹是为了查看您的二维数组是否有效，请不要做不必要的工作。一旦找到一个坏元素，您就可以从字面上停止。现在，我们可以通过在for 条件句中添加一个检查来做到这一点。例如。我们可以将您的变量重命名为valid（因为您应该根据变量所代表的内容命名变量），然后我们可以将您的外循环更新为for(int i=0; valid && i < arr.length; i++) { ... }，同时在内循环中使用相应的valid && ...，但我们是不会。这段代码“做一件事”（即它“测试任何数组数组的元素有效性”），“做一件事”的代码应该在它自己的方法中，并带有描述它做什么的方法名称。这使得“不做任何我们不需要做的工作”变得更加容易：当我们知道我们已经完成了足够多的工作来产生答案时，我们就退出函数。

那么让我们这样做吧：

// 1st fix: this code "does soemthing". Do that as a method.
public boolean testForValidContent(int[][] arr) {
  // We don't need a variable to track things: because we're
  // returning a value, we can simply return the right value
  // at the moment we return.

  for(int i = 0; i < arr.length; i++) {
    for(int j = 0; j < arr[i].length; j++) { // 2nd fix: check the right array's length
      if(arri][j] <= 1 || arr[i][j] >= 100) { // 3th fix: OR, not AND

        // At this point we know the array is not valid, so there
        // is no point in running even a single thing more:
        return false;

      }
    }
  }

  // We checked the entire array, and everything's valid:
  return true;
}

Answer 2

将您的 for 循环条件更新为 i < inArray.length && test 和 j < inArray[0].length && test，一旦发现无效号码，这将强制您的循环结束 和你的 if 条件 inArray[i][j] <= 1 || inArray[i][j] >= 100

boolean test = true

        for(int i = 0; i < inArray.length && test; i++){
            for(int j = 0; j < inArray[0].length && test; j++){ 
                if(inArray[i][j] <= 1 ||  inArray[i][j] >= 100){    
                    test = false;
                }
            }
        }

        return test;

您也可以跳过if 块，

boolean test = true

        for(int i = 0; i < inArray.length && test; i++){
            for(int j = 0; j < inArray[0].length && test; j++){ 
                test = (1 < inArray[i][j]  && inArray[i][j] < 100)  
            }
        }

        return test;