Почему функция в node.js возвращает undefined?

Question

[Ксения]

только начинаю изучать node и асинхронность, пишу функцию, которая рекурсивно поищет файл package.json и вернет путь, где она его найдет.

function getRoot(startPath) {
        let isRoot = false;
        if (isRoot) {
            return startPath
        }

        fs.access(`${startPath}/package.json`, fs.constants.F_OK, (err) => {
            if (!err) {
                isRoot = true;
            } else {
                const arr = startPath.split('/');
                arr.pop();
                const newRoot = arr.join('/');
                return getRoot(newRoot);
            }
        })
    }
    console.log(getRoot(process.env.PWD));

Мне возвращается undefined, хотя если поконсолись, то функция вызывается каждый раз с ожидаемым startPath и останавливается как надо. Не пойму, где что не так?

Answer 1

[Stockholm Syndrome]

попробуйте так:

async function getRoot(startPath) {
    return new Promise(async (resolve, reject) => {
        fs.access(`${startPath}/package.json`, fs.constants.F_OK, (err) => {
            if (!err) {
                resolve(startPath);
            } else {
                const arr = startPath.split('/');
                arr.pop();
                const newRoot = arr.join('/');
                resolve(await (getRoot(newRoot)));
            }
        });
    });
}
getRoot(process.env.PWD).then((data) => console.log(data));

Answer 2

[Pavel Shvedov]

Ты ничего не возвращаешь из своей функции, поэтому она возвращает undefined, тот return где рекурсивно вызывается функция, относится к колбэку, который передается в fs.access, как исправить? Нельзя просто так синхронно вернуть данные из асинхронной функции, можно вернуть промис, и подписаться на его resolve, или пробросить колбэку внутрь, который нужно будет дергать по окончанию работы

Comments

[Ксения]

то есть мне из моей функции возвращать промис, ли из колбэка возвращать промис?

[Pavel Shvedov]

Примерный код двух вариантов: с промисом, и с колбэком.

const fs = require('fs');

const promiseGetRoot = (startPath) => {
    return new Promise((resolve) => {
        const f = (path, resolve) => {
            fs.access(`${path}/package.json`, fs.constants.F_OK, err => {
                if (!err) {
                    resolve(path);
                } else {
                    const newPath = path.split("/").slice(0, -1).join("/");
                    f(newPath, resolve);
                }
            });
        };
        f(startPath, resolve);
    });
};

const callbackGetRoot = (path, callback) => {
    fs.access(`${path}/package.json`, fs.constants.F_OK, err => {
        if (!err) {
            callback(path);
        } else {
            const newPath = path.split("/").slice(0, -1).join("/");
            callbackGetRoot(newPath, callback);
        }
    })
};


promiseGetRoot(process.env.PWD).then(root => console.log(root));
callbackGetRoot(process.env.PWD, (root) => console.log(root));

[Ксения]

Pavel Shvedov, теперь понятно, спасибо!

Answer 3

[RidgeA]

callback

Comments

[Ксения]

что с ним не так, где?)