if (document.formular.garnitur.checked == true) --> ERROR

[EKKi]

Mahlzeit Shadow89,

function berechnung() {
var preis = 0;
}

Diese Funktion ist immer noch sinnlos - informiere Dich über die <http://de.selfhtml.org/javascript/sprache/variablen.htm@title=Gültigkeitsbereiche von Variablen>.

{
if (document.formular.garnitur[0].checked == true) {
    preis = preis + 208;
  } else if (document.formular.garnitur[1].checked == true) {
    preis = preis + 0;
  }
   alert(preis);
}

Was sollen hier die geschweiften Klammern? Weiterhin ist es unnötig abzufragen, ob etwas Wahres auch wahr ist: die Eigenschaft http://de.selfhtml.org/javascript/objekte/elements.htm#checked@title=checked liefert bereits true oder false zurück, Du musst das nicht nochmal explizit auf Gleichheit mit true überprüfen. Folgendes würde reichen:

if (document.formular.garnitur[0].checked) {  
  ...  
}

MfG,
EKKi