Function.prototype как функция заглушка?

Question

[lui]

Возьмём к примеру код, весьма типичный при асинхронных вызовах:

var doSomething = function(callback) {
  if ("function" !== typeof callback) {
    callback = Function.prototype;
  }

  // .. function body ..

  callback();
}

До этого обычно писал callback = function(){}, но сдаётся мне, что при таком подходе на каждое переопределение колбэка будет создаваться новая функция. Приведённый же выше вариант, вроде бы, должен быть поэкономнее.


Спецификации (15.3.4) на моей стороне, но при этом не покидает ощущение, что так делать не стоит.


Так как: function(){} или Function.prototype?

Comments

[avalak]

А что тут, собственно, экономить?

[Денис Пушкарев]

avalak, тык. Это еще если на память не смотреть, а ведь дело скорее в ней:) Хотя кого волнуют такие микро-оптимизации…

[avalak]

rock, эм, видимо не ясно выразился. мне, собственно, ближе вариант Silver_Clash. а непонятно что/зачем тут именно так экономить (как предложил товарищ lui).

jsfiddle.net/XTn2E/2/
варианты prepared и if callback в среднем наиболее быстрые. но if callback лаконичнее.

Answer 1

[Денис Пушкарев]

Интересное решение, но ведь и к этому калбэку могут что-либо прикрутить. callback.ololo=function(){...} — и это ололо у всех функций, так что аккуратней.

Comments

[lui]

Это один из поводов для сомнений. Но он весьма маловероятен, уж к столь специфичному типу функций как колбэки вряд ли появится желание повесить пару свойств.

[Stdit]

Это весомый довод, всё правильно. Я бы тоже не стал использовать прототип, такие баги потом отлавливать — с ума сойти можно.

[Silver_Clash]

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

[Денис Пушкарев]

Silver_Clash, привет, объекты в js передаются по ссылке, callback — ссылка на Function.prototype, Function.prototype в цепочке прототипов всех функций, ваш кэп, спасибо за минус:)

[Silver_Clash]

Описали то вы все правильно, вот только при чем тут:

но ведь и к этому калбэку могут что-либо прикрутить. callback.ololo=function(){...} — и это ололо у всех функций, так что аккуратней.

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

[Денис Пушкарев]

А вы проверьте.

[Silver_Clash]

Если вы говорите о том что вы в своей функции добавляете какие то методы к колбэку, то есть еще 100500 способов выстрелить себе в ногу, никто кроме стреляющего не виноват :)

[Денис Пушкарев]

Т.е. вы пишите код только для себя и со сторонним кодом он взаимодействовать не будет?) Замечательно. Хотя вы ушли от темы.

[Silver_Clash]

Мой код может взаимодействовать с чем угодно, но если мой код добавляет к колбэку какие то методы, то я сам себе злобный буратино (а я пока не буратино, ну или хотя бы не злобный :) ).

С таким же успехом еще до вызова вашей функции колбэку могут присвоить Function.prototype, что приведет к тем же самым результатам, даже если вы в своем коде не используете прототипы.

Answer 2

[mixrin]

Можно вот действительно простой вопрос, я чего-то непонимаю. А зачем вообще вызывать пустую функцию вместо коллбэка, если его не передали? В чем сакральный смысл?

Answer 3

[Silver_Clash]

А что если так:

if ("function" === typeof callback) {
    callback();
  }

ну или так:

"function" === typeof callback ? callback() : null;

Comments

[Vitaliy Petrychuk]

«function» === typeof callback && callback();
Немного короче :)

[Silver_Clash]

Согласен +1 в карму :)

[Александр Вульф]

@vermilion1 это гениально) Теперь буду вместо тернарного оператора ( v ? cb() : ''; ) использовать просто ( v && cb(); )

[Vitaliy Petrychuk]

@mannaro, еще можно вместо <code lang="javascript"> if (!condition) { method(); } </code>использовать <code lang="javascript"> condition || method(); </code>