打破承诺链的好方法是什么？

Question

我想知道如何才能在JS中正确地打破承诺链。

在这段代码中，我首先连接到数据库，然后检查集合是否已经有一些数据，如果没有添加它们。不要注意一些actionhero.js，code..it在这里并不重要。

主要问题是：使用抛出null来打破链行吗？

mongoose.connect(api.config.mongo.connectionURL, {})
        .then(() => {
            return api.mongo.City.count();
        })
        .then(count => {
            if (count !== 0) {
                console.log(`Amout of cities is ${count}`);
                throw null; // break from chain method. Is it okay ?
            }
            return api.mongo.City.addCities(api.config.mongo.dataPath + '/cities.json');
        })
        .then(cities => {
            console.log("Cities has been added");
            console.log(cities);
            next();
        })
        .catch(err => {
            next(err);
        })

非常感谢！

Answer 1

尽管它看起来像是一个聪明的技巧，并且会像您预期的那样起作用，我还是建议不要抛出无错误的对象。

如果您抛出一个实际的错误并显式地处理它，对于其他维护此代码的开发人员来说，这将是更可预测的。

Promise
  .resolve()
  .then(() => {
    throw new Error('Already in database');
  })
  .catch(err => {
    if (err.message === 'Already in database') {
      // do nothing
    } else {
      next(err);
    }
  });