Как корректно получить значения из вложенной функции при работе с mysql2?

Question

[Олег]

Видимо под трехсотую строку кода в конец отупел... Стыдно спрашивать, но выхода не вижу. Изчитал и изучил уже по моему все что только можно. Не выходит "каменный цветок"...

var temp; // Объявил глобальную переменную

function getOnline(){                   //функция возврата int значения из базы данных.
    const sql = `SELECT * FROM Online`;
    connection.query(sql, function(err, results) {
        if(err) console.log(err);
        for(let i=0; i < results.length; i++){       // Перебираем полученный массив и забираем значение.
           temp=results[i].online;    // Получаем корректный результат
        }
    });
}
console.log(temp);    // 'undefined' не используемая переменная (как выдает IDE), см. скрин.

Блин как?
Вангую может быть дело в том, что ответ приходит чуть позже чем присваивается переменной. Либо дело во вложенной функции.
Во всех примерах и мануалах инета все описание заканчивается фразой...."И дальше работаем как с обычным массивом". А вот как теперь получить результат работы с "обычным массивом" ?


Заранее ОГРОМНОЕ СПАСИБО!
p.s. Может чего толкового подскажите где посмотреть (почитать, послушать), чтобы в будущем не натыкаться на подобные проблемы )

Answer 1

[Алексей Уколов]

"Работать как с обычным массивом" нужно внутри колбэка - асинхронный код синхронным не сделать. Есть промисы и async/await, но они принципиально сути не меняют.

Comments

[Олег]

Спасибо.
Проблема ясна. Решение нет. Как мне вернуть данные из базы в код для дальнейшей обработки? )

[Алексей Уколов]

Никак. Но вы можете вернуть промис, а дальше работать с ним. Тот же самый колбэк, только код становится более линейным. Поверх промиса можно накрутить async/await.

[Олег]

Алексей Уколов, https://qna.habr.com/q/222364?_ga=2.205444112.1676...

Пишут можно... или я не понял что можно )
Читаю

[Алексей Уколов]

Пишут можно

Где же там такое пишут?

Answer 2

[legacy_js]

Если актуален еще вопрос, можете воспользоваться в mysql2 методом .promise()
Пример Вашего кода

//  создаем пул
    const pool = mysql.createPool({
        connectionLimit : 100,
        host: "localhost",
        user: "root",
        database: "test",
        password: ""
    });

    let arr;

    const promisePool  = pool.promise(); 

    async function getOnline() {
        const sql = 'SELECT * FROM Online';

        const [rows, fields] = await promisePool.query(sql);

        for(let i=0; i < rows.length; i++){
           arr = rows[i].online; 
        }

        return arr;
    }

    getOnline().then(r => console.log(r)) // Вам не нужно выводить глобальную переменную, ответ выведется тут

Comments

[Олег]

На самом деле я ее изначально и использую )
сейчас проверю ваш вариант, возможно он и подойдет

Answer 3

[Александр]

вам нужно прочитать про области видимости переменных.
объявленная вами переменная temp, для console.log(temp) - undefined, т.к её значение изменяется внутри функции и доступно лишь там.
если вы хотите вывести изменённую temp, вы должны делать это в пределах функции:

var temp; 

function getOnline(){ 
    const sql = `SELECT * FROM Online`;
    connection.query(sql, function(err, results) {
        if(err) console.log(err);
        for(let i=0; i < results.length; i++){
           temp=results[i].online; 
        }
      console.log(temp) // здесь temp = results[i].online
    })
}
console.log(temp) // undefined, здесь temp не равняется ничему
getOnline();

Comments

[Олег]

Это понятно. Я понимаю, что дело в асинхронных запросах.
Для меня открытие, что в ЯваСкрипт, работа с данными из БД это такая проблема )

Вроде нашел описанное решение, но по факту, оно все равно не передает значение в глобальную переменную.

var async = require('async');
var items_list, users_list;
async.parallel([
    function(callback) { // делаем первый запрос к базе
      db.model('items').find({}, function (err, data) {
        items_list = data;
        callback(); // данные получены, возвращаемся
      });
    },
    function(callback) { // параллельно делаем второй запрос к базе
      db.model('users').find({}, function (err, data) {
        users_list = data;
        callback(); // данные получены, возвращаемся
      });
    }
],
function() {
  // когда оба запроса уже завершены, то мы попадаем сюда
  console.dir({ items: items_list, users: users_list });
});

Но за-то передает ее в другую функцию.
А у меня бот... который забирает данные из глобальных переменных. И мне нужно их менять по ходу работы.... и вот как это сделать на JS пока для меня загадка )

p.s. Хоть возвращайся к истокам и работай с файлами JSON. Но это ж дурдом, если есть база. Я ее заполняю, меняю ... а забрать не могу )))))

[Александр]

Олег, асинхронные/синхронные функции - всё это не имеет значения, т.к вы всё равно не сможете считывать изменение переменной в функции, за её пределами - с этим нужно смириться.
единственный вариант - делать всю работу в самих функциях:

connection.query(sql, function(err, results) {
      // весь ваш код
})

[Алексей Уколов]

не сможете изменять глобальные переменные внутри функций

Изменять-то можно, с этим никаких проблем. Проблема с тем, что это изменение произойдёт уже после того, как значение переменной будет считано.

[Олег]

Александр, Вот это новость....
Т.е. городить ?

connection.query(sql, function(err, results) {
      // весь ваш код
connection.query(sql, function(err, results) {
      // весь ваш код
connection.query(sql, function(err, results) {
      // весь ваш код
})
})
})

Спорить не стану, но выглядит сомнительно. При большом кол-ве запросов к БД.

[Александр]

Алексей Уколов, упс, опечатка.

[Александр]

Олег, да, именно так - других вариантов нету.

При большом кол-ве запросов к БД.

это не будет сильно влиять на скорость.

[Алексей Уколов]

Спорить не стану, но выглядит сомнительно.

Добро пожаловать в callback hell.
Вообще, вопрос далеко не новый.

[Олег]

Алексей Уколов, ну его в трынду ) Ща на JSON все сделаю. Буду использовать его для передачи параметров в/из переменных и БД. Благо там по сути только одна цифра скачет.
Но ваше решение учту в любом случае.

Спасибо.