JS嵌套循环更新图书库存逻辑错误问题求助

错误原因分析 #

你的代码出现问题的核心是嵌套了两层数组循环：

• 外层遍历quantity数组的每个元素，内层又遍历booksId数组的所有图书ID，导致每本图书会被执行quantity.length次更新操作（这里是3次），每次减去当前循环的que值，最终每本图书的库存被减去了所有quantity元素的总和。
• 额外问题：两个循环都使用了变量i，内层的i会覆盖外层的索引变量，可能引发逻辑混乱；quantity数组中的元素是字符串类型，直接用于数值计算存在潜在风险；map结合async/await无法正确等待所有异步操作完成，可能导致更新不完整。

修正后的代码 #

我们需要让booksId和quantity按索引一一对应，去掉嵌套循环，同时处理类型转换并正确等待异步操作：

方式一：使用Promise.all批量处理 #

const booksId = ['63761881ec1e2022c0d9e987','637e262ea6fd0c1c544e9a19','637a2798b661e13c40433698'];
const quantity = [ '1', '2', '1' ];

const updateQuantityBooks = async (booksId, quantity) => {
    // 校验数组长度，避免索引越界
    if (booksId.length !== quantity.length) {
        throw new Error('booksId和quantity数组长度不匹配');
    }

    // 用Promise.all等待所有更新完成
    await Promise.all(booksId.map(async (bookId, index) => {
        // 将字符串转为整数，确保数值计算准确
        const que = parseInt(quantity[index], 10);
        if (isNaN(que)) {
            throw new Error(`第${index}个quantity值无效: ${quantity[index]}`);
        }
        await Book.findByIdAndUpdate(
            bookId,
            { $inc: { quantity: -que } },
            { new: true, runValidators: true }
        );
    }));
};

方式二：使用for...of循环（更直观的异步等待） #

const booksId = ['63761881ec1e2022c0d9e987','637e262ea6fd0c1c544e9a19','637a2798b661e13c40433698'];
const quantity = [ '1', '2', '1' ];

const updateQuantityBooks = async (booksId, quantity) => {
    if (booksId.length !== quantity.length) {
        throw new Error('booksId和quantity数组长度不匹配');
    }

    for (let index = 0; index < booksId.length; index++) {
        const bookId = booksId[index];
        const que = parseInt(quantity[index], 10);
        if (isNaN(que)) {
            throw new Error(`第${index}个quantity值无效: ${quantity[index]}`);
        }
        await Book.findByIdAndUpdate(
            bookId,
            { $inc: { quantity: -que } },
            { new: true, runValidators: true }
        );
    }
};

关键注意点 #

• 数组长度校验：添加长度匹配检查，避免因数组长度不一致导致的索引越界或错误更新。
• 类型转换：将字符串类型的quantity转为整数，确保数值计算的准确性。
• 异步操作等待：使用Promise.all或for...of确保所有数据库更新操作完成后再结束函数，避免出现未完成的异步操作。

内容的提问来源于stack exchange，提问作者elyasaf