javascript - Nodejs Promise reject 不会转发错误消息

Question

我试图捕获一个错误，指出我试图读取的文件不存在。我故意将文件命名错误。

在我的 Controller 中，我有 getAllProducts() 将返回一个 promise ，如果 promise 解决了，那么我呈现一个页面，如果没有，则将错误显示到控制台

// inside productController.js 
exports.getProductForShop = (req, res, next) => {
    productModel.getAllProducts()
        .then(products => {
            res.render('shop', {
                title: 'shop',
                products,
                path: '/'
            });
        })
        .catch((error) => {
            console.log(error);
        });
}

在我的模型类中，我已经实现了上面提到的函数，即:getAllProducts()这里是代码，它是一个静态函数，如果这很重要的话

// inside productModel.js
static getAllProducts() {
    return new Promise((resolve, reject) => {
        fs.readFile('data/wrongName.json', (error, products) => { // wrong file name
            if (error) {
                reject(new Error(error)); // <-- line that sends error message
            }

            resolve(JSON.parse(products)); // this is line 52, it should not be executed
        })
    })
}

当我运行代码时，我收到以下错误消息

SyntaxError: Unexpected token u in JSON at position 0
    at JSON.parse (<anonymous>)
    at ReadFileContext.callback (C:\Users\user\Desktop\JS\NodeJs\models\productModel.js:52:30)
?[90m    at FSReqCallback.readFileAfterOpen [as oncomplete] (fs.js:239:13)?[39m

这一定意味着 fs.readFilee 收到了错误的文件名，抛出了一个错误，reject(new Error(error)); 被调用了，仍然执行执行第 52 行，其中包含 resolve();

当我在 if(error) block 中控制台记录错误时，我得到正确的错误消息

Error: Error: ENOENT: no such file or directory, open 'C:\Users\user\Desktop\JS\NodeJs\data\wrongName.json'
    at ReadFileContext.callback (C:\Users\user\Desktop\JS\NodeJs\models\productModel.js:49:35)
    at FSReqCallback.readFileAfterOpen [as oncomplete] (fs.js:239:13)

当我执行 return reject(new Error(error)); 时，我得到正确的错误消息，与上面显示的相同。

有人可以向我解释为什么传递给 reject(new Error(error)); 的错误消息没有在 catch block 中捕获吗？

更新:添加 catch block 也不起作用，似乎 return render 是唯一的方法？!我的印象是拒绝将 promise 置于拒绝状态并阻止进一步执行

// inside productModel.js
static getAllProducts() {
    return new Promise((resolve, reject) => {
        fs.readFile('data/productseData.json', (error, products) => {
            if (error) {
                reject(new Error(error));
            }

            resolve(JSON.parse(products));
        })
    }).catch((error) => {
        console.log(error);
    })
}

Answer 1

你对这里发生的事情有点误解。调用 reject 将拒绝 promise ，随后的 resolve 将不会解析它。到目前为止，一切都很好。问题是它不会在没有return 的情况下停止执行。因此，您实际上是在运行不必要的代码，而这些代码恰好会导致错误。

您应该确保 resolve 代码不会在发生错误的情况下执行。您可以使用 return reject 或将 resolve 包装在 else block 中来完成此操作。

请记住，resolve/reject 没有什么特别之处。它们是简单的函数调用，函数调用不会停止执行。