Почему return не прерывает исполнение forEach у массива?

Question

[Tyler Lowell]

Тестировал авторизацию и наткнулся на такое чудо, что в функции находится массив и его перебор (forEach). В переборе предусмотрен поиск и в лог действительно пишется, что что-то найдено. Там стоит return, который, по-идее, должен завершить выполнение функции, но выполняется ещё и код после перебора.

Вот сама функция:

Код функции

async search(login) {
        fs.readFile(`${__dirname}/data/users.json`, (err, data) => {
            if(err) return false

            // Convert to JSON
            let parsed = JSON.parse(data.toString())

            // If user found, return functions. If not - true (data is correct, but user not exists)
            parsed.forEach((user, i) => {
                console.log('Parsing users. Index', i, 'Data:', user)
                console.log(`${user.auth.login} = ${login},`, user.auth.login === login)
                if(user.auth.login === login) {
                    console.log('All ok')
                    return {
                        validPassword: password => {
                            return user.auth.password === argon.hash(crypt.decrypt(password, 'smthkey').toString()).toString()
                        },
                        getProfile: () => {
                            return user.profile
                        },
                        getID: () => {
                            return user.id
                        }
                    }
                }
            })

            console.log('Nothing is found')
            return true
        })
    }

Вот, что уходит в лог:

Лог

TyLoDesu = TyLoDesu, true
All ok
Nothing is found

Comments

[Lynn «Кофеман»]

forEach работает нормально.
А что вы вообще ожидали от этого кода в котором плохо почти всё?

Answer 1

[Алексей Ярков]

Там стоит return, который, по-идее, должен завершить выполнение функции

С фига ли это?

Примечание: Не существует способа остановить или прервать цикл forEach() кроме как выбрасыванием исключения. Если вам необходимо такое поведение, метод forEach() неправильный выбор.

Answer 2

[Rsa97]

return срабатывает и возвращает управление из анонимной функции в основной цикл forEach, после чего forEach работает дальше.
Используйте .find

Answer 3

[scottparker]

это ретурны для стрелочных функций, например этой

password => {
  return user.auth.password===argon.hash(crypt.decrypt(password, 'smthkey').toString()).toString()
}

Answer 4

[Евгений]

Пачка замечаний по коду
1. Если уж используется async, то почему не использовать fs-promises ?
const fs = require('fs').promises
2. Зачем делать data.toString(), если можно указать кодировку и сразу вернется строка, а не буфер?
fs.readFile(file, 'utf-8')
3. всю эту простыню можно переписать на обычный find

Comments

[Tyler Lowell]

Можно про третий пункт подробнее?