✔ Eigene Funktion getElementsByClassName funktioniert findet zu viele Elemente

[timestamp]

Hallo Forum

ich stehe hier for einem Rätsel. Ich habe mir eine Funktion geschrieben um HTML Objekte mit bestimmtem Klassennamen zu finden:

function getElementsByClassName(class){
  elements = document.getElementsByTagName("*");
  returnArray = new Array();
  for (i = 0; i < elements.length; i++) {
    if( elements[i].className == class){
      returnArray.push(elements[i]);
    }
  }
  return returnArray;
}

Das Problem dabei ist, dass ich nur einen div mit entsprechender Klasse habe, dieser aber zweimal gefunden wird. Wie kann ich den Fehler beheben?

 

[Sven Mintel]

Moin,

das kann ich nicht nachvollziehen, bei mir liefert dies 1 Element, wenn nur eins da ist.

Das einzig fehlerhafte am Skript ist die Nutzung des Variablennamens "class"...dies ist ein geschützer Name in JS, mein IE6 honoriert dies sogar damit, dass er das Skript nicht ausführt.

Gut wäre auch noch, wenn du die Variablen per var deklarierst, damit sie nicht global sichtbar sind....dies könnte zu Wechselwirkungen führen.

 

[timestamp]

Hmm

ich habe es jetzt auch noch einmal getestet und die Funktion funktioniert einwandfrei. Der Fehler muss also wohl woander liegen, danke

 

[Matthias Reitinger]

Hallo,

einige Browser unterstützen diese Funktion auch schon von Haus aus. Eine robuste Implementierung, die das berücksichtigt und als Fallback eine einfache Schleife verwendet (wie bei deinem Code), findet man z.B. unter http://code.google.com/p/getelementsbyclassname/

Grüße,
Matthias

 

[hela]

Hallo,
bei mir hat bisher folgendes prima funktioniert, da auch multiple Klassen berücksichtigt werden:

function getElementsByClassName(class_name){
  var returnArray = new Array(), i, j=0, testStr;
  var allElements = document.getElementsByTagName("*");

  for (i = 0; i < allElements.length; i++) {
    if(allElements[i].className.indexOf(class_name) > -1){
      testStr = ","+allElements[i].className.split(" ").join(",")+",";
      if(testStr.indexOf(","+class_name+",") > -1) {
        returnArray[j] = allElements[i];
        j++;
      }
    }
  }
  return returnArray;
}