帶數組參數的布爾方法

Question

我花了很長時間才能弄清楚這些問題，但是我被困在這里。 我需要編寫一個布爾方法，該方法將數組作為參數，如果數組中的數字降序，則應返回true。 卜，只要我嘗試我有相同的值或錯誤。 這是我的代碼：

public class Question1c{
    public static void main (String[] args){
        int[] arr = {1, 9, 3, 4, 5, 6};
        boolean product = isDecreasing(arr);
        System.out.println(product);
    }

    public static boolean isDecreasing (int[] numbers){
        int first = numbers[0];

        for (int i : numbers){

            if(first <= i){
                first = i;
                return true;
            }

            //else{
            //  return false;
            //}
        }return false;
    }
}

Answer 1

您的代碼中的問題是，直到遍歷整個數組之后，您才能返回true 。 但是，一旦檢測到“倒置”，即可能出現false ，即跟在您之前看到的數字之后的數字大於前一個數字的情況。

您合理地接近可行的解決方案-您需要刪除return true ，取消注釋else ，然后將最終return false更改為return true 。

為了使代碼更具可讀性，請first將其重命名為prior 。 還可以考慮將for循環的“ foreach”版本更改for跳過數組初始元素的常規for循環。 這樣，您就可以檢測到當前檢測到的遞減順序，而不是不遞增的順序。

Answer 2

不確定我是否了解，但這不能解決您的問題嗎？

public class Question1c{
    public static void main (String[] args){
        int[] arr = {1, 9, 3, 4, 5, 6};
        boolean product = isDecreasing(arr);
        System.out.println(product);
    }

    public static boolean isDecreasing (int[] numbers){
        for (int i = 0; i < numbers.Length; i++){
            if (i == 0)
                continue;
            if (numbers[i - 1] >= numbers[i])
                return false;
        }
        return true;
    }
}

您實質上只是在檢查數組中的前一項是否大於或等於數組中的當前項，不是嗎？

Answer 3

首先使用numbers[0]初始化，這會導致幾個問題：

• 空數組將引發IndexOutOfBoundsException
• 第一張支票自動通過（ numbers[0] <= numbers[0] ）

您最好檢查numbers的長度是否為0，並使用“正常”循環（使用索引）。

您的返回值也應該是負數。

Answer 4

這可能會解決您的問題。

public class Question{
public static void main (String[] args){
    int[] arr = {1, 9, 3, 4, 5, 6};
    boolean product = isDecreasing(arr);
    System.out.println(product);

}

public static boolean isDecreasing (int[] numbers){
    int first = numbers[0];
    for (int i = 1; i < numbers.length ; i++) {

        if (first <= numbers[i]) {
            return false;
        } 
        first = numbers[i];
    }
    return true;
}

}