script funzt nur mit wert "0"?!

[theo]

moin moin,

seit gestern suche ich nach dem fehler in folgendem script:

function galSelect() {
  var sel = document.forms[0].categories.value;
  for(var i=1; i<=document.forms[0].categories.length; i++) {
    document.getElementById('gal'+i).style.display = (sel == i) ? 'block' : 'none';
    alert('sel '+sel+'\ni '+i);
  }
}

mit dem augangswert "0" funzt es. alle anderen werte werden (meiner ansicht nach im if-construct) nicht korrekt verglichen.
ueber den alert sehe ich aber, dass min. ein wert mit der vorgabe uebereinstimmt. d.h. eine der galerien muesste angezeigt werden. sind aber alle auf display=none. bei "0" wird die erste allerdings korrekt angezeigt.
ich weiss einfach nicht mehr weiter. hat von euch jemand ne idee?

gruss
hardy

[Nightline]

Bißchen schwer ohne weiteren Code, mit dem man die Funktion testen kann!
Aber warum fängst du in der for-Schleife mit i = 1 statt i = 0 an?

[theo]

ok, stimmt. war jetzt ein bisschen verwirrend.
ich hatte frueher in der schleife "0" als ersten wert stehen. aber da ich die div´s erst ab "1" beginnend beschriftet habe, hab ich die for-schleife auch mit 1 starten lassen. ich haette sonst ueberall +1 rechnen muessen.

zum code: ich habe einen selector, der dyn. per php aus einer db erzeugt wird. in dem ding stehen objektgruppen. zu jeder gruppe gibt es eine anzahl bilder. wenn ich eine gruppe auswaehle, dann soll die entsprechende "galerie" angezeigt werden. alle anderen werden ausgeblendet (jede galerie steht in einem eigenen div mit eigener id).
da das script auf unterschiedliche dateien und verschiedene sprachen/codes verteilt ist, kann ich hier schwer einfach phragmente posten. die tragen nicht wirklich zur verstaendlichkeit bei.

aber reicht die erklaerung erstmal?

[Malleus]

Hi Hardy,
besuch' mal folgendes:

function galSelect() {
var sel = 1 * document.forms[0].categories.value; 
// Den Value in eine Zahl umwandeln !

Du vergleichst einen String "1" mit der Zahl 1, vielleicht war's ja das

[theo]

juhuu ... einmal war ich schon schneller

das war meine erste vermutung und darum hatte ich bereits folgendes probiert:

function galSelect() {
  var sel = document.forms[0].categories.value;
  for(var i=1; i<=document.forms[0].categories.length; i++) {
    document.getElementById('gal'+i).style.display = ((sel*1) == (i*1)) ? 'block' : 'none';
    alert('sel '+sel+'\ni '+i);
  }
}

aber es will einfach nicht ...

[Malleus]

Kannst Du mal das "generierte" posten bzw. online stellen?

[theo]

ok ... mein fehler! ich fasse es nicht. der fehler lag nicht im js sondern im php. hatte einen schreibfehler in einer variable und darum wurde das erste div nie geschlossen. darum klappte es nur mit div1.

sorry ... und danke fuers helfen. wenn ich den quellcode nicht haette posten wollen, dann waere ich nie dahinter gekommen.

[Malleus]

Alles klar, bis zum nächsten Mal