50

这是我的函数,它应该改变 HTML 输入的 onClick 属性,但如果我使用

document.getElementById('buttonLED'+id).onclick = "writeLED(1,1)";

它根本不起作用,但如果我使用

document.getElementById('buttonLED'+id).onclick = writeLED(1,1);

该函数自行执行!任何想法在单击按钮之前,我必须使用什么代码来更改 onCLick 属性而不执行该函数?
如果重要的话,这是完整的功能:

function showLED(id){
    if(color == 0){
        document.getElementById('buttonLED'+id).onclick = "writeLED(1,1)";
        document.getElementById('buttonLED'+id).value="light is on";
        //document.getElementById('buttonLED'+id).disabled = false;
    }else{
        document.getElementById('buttonLED'+id).onclick = "writeLED(1,0)";
        document.getElementById('buttonLED'+id).value="light is off";
        //document.getElementById('buttonLED'+id).disabled = false;
    }
}
4

6 回答 6

96

好吧,只要这样做,你的问题就解决了:

document.getElementById('buttonLED'+id).setAttribute('onclick','writeLED(1,1)')

祝你有美好的一天XD

于 2014-05-01T17:17:12.283 回答
47

您想要这样做 - 设置一个将被执行以响应 onclick 事件的函数:

document.getElementById('buttonLED'+id).onclick = function(){ writeLED(1,1); } ;

你正在做的事情不起作用,因为:

  1. onclick 事件处理程序期望有一个函数,在这里您要分配一个字符串

    document.getElementById('buttonLED'+id).onclick = "writeLED(1,1)";
    
  2. 在此,您将执行 writeLED(1,1) 函数的结果分配为 ​​onclick 事件处理程序:

    document.getElementById('buttonLED'+id).onclick = writeLED(1,1);
    
于 2013-02-26T19:12:56.297 回答
3

该行 onclick = writeLED(1,1)表示您要立即执行带有参数 1, 1 的函数 writeLED(arg1, arg2) 并分配返回值;您需要创建一个将使用这些参数执行并分配它的函数。最上面的答案给出了一个例子 - 另一个是使用 bind() 函数,如下所示:

    var writeLEDWithSpecifiedArguments = writeLED.bind(this, 1,1);
    document.getElementById('buttonLED'+id).onclick = writeLEDWithSpecifiedArguments;
于 2015-07-02T18:21:14.303 回答
1

使用 Jquery 而不是 Javascript,使用'attr'属性而不是'setAttribute'

$('buttonLED'+id).attr('onclick','writeLED(1,1)')
于 2018-03-15T08:00:18.953 回答
0

您实际上并没有更改功能。

onClick被分配给一个函数(这是对某物的引用,在这种情况下是一个函数指针)。传递给它的值无关紧要,不能以任何方式使用。

另一个问题是您的变量color似乎无处不在。

理想情况下,你应该把这个逻辑放在函数内部,让它弄清楚要写什么。(开/关等)

于 2013-02-26T19:15:16.533 回答
0

另一种解决方案是将“onclick”属性设置为返回 writeLED 函数的函数。

document.getElementById('buttonLED'+id).onclick = function(){ return writeLED(1,1)};

当您在 JavaScript 中创建尚未在浏览器中绘制的元素时,这对于其他情况也很有用。

于 2019-11-05T11:08:17.457 回答