Hallo wahsaga,
if (document.Formular.E-Mail.value.IndexOf ("@" == -1) {
Good point :-)
das - als der Minus-Operator erkannt - es würde also versucht, von document.Formular.E den Wert von Mail.value.IndexOf abzuziehen.
Aber ich würde dir eher raten, gleich den Feldnamen in einen "javascript-kompatiblen" zu ändern - bspw. name="EMail" - um möglichen weiteren zukünftigen Problemen dieser Art von vornherein aus dem Weg zu gehen.
Abgesehen von dem Minus-Operator gibt es in dieser Zeile auch zwei öffnende bei nur einer schließenden runden Klammer.
Gruß Gernot