function CheckSubmit()

[jaylibii]

Guten Morgen,

ich stehe mal wieder vor einem Problem. Und zwar möchte ich gerne eine FUnktion haben, dass JavaSkript mir was überprüft ob in dem Feld was steht. Das habe ich nun so gemacht:

<script language="javaScript">

function CheckSubmit()
if (document.post.post.value == "") {
    alert("Du musst schon ein Text eingeben!");
    document.post.post.focus();
    return false;
  }
</script>

Das hoffe ich doch mal, dass das Richtig ist. Aber wenn ich nun auf absenden klicke, dann passiert einfach gar nichts. Es kommt nicht mal eine Meldung, dass ich was eingeben muss.

So sieht dann das Formular aus:

<form name="form1" method="post" action="antworten.php?id=<?php echo $id;?>" onsubmit="return CheckSubmit()">
                                                                <table width="100%"  border="0">
                                                                  <tr>
                                                                    <td align="right" valign="top"><p><b>Text:</b> </p></td>
                                                                    <td valign="top"><p>
                                                                        <textarea name="post" cols="80" rows="20" wrap="physical" class="chatfields_tall" id="post"></textarea>
                                                                    </p></td>
                                                                  </tr>
                                                                  <tr>
                                                                    <td width="86%" height="34" colspan="2"><div align="right">
                                                                      <input name=absenden type=submit class="chatfields" id="absenden" value=Abschicken>

Ich bin nun mal gespannt was Ihr dazu sagt und wo der Fehler liegt.

MFG jaylibii

[Rainer]

if (document.form1.post.value == "") {

So müsste es gehen ;-)

Gruss Rainer

[mark chopin]

[code=html]
  <form name="form1" method="post" action="antworten.php?id=<?=$id?>" onsubmit="CheckSubmit();">
[/code]

Und so noch besser.

[Struppi]

<form name="form1" method="post" action="antworten.php?id=<?=$id?>" onsubmit="CheckSubmit();">

Und so noch besser.

Quatsch!

so:
<form method="post" action="..." onsubmit="return CheckSubmit(this)">

  
<script type="text/javaScript">  
  
function CheckSubmit(form)  
{  
if (!form.post.value) {  
    alert("Du musst schon ein Text eingeben!");  
    form.post.focus();  
    return false;  
}  
return true;  
}  
</script>  

Struppi.

[Tobias Kloth]

Hallo Mark,

<form name="form1" method="post" action="antworten.php?id=[code lang=html]<?=$id?>

> [/code]  
> Und so noch besser.  

Unsinn - nur weil du den Code von der Einstellung short\_open\_tag abhängig macht, ist er noch lange nicht besser. Das vom OP verwendete `<?php echo $id;?>`{:.language-php} ist völlig in Ordung.  
  
Grüße aus Nürnberg  
Tobias

[Struppi]

Und so noch besser.
Unsinn - nur weil du den Code von der Einstellung short_open_tag abhängig macht, ist er noch lange nicht besser. Das vom OP verwendete <?php echo $id;?> ist völlig in Ordung.

Das Problem ist das return.
Mark hat das return vor der JS Funktion rausgeschmissen und das ist absolut notwendig.

Struppi.

[Tobias Kloth]

Hallo Struppi,

Das Problem ist das return.
Mark hat das return vor der JS Funktion rausgeschmissen und das ist absolut notwendig.

stimmt, das hatte ich übersehen - aber auch die Kurzschreibweise (<?=$foo?>) sollte man trotzdem nicht verwenden.

Grüße aus Nürnberg
Tobias