Don P: Fehlermeldung: function not defined

Beitrag lesen

Hallo,

ich lerne gerade js.

Schön. Dann fange am besten am Anfang an. Lerne z.B. erst mal, wie man lesbaren Code schreibt, mit vernünftigen Einrückungen und so. Deine (übrigens fehlerhafte) Funktion sollte etwa so ntiert werden:

function hideshow(divid){  
  
    var collection = window.document.getElementsByTagName('div');  
  
    for (var x=0; x<collection.length; x++) {  
  
        if (collection[x].id.match(/slide/gi) AND collection[x].id.length > 0 AND collection[x].id != divid) {  
  
            var divhide = "div#" + collection[x].id;  
            $(divhide).hide();  
  
        } else if (collection[x].id = divid) {  
  
            var divshow = "div#" + divid;  
            $(divshow).slideDown("slow");  
        }  
    }  
}  
  

Wenn du im Eingabefeld deines Postings den Code markierst und den "JavaScript"-Button klickst, wird er nachher so schön bunt wie hier. Auch das erleichtert die Lesbarkeit sehr.

if (collection[x].id.match(/slide/gi) AND collection[x].id.length > 0 AND collection[x].id != divid) {

Ein Operator namens "AND" existiert in JS nicht.

jquery wurde im head eingebunden

Als JS-Anfänger würde ich nicht direkt mit jquery beginnen.

es kommt im firefox die fehlermeldung "function hideshow not defined". woran kann das liegen?

Das ist vermutlich ein scope-Problem.

Gruß, Don P