Почему выводится undefined в замыкании?

Question

[Александр Дегтярев]

"use strict"

  !function () {
    function lodash() {
      alert("Lodash");
    }

    function set(item) {
      alert(item);
    };

    lodash.set = set();
  }();

lodash.set(150);

Почему выводится undefined при выводе?

Answer 1

[Михаил Белошицкий]

Почему выводится undefined при выводе?

Потому что в строке
lodash.set = set();
Вы присвоили имени lodash.set результат выполнения функции set с пустым аргументом (что эквивалентно вызову set(undefined)), а не саму функцию, как вы, вероятно, хотели.

Comments

[Александр Дегтярев]

Что можно сделать? я далее передаю значение 150?

[Михаил Белошицкий]

Например попробуйте написать lodash.set = set вместо lodash.set = set()

[Михаил Белошицкий]

а нет так тоже не сработает. Вот так сработает https://jsfiddle.net/8xjh9xs9/

[Александр Дегтярев]

Все работает но только не под директивой "use strict" без нее все норм, может его нет в новом стандарте такого подхода, через объекты и return модули работают прекрасно

[Михаил Белошицкий]

Но вообще это очень плохой пример на замыкания.

[Александр Дегтярев]

Михаил Белошицкий: самый эффективный через return? возвратом объекта делать? или делать сразу как объект с набором методов? Как наиболее эффективно и производительно?

[Михаил Белошицкий]

Александр Дегтярев: Вы шутите? Если да, то это плохая шутка. На Safari и Chrome проверил.

[Михаил Белошицкий]

Александр Дегтярев: Яваскрипт - не тот язык, в котором использование return или присвоение переменной может определять производительность и эффективность. Мой вариант мне кажется прежде всего наиболее понятным.

[Михаил Белошицкий]

Александр Дегтярев: Еще не очень хорошо использовать функцию с одним и тем же именем в глобальном и локальном контекстах, вообще есть сомнения в нужности function lodash() {}. Без нее я бы сделал так https://jsfiddle.net/8xjh9xs9/1/

[Александр Дегтярев]

Михаил Белошицкий: если в спрашиваете про шутку про директиву "use strict" то смею вас огорчить это не шутка, Яндекс.Браузер ничего не выводит

[Александр Дегтярев]

Михаил Белошицкий: я обучаюсь, подобные примеры прорабатываю

[Михаил Белошицкий]

Александр Дегтярев: Все понял.

Answer 2

[gleendo]

Что то я про такую реализацию модуля не слышал.

Вот так будет работать:

(function () {
    function lodash() {
      alert("Lodash");
    }

    function set(item) {
      alert(item);
    };

    lodash.set = set;

    window.lodash = lodash;
  })();

lodash.set(150);

Comments

[Александр Дегтярев]

Так тоже не работает

[gleendo]

Александр Дегтярев: storage1.static.itmages.ru/i/16/0703/h_1467535083_...

[Александр Дегтярев]

evg_: с директивой "use strict" не работает, без нее нормально работает, странно, к чему это может быть?

[Виталий]

Имхо надо автору читануть теорию побольше.

[Александр Дегтярев]

Виталий: это задание на обучение, от его и прохожу, разными методами. Отсальные методы отлично сработали, кроме этого

[Виталий]

Александр Дегтярев: lodash.set = set() создаст лишь очередной указатель (ссылку) на функцию set(), которая сразу же выполнится и вернет значение в переменную lodash.set а так как ретурна в функции сет нет, оно вернет туда undefined . очень рекомендую от корки до корки прочесть learn.javascript.ru . Его автор к обучению отнесся фундаментально и очень очень структурировано.

[Александр Дегтярев]

Благодарю))