Почему постоянно лезет undefined?

Question

[nepster-web]

Работаю с node.js. Есть вот такая конструкция:

var result = findUserByHash('hash', function(user){
            console.log (user);
            return user;
        });

        console.log (result);

Момент в том, что внутри колбэка все хорошо, однако при вызове console.log (result), показывает undefined.
Насколько я понял, это фишка асинхронности?
Это получается, нужно код выстраивать цепочкой из колбэков?

Comments

[Илья Шатохин]

Небольшое замечание - первый аргумент всегда должен возвращать ошибку (или null, если все хорошо), а вот результаты асинхронных функций вторым и т.д. аргументом. Иначе вам будет ОЧЕНЬ сложно совмещать свой код со стандартными библиотеками для node (например, async). Такое вот устоявшееся соглашение обработки ошибок.

[Илья Шатохин]

для примера у вас должно быть

findUserByHash('hash', function(err, user){
  if (err) {
    /* */
    return;
  }

  /* */
});

Answer 1

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

На сколько я поля это фишка асинхронности !?

Правильно поняли.

Это получается нужно код выстраивать цепочкой из калбэков ?

Необязательно. (для nodejs, понятно, своя специфика, но общие принципы такие же, естественно).

Answer 2

[Сергей Протько]

Да, именно так, на момент вызова console.log колбэк еще не отработал. Добро пожаловать в callback hell. Альтернативы:
- промисы, позволяют уменьшить вложенность колбэков
- корутины - сложно но можно писать асинхронный код в синхронном стиле, повторюсь это сложно
- async/await - еще не запилили в js.

Comments

[nepster-web]

понял, у меня кода не очень много на node.js, смогу себе позволить вложенность или есть последствия ?

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

nepster09: почитайте - callbackhell.com

[Сергей Протько]

nepster09: ну люди ж как-то живут. Хотя промисы вам бы думаю пригодились, с ними код хоть немного более читабельнее.

[Александр Прозоров]

Вы забыли упомянуть замечательную библиотеку async (https://github.com/caolan/async). Несложно и эффективно упрощает проблему с callbackHell

[Сергей Протько]

Александр Прозоров: ну это всего-лишь своего рода надстройка над промисами и хороший пример как они помогают жить лучше...

[Александр Прозоров]

Сергей Протько: Библиотека async - надстройка над промайсами? Вы серьёзно? Это альтернатива, которая к промайсам не имеет никакого отношения. И альтернатива более удобная в условиях работы с потоками ввода/вывода в node.js.

[Сергей Протько]

Александр Прозоров: я не о конкретной реализации а о подходе - все это можно спокойно организовать на промисах, они дают более примитивные элементы flow control. Важна не реализация а идея.

p.s. корутины всеравно поудобнее будут, хоть и сложнее.

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

Александр Прозоров: извините, но "промайс" - это мыши-профессионалы.

[Александр Прозоров]

Алексей Уколов: ок, вам виднее