使用splice操作数组：购物车删除函数异常排查求助

嘿，我一眼就看出问题出在哪了——你调用splice()的时候，肯定是没传对目标元素的正确索引，反而传了数组最后一个位置的索引，所以才会每次都删最后一个。咱们来把这个问题彻底解决掉：

先补全你没写完的添加函数（方便测试） #

首先先把你的AddtoCart函数补完整，确保能正常往购物车里加商品：

var shoppingCart = [];

function AddtoCart(name, description, price) {
    // 构建商品对象
    var singleProduct = {
        name: name,
        description: description,
        price: price
    };
    // 推入购物车数组
    shoppingCart.push(singleProduct);
    console.log(`${name} 已成功加入购物车`);
}

修复删除函数的核心问题 #

你的删除函数之所以失效，本质是没精准定位到要删的商品在数组里的位置。下面给你两种靠谱的修复方案：

方案1：用findIndex()快速定位（最简洁）

findIndex()会帮你遍历数组，直接返回第一个匹配元素的索引，找不到就返回-1，非常适合这种场景：

function removeFromCart(targetProductName) {
    // 找到目标商品的索引
    const targetIndex = shoppingCart.findIndex(product => product.name === targetProductName);
    
    // 先检查是否找到商品，再执行删除
    if (targetIndex !== -1) {
        shoppingCart.splice(targetIndex, 1);
        console.log(`${targetProductName} 已从购物车移除`);
    } else {
        console.log(`购物车里找不到 ${targetProductName} 哦`);
    }
}

方案2：用普通for循环遍历（适合需要更精细控制的场景）

如果你习惯用for循环，要注意找到索引后直接用这个索引调用splice，而且记得找到后可以跳出循环（避免重复删除同名商品，如果你允许多个同名商品，就去掉break）：

function removeFromCart(targetProductName) {
    for (let i = 0; i < shoppingCart.length; i++) {
        if (shoppingCart[i].name === targetProductName) {
            // 用当前的i（目标元素的索引）调用splice
            shoppingCart.splice(i, 1);
            console.log(`${targetProductName} 已从购物车移除`);
            // 找到后跳出循环，只删第一个匹配的商品
            break;
        }
    }
}

为什么你的原代码会删最后一个元素？ #

举个例子，如果你原代码里写的是shoppingCart.splice(shoppingCart.length - 1, 1)，那splice的第一个参数是数组最后一个元素的索引（因为数组索引从0开始，length-1就是最后一位），自然每次都删最后一个。只要把这个参数换成目标元素的真实索引，问题就解决了。

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