使用live jQuery时Ajax请求的数量增加

Question

我有以下代码，其中使用了clone()和live()。该代码在一个带有链接的jQuery UI对话框中加载。每当我点击链接时，它就会去服务器上获取下面的代码并将其填充到对话框中。第一次加载页面时，它能够正常工作，但是如果我关闭对话框并再次点击链接以获取对话框，则发送的Ajax请求数量会增加。

第一次触发更改时，我只发送一个请求，然后关闭对话框并再次加载它，然后触发更改，它会同时发送两个Ajax请求，第三次则同时发送三个请求，以此类推。

您认为我的问题出在哪里？

<input id="TaskId" name="TaskId" type="hidden" value="18" />
<div id="MainDiv">
    <div id="toClone">
        <div style="display: inline;">
            <select id="Tasksess">
                <option value="">لطفاً کار را انتخاب کنيد</option>
                <optgroup label="کار های جديد">
                        <option value="16"style="">q3fe</option>
                        <option value="18"style="">fjhv j</option>
                        <option value="19"style="">wref</option>
                        <option value="25"style="">ff</option>
                </optgroup>
                <optgroup label="کار های در دست اقدام">
                        <option value="13"style="">rr</option>
                        <option value="15"style="">yy</option>
                </optgroup>
                <optgroup label="کار های تمام شده">
                        <option value="14"style="">tt</option>
                        <option value="18"style="">fjhv j</option>
                </optgroup>
            </select>
        </div>
        <div style="display: inline;">
            <select id="Statusess" name="Statusess"><option value="">لطفاً وابستگی را انتخاب کنيد</option>
<option value="1">پيشنياز</option>
<option value="2">همنياز</option>
</select>
        </div>
        <div style="display: none;" id="Ok">
            ok
        </div>
        <div style="display: none;" id="noOk">
            تکراری
        </div>
        <div id="loadingGif" style="display: none;">
            <img src="/Content/Images/ajax-loader/253L.gif" alt=""/>
        </div>
    </div>
</div>
<script type="text/javascript">
    $(document).ready(function () {

        var Maindiv = $("#MainDiv");
        var toClone = $("#toClone");

        //$("#Statusess").each(function () {
            $("#Statusess").live('change', function () {
                if ($(this).find(":selected").val() != "") {                    
                    if ($(this).parent().prev().find(":selected").val() != "") {
                        $(this).parent().parent().find("#loadingGif").attr("style", "display: inline;");
                        $.ajax({
                            url: '/ProjectAdmin/Project/AddTaskDependency?MainTaskId=' + $("#TaskId").val() + '&DependentTaskId=' + $(this).parent().prev().find(":selected").val() + '&Status=' + $(this).find(":selected").val(),
                            type: 'GET',
                            success: function (data, status) {
                                if (data != "0") {
                                    $(this).parent().parent().find("#Ok").attr("style", "display: inline;");
                                    $(this).parent().parent().find("#noOk").attr("style", "display: none;");
                                }
                                else if (data == "0") {
                                    $(this).parent().parent().find("#Ok").attr("style", "display: none;");
                                    $(this).parent().parent().find("#noOk").attr("style", "display: inline;");
                                }
                                var div = $('div:eq(0)', Maindiv).clone();
                                Maindiv.append(div);
                            }
                        });
                        $(this).parent().parent().find("#loadingGif").attr("style", "display: none;");

                    }
                }
            });
        //});

    });
</script>

- ePezhman

---

旁注：不要使用 live()，而是使用 on()。 - Rene Pot

我在Firefox和Chrome中尝试了你的代码（请参见http://jsfiddle.net/PXhdj/），但它只为每个“change”事件发出一个Ajax请求。也许你有一些其他的代码正在进行Ajax调用/引起“change”事件？顺便说一下，要在元素上设置样式，请使用`.css('display', 'inline')而不是.attr('style', ...)`。 - Jeffery To

Answer 1

你应该尝试：

$("#Statusess").off('change').on('change', function() {

});

PS: 只能使用 jQuery 1.7+ 版本。如果您无法升级：

$("#Statusess").die('change').live('change', function() {

});

- sp00m

---

如果他们无法升级并使用.on()，则应改用.delegate()。 - Anthony Grist

我使用了live，因为我想克隆我的代码，并且新的代码也会响应change()，但是新注入的代码不会触发change。我该怎么办？谢谢。 - ePezhman

Answer 2

在这种情况下，您不应该使用live绑定，因为每次打开对话框时都会“记住”绑定。如果我正确地理解了您的问题，那么您应该使用“click”而不是“live”绑定。

- NickGreen

Answer 3

你的服务器代码是否也会响应底部的javascript代码？

如果是这样，那么你的问题就在于每次加载ui对话框时都会执行一次javascript代码，导致额外的绑定，因此每次执行代码时，都会添加一个额外的请求。

在这种情况下，解决方法很简单，将javascript代码移动到加载ui对话框的页面上即可。

- user999305

Answer 4

如上所述，您应该使用.delegate()而不是.live()。

为了使用delegate，您只需要指定一个父级元素来监听和一个选择器来执行操作。

请尝试以下代码：

<script>
$(function(){
var Maindiv = $("#MainDiv");
var toClone = $("#toClone");

$("#MainDiv").delegate('#Statusess','change', function () {
    if ($(this).find(":selected").val() != "") {                    
      if ($(this).parent().prev().find(":selected").val() != "") {
        $(this).parent().parent().find("#loadingGif").attr("style", "display: inline;");
        $.ajax({
            url: '/echo/html/',
            type: 'POST',
            success: function (data, status) {
                data = "1";
                if (data != "0") {
                    $(this).parent().parent().find("#Ok").attr("style", "display: inline;");
                    $(this).parent().parent().find("#noOk").attr("style", "display: none;");
                }
                else if (data == "0") {
                    $(this).parent().parent().find("#Ok").attr("style", "display: none;");
                    $(this).parent().parent().find("#noOk").attr("style", "display: inline;");
                }
                var div = $('div:eq(0)', Maindiv).clone();
                Maindiv.append(div);
            }
        });
        $(this).parent().parent().find("#loadingGif").attr("style", "display: none;");

    }
  }
});
});
</script>

这里有一个可用的jsfiddle，另外附上一个关于.delegate()函数的参考链接。

- KoU_warch

Answer 5

要么直接绑定事件处理程序，而不使用live()或任何其他委托方法，要么将代码移动到外部脚本文件中，这样您就不会在每个ajax调用上加载相同的代码。使用die()只是一个简单的解决方法，但实际上并不需要存在，并且可以简单地修复。

如果保留代码不变，可以简单地使用：

  $("#Statusess").change(function () {...

- charlietfl

Answer 6

或许你可以尝试一下：

        jQuery(function ($) {

            var Maindiv = $("#MainDiv");
            var toClone = $("#toClone");

                $("#Statusess").on('change', function () {
                    if ($(this).find(":selected").val() != "") {                    
                        if ($(this).parent().prev().find(":selected").val() != "") {
                            $(this).parent().parent().find("#loadingGif").css("display", "inline");
                            $.ajax({
                                url: '/ProjectAdmin/Project/AddTaskDependency?MainTaskId=' + $("#TaskId").val() + '&DependentTaskId=' + $(this).parent().prev().find(":selected").val() + '&Status=' + $(this).find(":selected").val(),
                                type: 'GET',
                                success: function (data, status) {
                                    if (data != 0) {
                                        $(this).parent().parent().find("#Ok").css("display", "inline");
                                        $(this).parent().parent().find("#noOk").css("display", "none");
                                    }
                                    else if (data == 0) {
                                        $(this).parent().parent().find("#Ok").css("display", "none");
                                        $(this).parent().parent().find("#noOk").css("display", "inline");
                                    }
                                    var div = $('div:eq(0)', Maindiv).clone();
                                    Maindiv.append(div);
                                    div.empty();
                                }
                            });
                            $(this).parent().parent().find("#loadingGif").css("display", "none");

                        }
                    }
                });


        });

- Jean G.T