错误：函数返回未定义、预期的 Promise 或值

Question

我设置了一个带有 firebase 函数的 pubsub 函数，以便经常在 Firestore 中执行一些操作。为此，我需要向第三方 API 发出请求以获取更新的数据，然后我想将该数据插入到 Firestore 中正确的集合和文档中。

const request_promise = require('request-promise')

exports.scheduledFunction = functions.pubsub.schedule('every 2 minutes').onRun((context) => {
    console.log('This will be called every 2 minutes')
    var username = ''
    var password = ''
    var options = {
        url: 'path.to.api.com',
        auth: {
            user: username,
            password: password
        },
        json: true
    }

    request_promise(options)
        .then(function (product) {
            console.log(product.product_id)
            db.collection('products').doc(product.product_id).set(product)
                .then(() => {
                    console.log('Document successfully written')
                })
                .catch(error => {
                    console.log('Error writing product to firestore', error)
                })
        })
        .catch(function (err) {
            console.log('Failed to get product', error)
        })
  });

在上面的代码中，如果我注释掉将数据添加到 Firestore 的调用，我会将正确的 product_id 打印到控制台，因此我知道请求正在运行，但如果将其留在我得到“函数返回未定义，预期的承诺”或价值'。

Answer 1

通过适当的承诺链，这看起来更干净

      rp(options)
        .then((product) =>
        {
            console.log(product.product_id)
            // add an implicit return here
            return db.collection('products').doc(product.product_id).set(product)
        })
        .then(() =>
        {
            console.log('Document successfully written')
            // need's to return something here, using a boolean for simplicity
            return true;
        })
        .catch(function (err) 
        {
        console.log('Failed to get product', error);
        // throw will exit the function call
        throw Error('Failed to get product', error);
        });

另外，不建议从 catch 块中抛出错误，catch 块是为了捕获错误并处理它们，而不是抛出错误。您的代码中有一些地方需要改进，但这不是这个问题的一部分

干杯， 快乐编码

Answer 2

您在执行时不会返回任何东西。 console.log 不被视为 return

request_promise(options)
        .then(function (product) {
            console.log(product.product_id)
            // add an implicit return here
            return db.collection('products').doc(product.product_id).set(product)
                .then(() => {
                    console.log('Document successfully written')
                    // need's to return something here, using a boolean for simplicity
                    return true;
                })
                .catch(error => {
                    console.log('Error writing product to firestore', error)
                    // throw will exit the function call
                    throw Error('Error writing product to firestore', error);
                })
        })
        .catch(function (err) {
            console.log('Failed to get product', error);
            // throw will exit the function call
            throw Error('Failed to get product', error);
        })