在函數中設置可選回調的最佳方法

Question

這樣的函數中可選成功和錯誤回調的最佳實踐是什麼？這種方法有意義嗎？我似乎有點臃腫。

函數聲明：

var myFunc = function(myNumber, options){ 
    options = options || {}; 
    options.onSuccess = options.onSuccess || function(){}; 
    options.onError = options.onSuccess || function(){}; 

    var myNewNumber = myNumber * 2; 

    if(newVar > 10){ 
     options.onSuccess(myNewNumber); 
    }else{ 
     options.onError(myNewNumber); 
    } 
} 

有回調調用它：

myFunc(2,{ 
    onError: function(myNewNumber){ 
     // do stuff 
    }, 
    onSuccess: function(myNewNumber){ 
     // do stuff 
    } 
}) 

調用它，而不回調：

myFunc(2); 

Answer 1

我會稱他們之前檢查的函數的存在做：

var myFunc = function(myNumber, options){ 
    options = options || {}; 

    var myNewNumber = myNumber * 2; 

    if(newVar > 10){ 
     if (options.onSuccess) { options.onSuccess(myNewNumber); } 
    }else{ 
     if (options.onError) { options.onError(myNewNumber); } 
    } 
} 

這取決於你有多少次會調用這些回調。如果它到處都是，那麼你的方式可能會更好，或者至少更乾淨的代碼。

Answer 2

有幾件事要注意;

1. options.onSuccess = options.onSuccess || function(){};正在檢查成員的存在，而不是檢查其功能。

   你可能想options.onSuccess = (typeof options.onSuccess == "function") ? options.onSuccess : function() { };

2. 這同樣適用於onError

3. 由於爲輕微的優化，你可以指向空函數相同的功能;而不是重新創建它兩次。如果你正在使用jQuery，jQuery的定義jQuery.noop()：

   options.onSuccess = options.onSuccess || jQuery.noop; 
   options.onError = options.onSuccess || jQuery.noop; 
   

4. 在回調是異步的情況下，你會離開自己開到options.onSuccess和options.onError您的支票後正在改變，但在此之前回調解僱;

   var myFunc = function(myNumber, options){ 
       options = options || {}; 
       options.onSuccess = options.onSuccess || function(){}; 
       options.onError = options.onSuccess || function(){}; 
   
       var myNewNumber = myNumber * 2; 
   
       setTimeout(function() { 
        if(newVar > 10){ 
         options.onSuccess(myNewNumber); 
        }else{ 
         options.onError(myNewNumber); 
        } 
       }, 2000); 
   } 
   
   var obj = { 
       success: function() { alert('foo'); }, 
       error: function() { alert('foo'); }, 
   }; 
   
   myFunc(10, obj); 
   delete obj.success; 
   delete obj.error; 
   

   當回調被執行，success和error將是不確定的。