Hi,
Lese ich Eure Ratschläge ned richtig oder denke ich zu einfach, bzw. kompliziert?
wahrscheinlich von allem etwas ... ;-)
<script language="JavaScript">
Hier fehlt das vorgeschriebene Attribut type="text/javascript", während dein language-Attribut als veraltet gilt. Aber das nur als formaler Hinweis nebenbei, das hat nichts mit deinem eigentlichen Problem zu tun.
var anz1;
anz1=parseInt(document.kontakt.anz1.value);
Das kannste zusammenziehen:
var anz1 = parseInt(document.kontakt.anz1.value);
var preis1;
var preis2;
var ts1;
var ts2;
var totall = ts1 + ts2;
Hier bildest du eine Summe aus zwei Variablen, die nicht initialisiert sind (keinen Wert erhalten haben). Die Summe totall ist damit auch völlig undefiniert.
var nn;
var vk;
var totalnn = totall + nn;
var totalvk = totall + vk;
Hier dasselbe in Grün.
}
Zu dem Dilemma mit den Klammern hat Sven dir ja schon einen Wink mit der Eisenbahnschwelle gegeben, nachdem du den Zaunpfahl nicht gesehen hast.
<select class="text" name="anz1" onchange="changed(this);">
Auch hier möchte ich eine Frage nochmal aufgreifen, die Sven schon angedeutet hat: Warum übergibst du hier eine Referenz auf das auslösende Element (this), wenn du sie in der Funktion gar nicht verwendest?
<input value="0" name="ts1" size="7" onchange="changed(this);" align="right">
Dito.
Und wenn das Gerüst mal soweit steht, dass es richtig funktioniert, würde ich dir empfehlen, dich etwas mit Gestaltung durch CSS zu befassen. Dein HTML-Quellcode ist ein ziemliches Kuddelmuddel.
So long,
Martin
Das einzige Problem beim Nichtstun: Man weiß nie, wann man damit fertig ist.