0

Есть функция:

function checkAnswer() {
  const x0 = document.querySelector('#x0_answer').value;
  if (x0 !== '') {
    var arr = x0.split('');
    arr.forEach(n => {
      if (!Number.isInteger(parseInt(n)) && n.toString() !== '.' && n.toString() !== ',') {
        alert('bad');
        return;
      }
    });
  }
}

После return не выходит из цикла и функции (несколько alert на экране). Что я делаю не так?

1
  • 1
    Данный цикл так не работает.. обычный for...in и for...of - да, forEach перебирает всё. 19 мая 2022 в 16:18

4 ответа 4

3

Если нужно проверить на то, что все элементы соответствуют условию, то проще воспользоваться методом every. Пример:

console.log([1,2,3].every( el => el !== 1 ))

1

arr.forEach(n => {

Выходит из lambda (анонимной) функции.. )))


function checkAnswer() {
    const x0 = document.querySelector('#x0_answer').value;

    var BreakException = {};

    try {

        if (x0 !== '') {
            var arr = x0.split('');

            arr.forEach(n => {
                if (!Number.isInteger(parseInt(n)) && n.toString() !== '.' && n.toString() !== ',') {
                    alert('bad');
                    throw BreakException;
                }
            });

        }

    } catch (err) {
        if (err !== BreakException) throw err;
    }
}

Или как альтернатива использовать

for (let n of arr) {
   if (isBad(n)) return;
}
1

Используйте for вместо forEach. В данном случае return выходит из forEach.

function checkAnswer() {
  const x0 = document.querySelector('#x0_answer').value;
  if (x0 !== '') {
    var arr = x0.split('');
    for(const n of arr) {
      if (!Number.isInteger(parseInt(n)) && n.toString() !== '.' && n.toString() !== ',') {
        alert('bad');
        return;
      }
    }
  }
}
1

используйте вместо forech метод - some. Она проверяет, чтоб хотяб раз выполнилось условие, короче и чище код:

function checkAnswer() {
    const x0 = document.querySelector('#x0_answer').value;

    if (x0 !== '') {
        var arr = x0.split('');

        const bad = arr.some(n => !Number.isInteger(parseInt(n)) && n.toString() !== '.' && n.toString() !== ',');

    if (bad) {
         alert('bad');
    }
  }
}

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.