Firebase云功能打字稿错误“并非所有代码路径都返回值”

Question

我正在使用firebase云功能和firestore事务来减少基于购买的产品的可用数量。在部署时，它返回错误“错误TS7030：并非所有代码路径都返回值”

这是代码

import * as functions from 'firebase-functions';
import * as admin from 'firebase-admin';


admin.initializeApp();
const db = admin.firestore()

exports.newOrder = functions.firestore
.document('orders/{orderId}')
.onCreate(async (snap, context) => {

    try {
        const data = snap.data();
        if (data === undefined) return null

        const itemList = data.list as Array<any>
        const productId: Set<string> = new Set<string>();

        itemList.forEach((item) => {
            productId.add(item.id)
        })
        return db.runTransaction(async t => {

            const promises: Promise<admin.firestore.DocumentSnapshot>[] = []
            productId.forEach(i => {
                const p = admin.firestore().doc('Products/' + i).get()
                promises.push(p)
            })
            const docs=await Promise.all(promises)
            docs.forEach(doc => {
                if (!doc.exists) {
                    return Promise.reject("Product deleted")
                }
            })
            itemList.forEach(j => {
                if (j.variation === '-') {
                    const available = docs[j.id].get('available')
                    const needed = j.quantity
                    if (available < needed) {
                        return Promise.reject("Product out of stock")
                    }
                }
                else {
                    const variations = docs[j.id].get('variation') as Map<string, any>
                    for (const i in variations.keys) {
                        if (i === j.variation) {
                            const needed = j.quantity
                            const available = docs[j.id].get('variation').get(i).get('quantity')
                            if (available < needed) {
                                return Promise.reject("Product out of stock")
                            }
                        }
                    }
                }
            })
            itemList.forEach(j => {
                if (j.variation === '-') {
                    const available = docs[j.id].get('available')
                    const needed = j.quantity
                    t.update(db.doc('Products/' + j.id), { 'available': available - needed })
                }
                else {
                    const variations = docs[j.id].get('variation') as Map<string, any>
                    for (const i in variations.keys) {
                        if (i === j.variation) {
                            const needed = j.quantity
                            const available = docs[j.id].get('variation').get(i).get('quantity')
                            t.update(db.doc('Products/' + j.id), { [`variation.${i}.quantity`]: available - needed })
                        }
                    }
                }
            })
            return Promise.resolve("Product quantity updated")
        })
    }
    catch (error) {
        console.log(`error ${error}`)
        return null
    }
});

以下是部署中显示的错误

 src/index.ts:30:30 - error TS7030: Not all code paths return a value.

 30                 docs.forEach(doc => {
                            ~~~~~~~~

 src/index.ts:35:34 - error TS7030: Not all code paths return a value.

 35                 itemList.forEach(j => {
                                ~~~~~~


 Found 2 error 

如何解决错误。

错误中提到的2个循环检查产品是否被删除，产品是否缺货。如果它满足我想退出函数的条件。请帮我。

Answer 1

问题比看起来更深。你可能会误解return声明在forEach内做了什么。代码的结构就好像假设检查是否有任何doc.exists为假并且如果是这样返回，但是如果这样写，它将从迭代回调中返回..并且因为forEach不使用回调返回值，承诺拒绝仍未得到处理。

实现这一结果的正确方法如下：

1）直接检查您需要检查的内容：

if (docs.findIndex(doc => !doc.exists) !== -1) {
    return Promise.reject("Product deleted");
}

2）使用for..in或for..of循环而不是forEach：

for (doc of docs) {
    if (!doc.exists) {
        return Promise.reject("Product deleted")
    }
}

3）使用map和await结果（不推荐，因为你真的不需要映射）：

await Promise.all(docs.map(doc => {
    if (!doc.exists) {
        return Promise.reject("Product deleted")
    }
    return null
})

请注意，在这种情况下，结果数组中的任何被拒绝的promise都应该触发对外部promise的拒绝。

---

旁注：你不需要明确的Promise.reject()调用。由于你的函数是异步的，你可以简单地将throw用作错误 - 这将转化为承诺拒绝。

Answer 2

错误消息告诉您有些函数在所有情况下都不返回值。它甚至告诉哪些功能违反了这一要求。这是第一个错误：

 src/index.ts:30:30 - error TS7030: Not all code paths return a value.

 30                 docs.forEach(doc => {
                            ~~~~~~~~

它告诉你，你传递给forEach的函数存在问题。

这是功能：

        docs.forEach(doc => {
            if (!doc.exists) {
                return Promise.reject("Product deleted")
            }
        })

请注意，当doc.exists为true时，该函数不返回值。如果您不关心这种情况，只需返回null：

        docs.forEach(doc => {
            if (!doc.exists) {
                return Promise.reject("Product deleted")
            }
            else {
                return null
            }
        })

现在错误消失了，因为所有代码路径都返回一个值。您可以将相同的逻辑应用于其他错误。