3

我的代码如下:

$(".qtyfield").each(function (index) {
    if (this.value != "" && this.value > 0) {
        var fieldname = this.id;
        countme = countme + 1;
        var tmpProductID = fieldname.split("_")
        var ProductID = tmpProductID[1];
        var ShowPrice = $(this).closest('td').prev('td').text();
        var Quantity = this.value;
        ShowPrice = ShowPrice.replace("$", "");
        isItemInCart(ProductID).done(function () {
            if (isItemInCartVar) {
                updateQuantityByProductID(ProductID, Quantity);
            }
            else {
                addToCartWithQty(ProductID, ShowPrice, Quantity);
            }

        });
        this.value = '';
    }
});

在运行任何进一步的语句之前,我需要确保此代码块完成(包括所有 ajax 调用)。

几点..

  1. isItemInCart是一个带有 Ajax 调用的函数
  2. updateQuantityByProductID是一个带有 ajax 调用的函数
  3. addToCartWithQty是一个带有 ajax 调用的函数
  4. 我不知道每个集合中有多少项目(可能会改变)
4

3 回答 3

2

我会重构代码,使each循环在服务器端。通过这种方式,您可以在客户端准备数组,使用 ajax(使用唯一调用)发送它,然后在您将拥有each循环的服务器上解压缩它。 如果您的每个循环都包含数十亿个元素怎么办?将其乘以 html 标头的长度。

相反,您可以像这样准备数组。

var serverSide = [];

$(".qtyfield").each(function (index) {

    serverSide.push("the data you need");

});

然后您可以将其作为 JSON 字符串发送

var myArg = JSON.stringify(serverSide)

或逗号分隔列表(用于更简单的结构)

var myArg = serverSide.join(',');

并有一个独特的 ajax 调用,可以返回任何值/错误消息

$.post("link", myArg).success(function(data){ alert("data"); })

我不知道你有哪个服务器端支持,但如果你使用 PHP,你可以用json_decode.

如果你使用 C#,你可能想看看JSON.net

无论哪种方式,您都将拥有更多的灵活性。相反,虽然在客户端上执行每个循环是可能的,但它并不是最好的解决方案。Ajax 调用具有标头。

于 2013-09-10T09:11:34.610 回答
2

我的解决方案是将所有这些逻辑整合到一个调用中,例如updateCart().

在客户端,您将创建需要更新的产品列表,例如

[
  {productId: 123, quantity: 2, price: 123},
  {productId: 456, quantity: 1, price: 200}
]

该数据被发送到服务器端,会话数据将在服务器端更新为新的数量。基本上,服务器端的代码将执行与您在所有单独调用中相同的逻辑,但它会更快,因为只有一个请求。

单个请求还可以降低会话锁争用的可能性并提高购物车状态的一致性。

在客户端,只有一个回调函数的优点是不必创建需要同步的单个请求的管道(阅读:屁股的大痛苦)。

于 2013-09-10T09:20:37.177 回答
0

寻找 jQuery 延迟。每个$.ajax电话都会返回一个“承诺”,您可以等待解决:

var promise1 = $.ajax(...)
var promise2 = $.ajax(...)

然后等待他们解决:

$.when(promise1, promise2).then(function(){
  //ready to rumble
});

正如您提到的,您不知道每个集合中有多少项目,只需将每个承诺推入一个数组,可以$.when按如下方式使用:

var promises = [];
promises.push($.ajax(...));
$.when.apply(this, promises).then(function(){...});
于 2013-09-10T09:02:47.900 回答