節點返回BluebirdJS答應

Question

我有一個小問題，這個腳本作品完美，有一個問題，「runTenant」方法沒有返回一個承諾（需要從「所有（）」解決

此代碼：

Promise.resolve(runTenant(latest)).then(function() { 
    end(); 
}); 

調用此代碼：

function runTenant(cb) { 
    return new Promise(function() { 
    //global var 
    if (!Tenant) { 
     loadCoreModels(); 
     Tenant = bookshelf.core.bs.model('Tenant'); 
    } 

    new Tenant().fetchAll() 
     .then(function(tenants) { 

     if (tenants.models.length == 0) { 
      return; 
     } else { 
      async.eachSeries(tenants.models, function(tenant, next) { 

      var account = tenant.attributes; 
      Promise.resolve(db_tenant.config(account)).then(function(knex_tenant_config) { 
       if (knex_tenant_config) { 
       db_tenant.invalidateRequireCacheForFile('knex'); 
       var knex_tenant = require('knex')(knex_tenant_config); 
       var knex_pending = cb(knex_tenant); 
       Promise.resolve(knex_pending).then(function() { 
        next(null, null); 
       }); 
       } else { 
       next(null, null); 
       } 
      }); 

      }); 
     }; 
     }); 
    }); 
} 

從runTenant代碼工作正常但它檔，不繼續執行「結束（）」，因爲自許「runTenant（最新）」 ISN」不要解決。

好像它不明顯，我對承諾感到恐怖。仍在努力讓我的頭在他們周圍。

非常感謝任何幫助/方向！

Answer 1

你不應該使用Promise構造都在這裏（基本上，其他任何地方都沒有），即使你讓它工作，它也會是antipattern。您從未解決過這個承諾 - 請注意resolve argument到Promise構造函數的回調函數與Promise.resolve函數是非常不同的函數。

如果你有一個功能強大的承諾庫，如藍鳥，你不應該使用async庫。

就好像它不明顯，我對承諾感到恐怖。

也許你會想看看我的rules of thumb寫作承諾函數。

這是你的功能應該是什麼樣子：

function runTenant(cb) { 
    //global var 
    if (!Tenant) { 
    loadCoreModels(); 
    Tenant = bookshelf.core.bs.model('Tenant'); 
    } 
    return new Tenant().fetchAll().then(function(tenants) { 
    // if (tenants.models.length == 0) { 
    // return; 
    // } else 
    // In case there are no models, the loop iterates zero times, which makes no difference 
    return Promise.each(tenants.models, function(tenant) { 
     var account = tenant.attributes; 
     return db_tenant.config(account).then(function(knex_tenant_config) { 
     if (knex_tenant_config) { 
      db_tenant.invalidateRequireCacheForFile('knex'); 
      var knex_tenant = require('knex')(knex_tenant_config); 
      return cb(knex_tenant); // can return a promise 
     } 
     }); 
    }); 
    }); 
} 

Answer 2

您需要返回所有嵌套的promise。我不能運行這個代碼，所以這不是一個解決方案。但希望它能幫助你理解缺失的東西。

function runTenant(cb) { 
    //global var 
    if (!Tenant) { 
     loadCoreModels(); 
     Tenant = bookshelf.core.bs.model('Tenant'); 
    } 

    return new Tenant().fetchAll() //added return 
     .then(function (tenants) { 

      if (tenants.models.length == 0) { 
       return; 
      } else { 
       var promises = []; //got to collect the promises 
       tenants.models.each(function (tenant, next) { 

        var account = tenant.attributes; 
        var promise = Promise.resolve(db_tenant.config(account)).then(function (knex_tenant_config) { 
         if (knex_tenant_config) { 
          db_tenant.invalidateRequireCacheForFile('knex'); 
          var knex_tenant = require('knex')(knex_tenant_config); 
          var knex_pending = cb(knex_tenant); 
          return knex_pending; //return value that you want the whole chain to resolve to 
         } 
        }); 
        promises.push(promise); //add promise to collection 
       }); 
       return Promise.all(promises); //make promise from all promises 
      } 
     }); 
} 

Answer 3

您的承諾在runTenant函數永遠不會解決。你必須調用resolve或reject功能來解決承諾：

function runTenant() { 
    return new Promise(function(resolve, reject) { 
    // somewhere in your code 
    if (err) { 
     reject(err); 
    } else { 
     resolve(); 
    } 
    }); 
}); 

，你不應該在runTenant功能通過cb，使用承諾鏈：

runTenant() 
.then(latest) 
.then(end) 
.catch(function(err) { 
    console.log(err); 
});