Question

给出一个整数数组，如果值3恰好出现在数组中3次，并且没有3相邻，则返回true。

我只是一个初学者，在编码技巧课程上遇到一些麻烦。逻辑似乎很好。我已经向“橡皮鸭”解释了1000次，但我发现它没有问题。除“其他测试”选项卡外，所有的codingbat测试均按预期运行，我无法看到数组中的特定数字，也无法与代码进行比较。我真的对此感到困惑，希望您能帮助我！

public boolean haveThree(int[] a) {

    int count = 0;           //to count the appearences of 3
    boolean doLado = false;   //to check if a 3 is next to another 3

    if(a[0] == 3)    // check if first index is 3
        count++;      // add one if it is

    for(int i=1; i<a.length ; i++) { //loop starting at 1 to check rest of array

        if(a[i] == 3) {     // check if i is 3
            if(a[i-1] == a[i]) // if i its 3, check if the previous index was also 3
                return false;   // if it was indeed {..,3,3,..} return false
            else
                count++;        // else add 1 to the counter
        }
    }

    if(count == 3) //if counter of 3s equals 3 return true
        return true;

    return false; //else return false
}


tests                                  Expected  Run        
haveThree([3, 1, 3, 1, 3])----------- → true    true    OK  
haveThree([3, 1, 3, 3])---------------→ false   false   OK  
haveThree([3, 4, 3, 3, 4])------------→ false   false   OK  
haveThree([1, 3, 1, 3, 1, 2])---------→ false   false   OK  
haveThree([1, 3, 1, 3, 1, 3])---------→ true    true    OK  
haveThree([1, 3, 3, 1, 3])------------→ false   false   OK  
haveThree([1, 3, 1, 3, 1, 3, 4, 3])---→ false   false   OK  
haveThree([3, 4, 3, 4, 3, 4, 4])----- → true    true    OK  
haveThree([3, 3, 3])------------------→ false   false   OK  
haveThree([1, 3])---------------------→ false   false   OK  
haveThree([3])------------------------→ false   false   OK  
haveThree([1])------------------------→ false   false   OK  

other tests-----------------------------X

Answer 1

您不处理null，也不使用doLado；另外，您无需在最后使用if来测试count == 3。我会将其简化为类似

public boolean haveThree(int[] a) {
    if (a == null || a.length < 3) {
        return false;
    }
    int count = 0;
    for (int i = 0; i < a.length; i++) {
        if (a[i] == 3) {
            if (i > 0 && a[i - 1] == 3) {
                return false;
            }
            count++;
        }
    }
    return count == 3;
}

Answer 2

您的代码中缺少

空检查来检查数组 a 是否为空。

如果单独添加 null 检查，您的代码将正常工作。

我似乎在代码中找不到任何逻辑错误，但仍然无法按预期运行

2 个答案: