JavaScript: getElementsByClassName funktioniert nicht

Friedel

Hallo,
ich verzweifle gerade mal wieder, weil etwas ganz einfaches einfach nicht funktionieren will. Ich habe eine Beispielseite angelegt, um das Problem zu demonstrieren:
http://www.friedels-home.de/JavaScripte/Gloweffekt/glow.htm

Wie ihr sehen könnt, gibt es 4 Elemente mit

Code

class="dbh"

In der glow.js steht:

HTML

function glow() {
  var bereiche = document.getElementsByClassName("dbh");
  alert(bereiche.length);
};


window.onload = glow();

Sollte die Alertbox nicht 4 ausgeben? Sie gibt aber 0 aus! Wo ist mein Fehler?

djheke

Versuch mal dein Script im Body aufzurufen.

Code

<body onload="glow();">

Friedel

??? Wozu? Warum sollte ich in eine valide Seite einen Fehler einbauen? Die Funktion wird völlig korrekt durch

Code

window.onload = glow();

aufgerufen und das funktioniert ja auch. Die Alertbox kommt ja. Aber warum ist der Wert von

Code

document.getElementsByClassName("dbh")

falsch?

- - - Aktualisiert - - -

Grrr. Mit

Code

<body onload="glow();">

geht es. Wenn ich die glow() mit

Code

var delay = setTimeout("glow()", 200);

aufrufe geht es auch. Danke. Ursache gefunden.

- - - Aktualisiert - - -

Und hier die Auflösung für alle, die den Thread wegen ähnlicher Probleme finden:

Code

window.onload = glow();

ist nicht korrekt. window.onload erwartet nur einen Funktionsnamen! Man kann nicht nur keine Parameter übergeben, man darf auch die Klammer für die Parameter nicht eintragen. Richtig ist also

Code

window.onload = glow;

Und damit gibt die Alertbox auch 4 aus, wie es sein soll.

Arne Drews

Noch cleverer ist es, wenn Du es richtig verwendest:

Code

window.onload = function() {


    glow();
    glow( 123 );
    // ...and many many many code more, what ever you want...


};

Friedel

??? Und warum soll nur das richtig sein? Was soll an

Code

window.onload = glow;

falsch sein?

Arne Drews

Wer hat was von falsch gesagt?

Fakt ist: Die Art und Weise aus #4 ist die, nach der es gedacht ist.
Ansonsten wäre ja nur ein Aufruf möglich, wie Du selber festgestellt hast. Welchen Vorteil sollte das also haben?
Wenn Du zwei Dinge nach dem OnLoad verarbeiten willst, müsstest Du entweder mehrfach window.onload bemühen oder alles in der aufgerufenen Funktion ( hier: glow ) abarbeiten.
Wozu aber eine neue Funktion, wenn das Event-Objekt in Javascript dies bereits vorsieht?!

Das meine ich mit richtig machen...
Du musst die Antworten anderer nicht umdrehen, um recht zu behalten.

Friedel

Ich drehe nicht rum, sondern ich frage nach, wie es gemeint ist. Jetzt weiß ich, dass du nicht Recht hast und brauche mich nicht mehr dafür zu interessieren.

Über deinen letzten Satz solltest du imho mal nachdenken.

Arne Drews

Zitat

Über deinen letzten Satz solltest du imho mal nachdenken.

Habe ich, bevor ich ihn geschrieben habe... Das unterscheidet uns...

modem-kind

ich find das einleuchtend ! ..und fantastisch Erklärt

window.onload = function() {

glow();
glow( 123 );
// ...and many many many code more, what ever you want...

};

Danke
und schönes Wochenende @all

JavaScript: getElementsByClassName funktioniert nicht

Tags

Benutzer online in diesem Thema