Как проверить, что переданный параметр это функция?

Question

[Святослав]

У меня есть функция, которая одним из параметров принимает колбэк:

function foo(callback) {
  callback();
}

Но колбэк — необязательный параметр. Как наиболее правильным образом проверить, что callback — функция и что она действительно передана? Я сделал так:

function foo(callback) {
  if (typeof callback === 'function') {
    callback();
  }
}

Насколько это правильное решение в современном JavaScript?

Answer 1

[Alex]

Вполне правильно, как по мне

Answer 2

[Сергей Соколов]

typeof будет равен "function" и для Object, String, Date, например — что не входит в ваши планы.

С другой стороны, если весь код ваш, то достаточно проверять на не-пустоту
callback && callback();

Библиотека UnderscoreJS так определяет, функция ли аргумент:

Object.prototype.toString.call(callback) === '[object Function]'

и для вас можно так:

callback  &&  Object.prototype.toString.call(callback) === '[object Function]'  &&  callback();

Comments

[Владимир]

чем может помешать то, что typeof Object === "function"? Не вижу проблем, как и в использовании typeof callback === 'function'

[Святослав]

Спасибо за подробный ответ!