Q約束チェーン、

Question

呼ばれないエラーハンドラは、このコード

var tryWithoutReindexing = function(indexName, properties) { 
     var settings = properties["settings"]; 
     var mappings = properties["mappings"]; 
     return elastic.closeIndex(indexName) 
      .then(elastic.putSettings(indexName, settings)) 
      .then(elastic.putMapping(indexName, mappings)) 
      .then(elastic.openIndex(indexName)); 
}; 

、コール考えてみましょう：

tryWithoutReindexing(indexName, newProperties) 
.then(function success(value){ 
     console.log('migration successful'); 
    }, function error(){ 
     console.log('migration unsuccessful'); 
    }); 

をメソッドelastic.putSettingsでエラーが発生しますが、いくつかの理由で、consoleログ'migration is successful'ため。エラーハンドラが呼び出されると思います。

私はこれまでの方法を変更する場合：

var tryWithoutReindexing = function(indexName, properties) { 
     var settings = properties["settings"]; 
     var mappings = properties["mappings"]; 
     return elastic.closeIndex(indexName) 
      .then(elastic.putSettings(indexName, settings)) 
       .then(function success() { 
       console.log('err'); 
      }, function(error) { 
       console.log(error); 
      }) 
      .then(elastic.putMapping(indexName, mappings)) 
      .then(elastic.openIndex(indexName)); 
}; 

、およびラインconsole.log(error);にブレークポイントを入れて、エラーハンドラが呼び出されるので、putSettingsメソッドが正しく動作するように思われます。

誰も私の最初の例では、約束のチェーンで発生したエラーを処理しない理由を説明できますか？

Answer 1

私はelastic.putSettings()らが約束を返すと仮定します。 .thenの引数として約束を使うことはできません。そのメソッドは関数引数を期待しています。しかし、これらの関数は約束を返すことができます。

したがって、約束返却関数を無名関数でラップし、その関数を.thenの引数として使用する必要があります。同様に：

var tryWithoutReindexing = function(indexName, properties) { 
    var settings = properties["settings"]; 
    var mappings = properties["mappings"]; 

    return elastic.closeIndex(indexName) 
       .then(function() { 
        return elastic.putSettings(indexName, settings); 
       }) 
       .then(function() { 
        return elastic.putMapping(indexName, mappings); 
       }) 
       .then(function() { 
        return elastic.openIndex(indexName); 
       }); 
};