Funkcja Javascript zwraca "undefined" mimo tego, że może zwrócić tylko "true/false".

Question

Witajcie, mam problem z funkcją (isAdmin) której zadaniem jest zwracanie wartości true albo false, jednak zwraca ona zawsze wartość "undefined". W czym może tkwić problem?

function isAdmin(req, res) {
    dbConnection.getConnection((err, connection) => {
        connection.query(`SELECT id FROM users WHERE id = '${req.session.userId}' AND admin = 1`, (err, rows) => {
            connection.release();
            if (err) throw err;
            if (rows.length == 1) return true;
            else return false;
        });
    });
}

exports.getDbTable = (req, res, next) => {
    let table = req.params.table;
    console.log(isAdmin(req, res)); // undefined
    if (isAdmin(req, res) != true && table == "users") return res.status(401).json({error: 6, errorMsg: "Access Denied"});
    dbConnection.getConnection((err, connection) => {
        connection.query(`SELECT * FROM ${table}`, function (err, rows) {
            if (err) throw err;
            else {
                console.log(rows);
            }
            connection.release();
        });
    });
}

 

Comments

Z góry zaznaczę, że nie znam się na programowaniu asynchronicznym w JavaScripcie.

Funkcja isAdmin nie zwraca żadnej wartości. Wewnątrz niej definiujesz funkcję asynchroniczną która tworzy kolejną funkcję asynchroniczną. W żaden sposób nie przechwytujesz wyniku wewnętrznej funkcji asynchronicznej.

Answer 1

Rozwiązane poprzez zastosowanie obietnic:

function isAdmin(req, res) {
    return new Promise(function (resolve, reject) {
        dbConnection.getConnection((err, connection) => {
            connection.query(`SELECT id FROM users WHERE id = '${req.session.userId}' AND admin = 1`, (err, rows) => {
                connection.release();
                if (err) throw err;
                if (rows.length == 1) resolve();
                else reject();
            });
        });
    });
    
}

exports.getDbTable = (req, res, next) => {
    let table = req.params.table;
    if (table == "users") {
        isAdmin(req, res)
        .catch(function () {
            return res.status(401).json({error: 6, errorMsg: "Access Denied"});
        });
    }
    dbConnection.getConnection((err, connection) => {
        connection.query(`SELECT * FROM ${table}`, function (err, rows) {
            if (err) throw err;
            else {
                console.log(rows);
            }
            connection.release();
        });
    });
}

 

Comments

Możesz też skorzystać z metody promisify z modułu utils, żeby owinąć metodę dbConnection.getConnection w promisa i zwrócić go bezpośrednio z funkcji isAdmin - wtedy nie będziesz musiał tworzyć promisa samodzielnie.