数组元素全相等校验函数异常：无论输入何数组均返回true求助

嘿，我一眼就瞅到你这个函数的问题啦——不管传什么数组它都返回true，根源全在循环里的返回逻辑上！

错误原因 #

你看看这段循环代码：

for (var i = 0; i < uni.length; i++ ) {
    if(uni[i] !== base){
        return false;
    } else{return true;}
}

第一次循环i=0的时候，uni[i]就是你定义的base（毕竟base = uni[0]），这时候直接就走进else分支返回true了，后面的元素根本没机会被检查！哪怕数组后面跟着完全不一样的元素，函数在第一次迭代就直接结束，自然永远返回true。

修复后的代码 #

我们只需要调整返回true的时机，把它移到循环结束之后，确保所有元素都被校验过：

function isUniform(uni) {
    // 处理空数组的边界情况（可根据需求调整，比如返回false）
    if (uni.length === 0) {
        return true;
    }
    
    var base = uni[0];
    // 从第二个元素开始遍历，跳过和基准相同的第一个元素
    for (var i = 1; i < uni.length; i++ ) {
        if(uni[i] !== base){
            return false;
        }
    }
    
    // 所有元素都和基准相等，才返回true
    return true;
}

额外优化点 #

• 循环从i=1开始，避免了对第一个元素的冗余判断（它本身就是基准）
• 增加了空数组的处理，防止传入空数组时uni[0]出现undefined的问题（你可以根据实际需求决定空数组返回true还是false）

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