Функция возвращает undefined, как это решить?

Question

[AndreyKawilov]

index.js

var mysql = require('mysql');
const SQL = require('./functions/SQL.js')
var request = "SELECT * FROM users"
console.log(
    SQL.GetFromSQL(pool, request) //возвращает undefined
)

SQL.js

module.exports = { GetFromSQL }

function GetFromSQL (pool, request) {
    pool.getConnection(function (err, connection) {
        if (err) throw err;
        connection.query(request, function (error, results, fields) {
            connection.release();
            if (error) throw error;
            return results
        });
    });
}

Comments

[Ярослав Иванов]

Потому что GetFromSQL ничего не возвращает

[AndreyKawilov]

Ярослав Иванов, Как сделать что бы вернул? Просто уже много вариантов попробовал ни один не работает.

[Ярослав Иванов]

AndreyKawilov, через callback

module.exports = { GetFromSQL }

function GetFromSQL (pool, request, callback) {
    pool.getConnection(function (err, connection) {
        if (err) throw err;
        connection.query(request, function (error, results, fields) {
            connection.release();
            if (error) throw error;
            callback(results)
        });
    });
}

SQL.GetFromSQL(pool, request, function(results) {
 // lol kek
})

P.S. Называйте методы без тавтологии. GetFromSQL уже понятно что SQL если вызывается у SQL

SQL.get()

[AndreyKawilov]

Ярослав Иванов, это всё хорошо, я так даже делал. У меня возникала проблема в том что мне надо присвоить переменной значений из этой функции. Если я пишу

var answer
SQL.GetFromSQL(pool, request, function(results) {
 answer = results
    console.log(answer)
})

то всё хорошо, а если я попробую написать так

var answer
SQL.GetFromSQL(pool, request, function(results) {
 answer = results
})
console.log(answer)

то получаю undefined

[Ярослав Иванов]

AndreyKawilov, логично. Потому что callback это ахинхронная функция. Добро пожаловать в мир асхинхроных функций.

[AndreyKawilov]

Ярослав Иванов, Да уж весело, насчёт названия методов учту, спасибо.

Answer 1

[Ярослав Иванов]

module.exports = { GetFromSQL }

function GetFromSQL (pool, request, callback) {
    pool.getConnection(function (err, connection) {
        if (err) throw err;
        connection.query(request, function (error, results, fields) {
            connection.release();
            if (error) throw error;
            callback(results)
        });
    });
}

SQL.GetFromSQL(pool, request, function(results) {
 // lol kek
})

P.S. Называйте методы без тавтологии. GetFromSQL уже понятно что SQL если вызывается у SQL

SQL.get()

Comments

[Сергей Шиловский]

Придираться к названиям не очень профессионально, если это не var1, var2, var3, хотя согласен, что иногда хочется.

[Ярослав Иванов]

Сергей Шиловский, Это рекомендация а не придирка с целью докапатся. И делать рекомендации - это профессионально если ваша цель научить человека делать лучше.