Как правильно оформлять необязательную функцию обратного вызова (callback)?

Question

[Сергей]

Есть функция с optional callback (callback может быть, а может и не быть). Как правильно оформить его в такой функции?

function (callback) {
  var request = new XMLHttpRequest();
  request.callback = callback;
  ...
}

Comments

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

Что значит "оформить"?

[Сергей]

Правильно записать :)

Answer 1

[Владимир Проскурин]

Проверить есть ли она, если есть присвоить, если нет, сделать какую-то штуку?

Comments

[Сергей]

Если нет - ничего не делать. Такая запись приемлема?
if (callback) request.callback = callback;

[Владимир Проскурин]

Сергей, да. Вообще, сейчас модно возвращать промис, но не будем об этом.

Answer 2

[Василий Мажекин]

function (callback) {
  var request = new XMLHttpRequest();
  request.callback = callback || function() {};  // ES6: ... callback || () => {}
  ...
}

Comments

[Сергей]

То есть идея в том, чтобы коллбек по умолчанию был пустой функцией? Это хорошая практика?

[Василий Мажекин]

хорошая ))

[Алексей Ярков]

function (callback = () => {}) {
  var request = new XMLHttpRequest();
  request.callback = callback
  ...
}