Gernot Back: Dynamische Button-Events mit korrektem Bezug

Beitrag lesen

Hallo NetizenKane,

Ich glaube ja, dass der Fehler gar nicht so sehr in deinem Code liegt, als vielmehr in dem, den du dir angeeignet hast

In der Funktion colorPicker() steht ja z.B. sowas:

<a class='o5582n66' href='javascript:onclick=putOBJxColor2("+xl+")'>

Was das "onclick=" nach dem "javascript:" zu suchen hat, ist mir schleierhaft. Auch sonst scheint mir das Script nicht ganz koscher, es arbeitet viel zu sehr mit Strings, über die dann mit getElementById() wieder auf Objekte zugegriffen wird.

Warum nicht gleich die Objekte, den Button und sein Anzeigefeld miteinander bekannt machen, sich gegenseitig in Eigenschaften als Referenzen verpassen und dann an die entsprechend zu modifizierende colorPicker-Funktion die entsprechnden Objekte übergeben, die in diese dann auch zurückschreibt, anstelle von Strings, die dann wieder über getElementById Objekte ansprechen. Das ist so alles andere als objektorientiert und von hinten durch die Brust ins Auge.

Gruß Gernot