0

Почему метод find возвращает только первый объект с массива?

Как правильно сделать чтобы проверяло целый массив на это условие?

Сейчас работает только первый логин и пароль, а когда вводишь второй - срабатывает второе условие, хотя должно сработать первое, потому что все верно.

const storedData = [
    {
        email: "ii@gmail.com",
        pw : 'qqq'     
    },

    {
        email : "pp@gmail.com",
        pw : "www"
    }
];

document.querySelector('#submit').addEventListener('click', e => {
    e.preventDefault();
    storedData.find(data => {
        console.log(data);
        if (data.email === document.querySelector('#login').value && data.pw === document.querySelector('#pw').value) {
            console.log("ok");
        }else {
            console.log("inccorect login or pw");
        }
        return true;
    });
});
 <form action="">
        login <input type="text" id="login">
        password <input type="password" id="pw">
        <input type="submit" value="Submit" id="submit">
      </form>    

1
  • можно использовать метод some
    – kertAW
    11 июл 2020 в 17:58

3 ответа 3

3

Вы используете метод find() не по назначению: он возвращает первый подходящий элемент (то есть элемент, для которого функция-параметр возвращает truthy значение) и дальше ничего не перебирает.

Если вам не нужно возвращаемое значение, а нужно просто перебрать весь массив с какими-то действиями, используйте forEach() или цикл for-of.

Или же, если при помощи find() нужно проверить, подходят ли данные хоть для одного элемента массива, возвращайте true только в случае совпадения и проверяйте возвращённое find() значение.

const storedData = [
    {
        email: "ii@gmail.com",
        pw : 'qqq'     
    },

    {
        email : "pp@gmail.com",
        pw : "www"
    }
];

document.querySelector('#submit').addEventListener('click', e => {
    e.preventDefault();
    const found = storedData.find(data => {
        console.log(data);
        return data.email === document.querySelector('#login').value &&
               data.pw === document.querySelector('#pw').value;
    });
    
    if (found) console.log("ok");
    else console.log("inccorect login or pw");
});

Можно ещё использовать метод some().

4
  • Хорошо метод forEach будет проверял каждый объект массива и возвращал Разные значения (и будет "ок" и затем "неверный логин и пароль"), вопрос как сделать если пароль верный остановить перебор и вывести сообщение "ок" в противном случае "неверный логин и пароль " 11 июл 2020 в 18:12
  • 1
    Запустите сначала проверку по массиву при помощи find() (проверьте, не равно ли значение undefined) или some() (проверьте, не равно ли значение false). Если таки равно, тогда уже выводите предупреждения о неправильности данных. Если элемент найден или значение не равно false, выводите 'ok'. 11 июл 2020 в 18:16
  • Понял, спасибо) 11 июл 2020 в 18:18
  • 1
    Добавил в ответ возможный вариант. 11 июл 2020 в 18:20
3

Метод find останавливается, в случае, если результатом выполнения callback функции является true.

Так как у тебя всегда возвращается true - то поиск заканчивается сразу.

1

Метод find возвращает первый элемент который удовлетворяет условию

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими пользовательским соглашением, политикой конфиденциальности и политикой о куки

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