Question

所以我有以下函数调用另一个名为'change_name（event，name）'的函数。原来'change_name'被调用不止一次？它使用的变量具有混合值。

function input_name(event, name) {
    event.stopPropagation();
    name.style.backgroundColor = "transparent";
    window.localStorage.removeItem("oldname");
    window.localStorage.setItem("oldname", name.value);
    $(this).click( function()
    { 

    change_name(event, name); } );

    $(name).keydown(function(event){
        if(event.keyCode == 13){
            change_name(event, name);
        }
    }); 
}


 function change_name(event, element) {
      event.stopPropagation();
      var name_input = element;
      var name = name_input.value;
      var oldname = window.localStorage.getItem("oldname");
    // new_name.innerHTML = name;
     console.log("Nombre viejo: " + oldname);
     console.log("Nombre nuevo: " + name);

 }

input_name函数是元素的属性

 input.setAttribute("onclick", "input_name(event, this);");

为什么我的价值观混淆了？有什么想法吗？

Answer 1

您每次点击click时都会添加新的keydown和input个活动。这些事件需要在click事件之外添加。

// on click, input_name is called
input.setAttribute("onclick", "input_name(event, this);");

// this function is called on every click of input
function input_name(event, name) {

  // new events are added on every click
  $(this).click(function() {/* ? */});
  $(name).keydown(function(event) {/* ? */}); 
}

所以做这样的事情：

// on click, input_name is called
input.setAttribute("onclick", "input_name(event, this);");

// events are added once
$(input).click(function() {/* ? */});
$(input).keydown(function(event) {/* ? */}); 

// this function is called on every click of input
function input_name(event, name) {
  /* ? */
}

还会考虑使用$().click创建onclick的原因以及input.setAttribute("onclick", ...)因为你有jQuery，所以更喜欢使用$().click来设置属性。

功能被多次调用？

1 个答案: