從承諾中回報價值：回調或承諾？

Question

（*我修改了最初的問題。*）

我有一個異步函數calculate從而重新計算Excel工作簿和打印需要多長時間。

然後，我想讓函數返回計算時間，以便我可以記錄它。它認爲使一個異步函數返回一個值。我看過幾個線程，並編寫以下2種方式，其工作：

function calculate1 (mode, fn) { 
    return Excel.run(function (ctx) { 
     ctx.workbook.application.calculate(mode); 
     var before = performance.now(); 
     return ctx.sync().then(function() { 
      var after = performance.now(); 
      var t = after - before; 
      document.getElementById("b").value += 'inside: ' + t + '\n'; 
      fn(t); 
     }) 
    }) 
} 

function calculate2 (mode) { 
    return new Promise(function (resolve, reject) { 
     return Excel.run(function (ctx) { 
      ctx.workbook.application.calculate(mode); 
      var before = performance.now(); 
      return ctx.sync().then(function() { 
       var after = performance.now(); 
       var t = after - before; 
       document.getElementById("b").value += 'inside: ' + t + '\n'; 
       resolve(t); }) 
      }) 
    }) 
} 

下面是測試：

function test() { 
    var a = []; 
    var pm = new OfficeExtension.Promise(function(resolve, reject) { resolve (null); }); 

    pm 
     .then(function() { return calculate1('FullRebuild', function (t) { 
      a.push(t); }); }) 
     .then(function() { return calculate1('FullRebuild', function (t) { 
      a.push(t); }); }) 
     .then(function() { return calculate2('FullRebuild').then(function (result) { 
      a.push(result); }); }) 
     .then(function() { return calculate2('FullRebuild').then(function (result) { 
      a.push(result); }); }) 
     .then(function() { 
      document.getElementById("b").value += a.toString() + '\n'; }); 
} 

我猜calculate1使用callback，而calculate2使用promise。有誰能告訴我哪種方式更好？

此外，是fn(t)（或resolve(t)）在正確的地方，或者我應該包裝在另一個then？

PS：Excel.run和ctx.sync()是JavaScript API for Office的功能;他們都回報了一個承諾。

Answer 1

calculate1()是承諾和回調的錯誤混合，calculate2()正在參展Explicit Promise Construction Antipattern。

這裏是你如何正確使用的承諾沒有任何多餘的浪費：

function calculate1 (mode) { 
    return Excel.run(function (ctx) { 
     ctx.workbook.application.calculate(mode); 
     var before = performance.now(); 

     return ctx.sync().then(function() { 
      var after = performance.now(); 
      var t = after - before; 
      document.getElementById("b").value += 'inside: ' + t + '\n'; 

      return t; 
     }); 
    }); 
} 

然後你就可以消費它這樣。注意，它使用比你嘗試的功能相當少：

return calculate1('FullRebuild') 
    .then(function(res) { 
     a.push(res); 
     return calculate1('FullRebuild'); 
    }) 
    .then(function(res) { 
     a.push(res); 
     return calculate1('FullRebuild'); 
    }) 
    .then(function(res) { 
     a.push(res); 
     return calculate1('FullRebuild'); 
    }) 
    .then(function(res) { 
     a.push(res); 
     document.getElementById("b").value += a.toString() + '\n'; 
    }); 

Answer 2

如果你想要一個異步函數返回值 - 而不是傳遞迴調到它 - 返回值需要被包裹在一個承諾和決心像在你的第二個例子中。

承諾的方便之處在於它避免了回調嵌套 - >當回調需要另一個回調... 和錯誤的方式傳播到最近Promise.catch（）塊

function calculate2 (mode) { 
    return new Promise(function (resolve, reject) { 

     Excel.run(function (ctx) {  
      ctx.workbook.application.calculate(mode); 
      var before = performance.now(); 

      ctx.sync().then(function() { 
       var after = performance.now(); 
       var t = after - before; 

       resolve(t); 
      }) 
     }) 
    }) 
} 

calculate2('FullRebuild') 
    .then(function (t) { 
     document.getElementById("b").value += 'inside: ' + t + '\n'; 
     // the return value of a promise resolver is wrapped in a promise too 
     // and the value of t will be available as parameter of the following .then(funtion(t)) block 
     return t; 
}) 
.catch(function(err) { 
    // Addresses potential async errors raised since calling calculate2() 
});