关于Javascript splice方法的一个坑。

时间:2023-03-10 07:23:26
关于Javascript splice方法的一个坑。

w3c相关文档:http://www.w3school.com.cn/jsref/jsref_splice.asp

bug:购物车计算价格的时候。加商品没问题,减商品的时候价格总是计算错误。

经排查发现在减商品计算总价的时候调用了splice方法,导致正在循环的数组发生了改变,价格计算错误。

修改方法:总价计算完成的时候再操作数组。

上代码:

 //错误代码
$.each(categoryJson.shop_cart_consume,function (i,prodItem) {
//S_IS_NEW_USER_PARSE 1新用户 2老用户
if(S_IS_NEW_USER_PARSE == '1' && parseInt(prodItem.unit_price_discount) > 0){
totalPrice += Number(prodItem.unit_price_discount * prodItem.count);
}else{
totalPrice += Number(prodItem.unit_price *prodItem.count);
}
if(prodItem.ware_sku_id == product.ware_sku_id){
prodItem.count --;
if (prodItem.count <= 0){
categoryJson.shop_cart_consume.splice(i,1);
}else{
categoryJson.shop_cart_consume[i] = prodItem;
}
}
});
 //正确代码
//存放数组中减去元素的下标。
var arrayIndex;
//用变量 来标识从0开始的下标。
var beginIndex = 0;
$.each(categoryJson.shop_cart_consume,function (i,prodItem) {
//S_IS_NEW_USER_PARSE 1新用户 2老用户
if(S_IS_NEW_USER_PARSE == '1' && parseInt(prodItem.unit_price_discount) > 0){
totalPrice += Number(prodItem.unit_price_discount * prodItem.count);
}else{
totalPrice += Number(prodItem.unit_price *prodItem.count);
}
if(prodItem.ware_sku_id == product.ware_sku_id){
prodItem.count --;
if (prodItem.count <= 0){
// categoryJson.shop_cart_consume.splice(i,1);
arrayIndex = i;
}else{
categoryJson.shop_cart_consume[beginIndex] = prodItem;
beginIndex++;
}
}
});
categoryJson.shop_cart_consume.splice(arrayIndex,1);