ändert onClick mit Javascript-Attribute

Question

Diese meine Funktion ist, und es sollte das onClick-Attribut der HTML-Eingabe ändern, aber wenn ich

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

es verwende gar nicht arbeiten, aber wenn ich

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

Die Funktion wird von selbst ausgeführt! Irgendwelche Ideen, welchen Code muss ich verwenden, um das onCLick-Attribut zu ändern, OHNE die Funktion auszuführen, bevor auf die Schaltfläche geklickt wird?
Hier ist die volle Funktion, wenn es darauf ankommt:

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; 
    } 
} 

Answer 1

Sie wollen, dies zu tun - eine Funktion festgelegt, die das Onclick-Ereignis reagieren ausgeführt werden:

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

Die Dinge, die Sie sind tut nicht funktionieren, weil:

1. Der Onclick-Ereignishandler, eine Funktion zu haben erwartet, hier zuweisen Sie einen String

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

2. Dabei zuweisen Sie als OnClick Ereignishandler das Ergebnis der writeLED (1,1) Funktion ausführt:

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

Answer 2

Sie sind eigentlich nicht die Funktion zu verändern.

onClick ist einer Funktion zugewiesen (Dies ist ein Verweis auf etwas, in diesem Fall ein Funktionszeiger). Die Werte, die daran übergeben werden, sind nicht wichtig und kann nicht in irgendeiner Weise verwendet werden. Ein anderes Problem ist Ihre Variable color scheint aus dem Nichts.

Idealerweise sollten Sie innerhalb der Funktion diese Logik setzen und herausfinden, was geschrieben werden soll. (On/off etc etc)

Answer 3

Nun, tun nur dies und das Problem ist gelöst:

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

Haben Sie einen schönen Tag XD

Answer 4

Die Linie onclick = writeLED(1,1) bedeutet, dass Sie wollen sofort ausführen die Funktion writeLED (arg1, arg2) mit den Argumenten 1, 1 und den Rückgabewert zuweisen; Sie müssen stattdessen eine Funktion erstellen, die mit diesen Argumenten ausgeführt wird und diese zuweisen. Die oberste Antwort ein Beispiel gab - eine andere ist es, die Funktion bind() wie so zu verwenden:

var writeLEDWithSpecifiedArguments = writeLED.bind(this, 1,1); 
    document.getElementById('buttonLED'+id).onclick = writeLEDWithSpecifiedArguments;