約束から値を返す：コールバックまたは約束？

Question

（*私の最初の質問を修正しました。*）

私はExcelワークブックを再計算し、所要時間をプリントする非同期関数calculateを持っています。

次に、私はそれを記録できるように、計算時間を返す関数を作りたいと思います。非同期関数が値を返すようにします。私は仕事、次の2つの方法を、複数のスレッドを読んで、書き込み：calculate2はpromiseを使用している間、私はcalculate1がcallbackを使用して推測

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'; }); 
} 

：ここ

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); }) 
      }) 
    }) 
} 

はテストです。誰がもっと良い方法を教えてもらえますか？

また、適切な場所にfn(t)（resp。、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

：それはあなたの試みよりもかなり少ないの機能を使用することに注意してくださいあなたの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() 
});