Style JavaScript pour les callbacks optionnels

Question

J'ai quelques fonctions qui, de temps en temps (pas toujours), vont recevoir un callback et l'exécuter. Est-ce que vérifier si la callback est définie/fonction est un bon style ou y a-t-il un meilleur moyen ?

Exemple :

function save (callback){
   .....do stuff......
   if(typeof callback !== 'undefined'){
     callback();
   };
};

Answer 1

Je préfère personnellement

typeof callback === 'function' && callback();

Le site typeof est cependant douteuse et ne doit être utilisée que pour les cas suivants "undefined" et "function"

Les problèmes liés à la typeof !== undefined est que l'utilisateur peut passer une valeur qui est définie et pas une fonction

Answer 2

Vous pouvez aussi le faire :

var noop = function(){}; // do nothing.

function save (callback){
   callback = callback || noop;
   .....do stuff......
};

C'est particulièrement utile si vous utilisez la fonction callback à quelques endroits.

De plus, si vous utilisez jQuery vous avez déjà une fonction de ce type, elle s'appelle $.noop

Answer 3

Faites simplement

if (callback) callback();

Je préfère appeler le callback s'il est fourni, quel que soit son type. Ne la laissez pas échouer silencieusement, afin que l'implémenteur sache qu'il a passé un argument incorrect et puisse le corriger.

Answer 4

Il est facile de le faire avec ArgueJS :

function save (){
  arguments = __({callback: [Function]})
.....do stuff......
  if(arguments.callback){
    callback();
  };
};

Answer 5

Si le critère d'exécution de la fonction de rappel est de savoir si elle est définie ou non, alors tout va bien. Aussi, je suggère de vérifier si c'est vraiment une fonction en plus.