Was ist falsch?

[Maus]

Hallo,

kann mir mal jemand sagen, was ich hier falsch gemacht habe:

<script language="JavaScript" type="text/JavaScript">
<!--
 function CheckInput() {
    var forentext   = document.getElementById('titel').value;

if (forentext == "")
    {
        alert("Du musst eine Betreff eingeben!");
        return false;
    }
 var forentext   = document.getElementById('post').value;

if (forentext == "")
    {
        alert("Du musst eine Nachricht eingeben!");
        return false;
    }
    if (Error = check_message(forentext))
    {
        alert(Error);
        return false;
    }
}
//-->
</script>

Es geht nicht :-(

Gruß Maus

[Struppi]

Es geht nicht :-(

Aha?
Wie wär's mit einer Fehlermeldung?
Oder einem Beispeil das nicht funktioniert, das was wir sehen, sollte ausgeführt werden, wenn alle Elemente mit den IDs vorhanden sind und diese alle eine Eigenschaft value haben. Aber du prüfst das auch nicht nach.

Struppi.

[Maus]

Hallo,

Wie wär's mit einer Fehlermeldung?

ja wenn ich das wüsste, dann währe ich schon schlauer, es passiert nichts, ich kann auf abschicken klicken und er macht es es kommt aber keine Meldung, dass ich noch was nichtd ausgefüllt habe.

Aber du prüfst das auch nicht nach.

hmm was meinst du denn mit Prüfen was soll ich denn noch prüfen?

Gruß Maus

[Struppi]

Wie wär's mit einer Fehlermeldung?

ja wenn ich das wüsste, dann währe ich schon schlauer, es passiert nichts, ich kann auf abschicken klicken und er macht es es kommt aber keine Meldung, dass ich noch was nichtd ausgefüllt habe.

Du kennst die JS Fehlerkonsole?

Aber du prüfst das auch nicht nach.

hmm was meinst du denn mit Prüfen was soll ich denn noch prüfen?

Die Elemente die du verwendest.

Struppi.

[FraFu]

Hallo!

"Geht nicht" ist keine Fehlerbeschreibung. Was sagt die JS Konsole?
Wie wird CheckInput() aufgerufen?
CheckInput() liefert nirgends true zurück.

mfg
  frafu

[Mort]

füge hinter

if (Error = check_message(forentext))
{
    alert(Error);
    return false
}

mal einem else-Zweig mit "return true;" ein.
wenn die funktion in einem formular verwendet werden soll muss ein bool- wert zurückgegeben werden sonst meint das formular es gibt false zurück und schickt die daten nicht ab.

Und ist die Zuweisung in der if-klausel überhaupt erwünscht? wenn es wirklich eine zuweisung ist, mach sie lieber vorher. und prüf dann erst "Error"

und wenn du dir das alert sparen kannst schreib doch einfach

return !check_message(forentext);