Hallo Markus,
Der else Block ist etwas falsch, aber das ist nicht das Problem.
ach was? Wie meinst du das nun?
Hier nochmals die funktion:
Ich stelle fest, dass du viermal getElementById() mit demselben Argument aufrufst. Dein Script durchsucht also viermal den Elementbaum nach demselben Element. Das kann man optimieren, indem man die Referenz, die man beim ersten Aufruf bekommt, in einer Variablen speichert:
function validate(id) {
if ((item=document.getElementById(id)) != null) {
if (isNaN(item.value)) {
item.value = -1;
item.focus();
}
} else {
alert("element is null!");
}
}
Ansonsten verstehe ich nicht, was dir an dem else-Zweig noch nicht gefällt.
So long,
Martin
--
Wer im Steinhaus sitzt, soll nicht mit Gläsern werfen.