Java布尔方法return语句问题：如何实现遇非倍数即返回false

你的代码存在两个核心问题：一是逻辑完全不符合需求，二是存在语法错误（return a;写在了方法体外）。

原代码的问题分析 #

1. 逻辑错误：当第一个元素是n的倍数时，你直接设置a=true并break终止循环，导致后续元素根本没被检查——哪怕后面有元素不是n的倍数，方法也会返回true，完全违背了「只要有一个元素不符合就返回false」的需求。
2. 语法错误：return a;语句写在了allMultipleOf方法的大括号外面，编译会直接报错。

正确实现方式 #

方式一：高效循环实现

这种写法会在发现第一个不符合条件的元素时立刻返回，无需遍历整个数组，性能更优：

public class test {
    public static void main(String[] args) {
        int number = 4;
        int[] intArray = {4, 8, 12, 16};
        System.out.println(allMultipleOf(intArray, number));
    }

    public static boolean allMultipleOf(int[] ary, int n) {
        // 处理数组为空或null的边界情况（可根据实际需求调整）
        if (ary == null || ary.length == 0) {
            return true;
        }
        // 遍历每个元素
        for (int num : ary) {
            // 只要有一个元素不是n的倍数，直接返回false
            if (num % n != 0) {
                return false;
            }
        }
        // 所有元素都符合条件，返回true
        return true;
    }
}

方式二：Java 8+ Stream简洁实现

如果使用Java 8及以上版本，可以用Stream API的allMatch方法，代码更简洁易读：

import java.util.Arrays;

public class test {
    public static void main(String[] args) {
        int number = 4;
        int[] intArray = {4, 8, 12, 16};
        System.out.println(allMultipleOf(intArray, number));
    }

    public static boolean allMultipleOf(int[] ary, int n) {
        if (ary == null) {
            return true;
        }
        // allMatch会判断所有元素是否都满足条件，正好匹配你的需求
        return Arrays.stream(ary).allMatch(num -> num % n == 0);
    }
}

这两种实现都严格遵循你的需求：仅当数组中所有元素都是指定整数的倍数时返回true，只要存在任意一个不符合的元素就返回false。

内容的提问来源于stack exchange，提问作者Sikefrikou