不能与JQuery中的attr（）一起使用“ onclick”

Question

I'm trying to add an "onclick" event with their respective function from js JQuery to an interactive table td I'm doing in another .php file, the problem is not executed in the place where I want to do it but if the same instruction is executed correctly within an AJAX request a few lines below, will show you the code:

This instruction is giving me the problem:

$(td).attr("onclick","agregar_pensum_etapa2 (this,'"+subject+"','"+level+"','"+stage+"');");

And this, the function I want to run with the "onclick"

function agregar_pensum_etapa2(td,subject,level,stage){

        $(document).ready(function(){
            // Capture variables.
            var id_periodo = $("#id_periodo").val();
            var id_asignatura = subject;
            var id_nivel = level;
            var id_etapa = stage;
            var id_mencion = null;
            if (level> 3) {
               id_mencion = 0;
            }

            // Modifiable column.
            var tr = $(td).parent();
            var index_td = $(td).index();

            // First field
            var valor_anterior = $(td).text();
            $(td).html("<img src =" images / save_64.png 'width = '16' height = '16 '> & nbsp; & nbsp; & nbsp;' + '<input value = "' + valor_anterior + '" type = "text" style = "width: 40px; border: 1px solid #aaa;" onkeypress = "return soloNumeros (event);" maxlength = "2"> ');
            $(td).removeAttr("onclick");
            $(td).find("input").focus();

            // Second field

            var valor_anterior_cs = $(tr).find("td:eq("+(index_td+1)+")").text();
            var checked_cs = "";
            if (valor_anterior_cs === "X"){checked_cs = "checked"}
            $ (tr) .find ("td: eq (" + (index_td + 1) + ")") html ("<input type = 'checkbox'" + checked_cs + ">").

            // Third field

            var valor_anterior_hum = $(tr).find("td:eq("+(index_td+2)+")").text();
            var checked_hum  = "";
            if(valor_anterior_hum === "X") {checked_hum = "checked"}
            $(tr).find("td:eq("+(index_td+2)+")").html("<input type = 'checkbox'"+checked_hum+">");

           / ************************************************* *********** /


            $(td).find("img").click(function(){

                var hora_asignatura  = $(td).find("input").val();

                var mencion_cs = "NO";
                if($(tr).find("td:eq("+(index_td+1)+")").find("input").is ("checked")){mencion_cs = "YES";}

                var mencion_hum = "NO";
                if($(tr).find("td:eq("+(index_td+2)+")").find("input").is("checked")){mencion_hum = "YES";}


                if(hora_asignatura === ""){

                   if(valor_anterior != ''){

                     $(td).html(valor_anterior);      
                     $(tr).find("td:eq("+index_td+1)+")").text(valor_anterior_cs);
                     $(tr).find("td:eq("+(index_td+2)+")").text(valor_anterior_hum);
                   }else{

                     $(td).html("");
                     $(tr).find("td:eq("+(index_td+1)+")").text("");
                     $(tr).find("td:eq("+(index_td+2)+")").text("");
                   }

                   \\// --> HERE IS NOT WORKING <-- \\//
                   $(td).attr("onclick","agregar_pensum_etapa2(this,'"+subject +"','"+level+"','"+stage+"');");
                }else if(hora_asignatura == "0"){

                   if(valor_anterior! = ''){

                      $(td).html(valor_anterior);
                      $(tr).find("td:eq("+(index_td+1)+")").text (valor_anterior_cs);
                      $(tr).find("td:eq("+(index_td+2)+")").text (valor_anterior_hum);
                   }else{

                      $(td).html("<img src =" images / diagonal.png 'height = '16' style = 'width: 15px ">");
                      $(tr).find("td:eq("+(index_td+1)+")").text("");
                      $(tr).find("td:eq("+(index_td+2)+")").text("");
                   }

                   \\// --> HERE IS NOT WORKING <-- \\//
                   $(td).attr("onclick","agregar_pensum_etapa2(this,'"+subject+"','"+level+ "','"+stage+"');");
                }else{

                    $.ajax({
                        async: true,
                        cache: false,
                        dataType: "html"
                        type: 'POST'
                        url: "../Controlador/CtrlPensum.php"
                        data: {

                            id_periodo: id_periodo,
                            id_asignatura: id_asignatura,
                            id_nivel: id_nivel,
                            id_etapa: id_etapa,
                            hora_asignatura: hora_asignatura,
                            mencion_cs: mencion_cs,
                            mencion_hum: mencion_hum,
                            id_mencion: id_mencion,
                            record: "register"
                        },
                        success: function (response) {
                            //alert (response);
                            if($.trim(answer) === "1") {

                               $(td).html(hora_asignatura);

                               var marcar_cs_x = "";
                               if(mencion_cs === "SI"){marcar_cs_x = "X";}

                               var marcar_hum_x = "";
                               if(mencion_hum === "SI"){marcar_hum_x = "X";}

                               $(tr).find("td:eq("+(index_td+1)+")").html (marcar_cs_x).
                               $(tr).find("td:eq("+(index_td+2)+")").html (marcar_hum_x).

                               \\// --> HERE IT WORKS <-- \\//
                               $(td).attr("onclick", "agregar_pensum_etapa2 (this,'"+subject+"','"+level+"','"+stage+"');");
                               cargarTablaResumen ();

                            } Else {
                                alert ("Error SQL statement is not executed." + response);
                            }
                            //tr.fadeOut(500).fadeIn(500);
                        },
                        beforeSend: function () {}
                        Error: function (objXMLHttpRequest) {}
                    });
                }
            });
        });
}

I try this:

$(td).click(function(){
     $(td).attr("onclick","agregar_pensum_etapa2 (this,'"+subject+"','"+level+ "','"+stage+"');");
});

And in the html it prints correctly but does not work, like he was not the "onclick" there. And the function if it works, I put the onclick to that function when I load the page and if it works, but when I click the td and give back to click to restore does not restore the onclick.

Answer 1

Instead of setting the onclick attribute, why not do use .click() again with closures? So your code would be something like...

$(td).click(function(){
    var td = this;
    $(td).click(function () {
         agregar_pensum_etapa2(td,subject,level,stage);
    });
});

Answer 2

是否用小写字母“ d”写$（Document）而不是$（document）。