Как вытащить переменную за рамки функции в JavaScript?

Question

[Roman Hinex]

Как можно вытащить данные за рамки вызываемой функции (коллбека)? Или как можно вытащить данные из модели в Mongoose и использовать значения далее в коде?

var items_list;
db.model('items').find({}, function (err, data) {  items_list = data });
console.log(items_list); // undefined

Answer 1

[Тимур Шемсединов]

Как раз этого делать не нужно. Вместо этого нужно понять идею асинхронного программирования. Все колбэки исполняются не по месту своего объявления, а тогда, когда к ним приходят данные, поэтому в Вашем примере кода сначала происходит console.log(items_list); а уже потом items_list = data И это хорошо, потому, что не происходит ожидания исполнения коллбеков, не происходит блокирования потока исполнения. Всю синхронную логику обработки полученных в коллбеке данных Вы можете написать прямо в коллбеке, а если нужно реализовать последовательный или параллельный вызов нескольких ассинхронных запросов (все, что связано с вводом/выводом, доступом к базе и файлам, например), то это можно сделать разными способами, самые популярные из них, это библиотека async и промисы. Я использую https://github.com/caolan/async Например:

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 });
});

Comments

[Roman Hinex]

Окей, суть задачи такая, есть либа в которой есть несколько функций, в одной из них нужно забрать значение из базы данных и вернуть. При этом понятное дело что либа будет использоваться в нескольких местах. Как будет лучше это сделать?

[Roman Hinex]

Причем вернуть значение в вызываемую функцию.

[Тимур Шемсединов]

Эта библиотека не сможет вернуть значение, получаемое в асинхронном режиме из базы через return. Вам нужно перестроиться и начать использовать коллбеки для возврата результатов. Т.е. самому начать писать асинхронные либы. Одним из параметров в функции будет функция, куда Вы и передадите первым параметром err, а вторым - значение. Если ошибки нет, то вернете null в err.

[Roman Hinex]

Тимур Шемсединов: плохо дело) тогда можете пнуть меня в таком вопросе, есть контроллер который во вьюхи возвращает айтемы (айтемы выводит хорошо, все гуд). Но к айтемам нужно выводить имя пользователя который и выложил айтем из отдельной таблицы. Как я могу это сделать?

[Тимур Шемсединов]

Какая вьюха, какой контроллер, Вы что, проектируете пользовательский интерфейс?

[Тимур Шемсединов]

Используете Mongo, откуда взялись таблицы? Тут коллекции документов, данные нужно хранить в ненормализованном виде, без реляционной декомпозиции, т.е. если у айтема есть связанный пользователь, то его имя или идентификатор нужно хранить в айтеме. Нет джоинов в монге, можно конечно для каждого айтема делать отдельный запрос в базу, но это будет огромный поток запросов, очень не оптимально, вся идея монги, чтобы сократить кол-во обращений к базе, в идеале вообще делать один запрос при каждом обращении клиента к сетевому API. Все разом читаем и отдаем.

[Roman Hinex]

Тимур Шемсединов: эм... не совсем, мне бы просто вытащить список айтемов, связать их с парой полей из другой таблицы и отдать в шаблон, хотя бы через json.

[Roman Hinex]

Тимур Шемсединов: то что мне и нужно, но покурив документацию я так и не понял как это сделать. С монго дело имею в первый раз, немного жмут сроки решения проблемы, отсюда и вопросы.

[Roman Hinex]

Тимур Шемсединов: документ и назвал таблицей, прошу прощения.

[Тимур Шемсединов]

Дело реально плохо, Вы, похоже, писали на PHP раньше, а это другой стек программирования, все концепции другие, так с налету не пойдет. Пока Вы не поймете как работает нода и как работает монга, то ни какого сетевого API не напишите. Берите Хабр и читайте неделю статьи, найдите тут на Тостере есть несколько подробных ответов, где даются ссылки на учебные материалы.

[Тимур Шемсединов]

Я предлагаю Вам отказаться от этих сроков, потому, что Вы сейчас напишете страшный говнокод, если не изучите новый подход, а начнете просто решать проблему в лоб. Если не откажитесь от сроков, то и сроки сгорят и задачу не решите и не научитесь и код на выброс пойдет.

[Roman Hinex]

Тимур Шемсединов: да, действительно так и есть. Думаю потратить выходные на изучение концепций и способов работы с ними. Но к сожалению задача весит сейчас и нужно ее хоть как-то решить. Пускай даже временным решением. Спасибо за помощь)

[Roman Hinex]

Тимур Шемсединов: хорошо, думаю действительно правильнее будет сделать так.

[Тимур Шемсединов]

Люди приходят с PHP, где привыкли рендерить страницы на сервере, т.е. использовать шаблоны и MVC. А для ноды характерно, когда клиент SPA (single-page application) отделен от сервера сетевым API. Сервер отдает данные в JSON, а клиент уже рендерит их в HTML через шаблонизатор, MVC, или вот на Angular или React. Если это понять, то уже много времени сэкономите, не пытаясь делать 2 MVC, одно на сервере, а второе на клиенте и соединять их при помощи скотча и туалетной бумаги.

[Тимур Шемсединов]

Если уже посмотрели скринкаст learn.javascript.ru/nodejs-screencast то попробуйте пройти лабораторки тут nodeschool.io/ru потренируйтесь решать задачи на ноде, чтобы адекватно оценить свои силы и понять в чем пробелы. Посмотрите вот этот вопрос Есть какие-нибудь ресурсы по построению правильной серверной архитектуры на node.js/io.js? и вот этот Какие связки лучше для web проекта (SAAS)? По использованию async: Как заставить Node JS Выполнять код по порядку? И еще вот эту статью советую: habrahabr.ru/post/204958 для общего развития

[Roman Hinex]

Тимур Шемсединов отлично) вот за это отдельная благодарность)

[Vitaly Sivkov]

Тимур Шемсединов: Спасибо за ссылку nodeschool.io/ru !

[Олег]

Тимур Шемсединов,

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 });
});
// ВЫТАЩИТЬ ЗА ФУНКЦИЮ
console.log(items_list); // Я так понял, что в данном случае мы не вытащили ничего. Данный запрос все равно закончится undefined

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

Видимо, что-то не понимаю.
Мне нужно забирать разные данные из базы, обрабатывать и отправлять обратно. Но решений пока я не получил (

Answer 2

[Назар Мокринский]

Наиболее близкую аналогию тому что вы хотите можно получить с помощью Promises.

Answer 3

[Дмитрий Авилов]

Гораздо удобнее (и правильнее) писать на async в таком стиле.

var async = require('async');
async.auto({
    'Aitems': function(callback) { // делаем первый запрос к базе
      db.model('items').find({}, function (err, data) {
        callback(err, data); // данные получены, возвращаемся
      });
    },
    'Busers': function(callback) { // параллельно делаем второй запрос к базе
      db.model('users').find({}, function (err, data) {
        callback(err, data); // данные получены, возвращаемся
      });
    }
},
function(err, results) {
  if (err){return console.error(err);} // если будут ошибки - их напишем.
  // ошибок нет, нам доступны результаты каждой операции
  console.dir({ items: results['Aitems'], users: results['Busers'] });
});

У вас есть обработка ошибок, плюс вы уверены, что ваши переменные items_list и user_list не будут изменены где-то ещё, во время обработки запросов. Некрасиво выносить переменные за функцию, которая их изменяет.

Comments

[Тимур Шемсединов]

parallel тоже можно сделать по хешу именованных функций, а не по массиву. Ни то ни то не более и не менее правильно. К обработке ошибок это вообще ни малейшего касания не имеет, обработка ошибок одинаковая в обоих случаях: ошибки возвращаются из функций через колбэки первым параметром и попадают в первый параметр финальной функции. Разница только в том, будет ли async сам собирать данные в один хеш или мы запишем данные в свою структуру. Переменные items_list и user_list они же не глобальные, поэтому есть полная уверенность, что они не будут изменены где-то во время запроса. Выносить переменные за функцию, которая их изменяет можно и нужно. JavaScript построен на иерархически вложенных контекстах, которые порождаются функциями и модулями (файлами). На самом деле, конечно, только функциями, а модули при загрузке через require оборачиваются автоматом в функцию, экранирующую контекст модуля от остального приложения. Так вот, совершенно нормально, если переменные будут объявлены в одном контексте, то могут быть изменены в любом вложенном в него. Другими словами, если есть функция, объявляющая переменную, и внутри этой функции определена так же и вложенная функция, то вложенная может изменять все переменные наружной. Это нормально, не нужно стараться в JavaScript сделать "чистые функции", как в языках исключительно функциональных.