问题描述
我做了一个循环来检查所有inputArr
元素,但是它只检查第一个元素,如果它是true,则不检查其他元素并返回true。我需要检查所有它们,如果它们中的任何一个为假,则希望得到一条错误消息。我也尝试过forEach,但仍然无法正常工作
这是我的代码
btn.addEventListener('click',function () {
let isValid = checkIfValid([firstName,surname,email,mobileNumber])
let userInfo = generateInputs(firstName.value,surname.value,email.value,mobileNumber.value);
if (isValid) {
console.log('all checked')
} else {
console.log('you need to fill all fields')
}
})
function checkIfValid(inputArr) {
for (let i = 0; i < inputArr.length; i++) {
if (inputArr[i].value === '') {
return false
} else {
console.log(inputArr[i].value)
return true
}
}
}
解决方法
您的return true
语句停止for循环。
这将起作用,在第一个无效项目上返回false,否则返回true:
function checkIfValid(inputArr) {
for (let i = 0; i < inputArr.length; i++) {
if (inputArr[i].value === '') {
return false
} else {
console.log(inputArr[i].value)
}
}
return true
}
现代替代方案:
您可以使用Array.prototype.every()
来简化代码:
function checkIfValid(inputArr) {
return inputArr.every(el => el.value !== '');
}