Mail-Versand abbrechen / Fehlermeldung

[Wolfgang G]

Aloha Gemeinde,

das folgende Mail-Script versendet korrekt. Nun will ich jedoch verhindern, dass ein Scherzbold meine Datei "mailer.php" direkt aufruft und da 100 mal auf F5 drückt und ich 100 leere Mails erhalte.

Mein Ansatz ist nun der Folgende:

PHP-Code:


<?php
$mymail = "Wolfgang-g@gmx.de";
$admin = "Wolfgang Giersberg";
$subject = "$Betreff";
$message = "Nachricht fuer $admin von $eMail:\n\n";

if(empty($_POST['Name']) || empty($_POST['eMail'])) 
{
    echo "Bitte geben Sie Ihren Namen und Ihre eMail-Adresse an";
    break;
}

while(list($name,$value)=each($_POST)) 
{
    if($name == "Betreff")
    continue;
    elseif($name == "Submit")
    break;
    $message.="$name: $value\n---------------\n\n";
}

mail($mymail,$subject,$message,"From: $eMail");

?>

Der Server (Apache 1.3.28, PHP 4.32) reagiert auf den direkten Aufruf mit der Ausgabe meines Textes (Bitte geben Sie...), jedoch kommt danach:

Fatal error: Cannot break/continue 1 level in /www/htdocs/*mein Kundenlogin* /mailer.php on line 10 .

Line 10 ist das "break" in der ersten if-Anweisung. Wieso kann er nicht abbrechen?

Wolfgang

Ach ja: Wg. Hinweise auf die Suchfunktion. Wenn ich da "cannot break" oder cannot AND break eingebe, bekomme ich 122 Treffer in denen cannot break nicht vorkommt, sondern nur break

[steff]

laut http://at.php.net/manual/de/control-...ures.break.php

Break bricht die Ausführung der aktuellen for, foreach while, do..while or switch Anweisungs-Sequenz ab.

gehts mit return false; ?

ich nutze if-strukturen meist mit else, um eine reaktion auszulösen. hab noch nie mit break bei if gearbeitet. bin aber auch kein php-spezie...

[holunda]

Bau einfach wies steff schon gesagt hat nach dem if ein else ein und da drinnen fährst du dann die while anweisung ... dann klappts auch mit dem nachbarn

[Wolfgang G]

Hm, jetzt habe ich eine else-Anweisung eingefügt und die Abfrage sieht jetzt so aus:

PHP-Code:


if(empty($_POST['Name']) || empty($_POST['eMail'])) 
{
    echo "Bitte geben Sie Ihren Namen und Ihre eMail-Adresse an";
}

else
{
    while(list($name,$value)=each($_POST)) 
    {
        if($name == "Betreff")
        continue;
        elseif($name == "Submit")
        break;
        $message.="$name: $value\n---------------\n\n";
    }
}

mail($mymail,$subject,$message,"From: $eMail"); 


Die Fehlermeldung kommt jetzt nicht mehr, der Hinweistext erscheint, jedoch bekomme ich nun wieder pro F5 eine Leermail zugestellt. D.h. er arbeitet beide Anweisungen ab, egal welchen Wert die erste if-Abfrage liefert.

Und was hat es mit "return" auf sich. Finde das weder in SelfPHP noch in meinem Buch "Einsteigerseminar PHP 4".

Wolfgang

[steff]

leite den user nach dem mail-versand weiter

PHP-Code:

//entweder nach mail()
header("Location: irgendeine andere seite.php");
exit;

// oder wenn ausgabe vorher erfolgt
echo "<meta http-equiv=\"refresh\" content=\"0; url: blabla.php\">"; 


[Wolfgang G]

Die Weiterleitung auf eine "Danke für Ihr Interesse"-Seite kommt später.

Im Moment geht's halt noch darum, den Leerversand komplett zu unterbinden, also dafür zu sorgen, dass wenn die if-Anweisung wahr ist, die else-Anweisung nicht ausgeführt wird.

Dachte eigentlich, das sei standardmäßig so. Also nach dem Motto:

if Bedingung1 erfüllt
mach dies;

in allen anderen Fällen (außer Bedingung1 erfüllt) (else)
mach jenes;

Es scheint jedoch so zu sein:

if Bedingung1 erfüllt
mach dies;

in jedem Fall (egal ob Bedingung1 erfüllt oder nicht) (else)
mach jenes;


Immer verwirrter ...

Wolfgang

[holunda]

wenn du so abfrägst if ($_POST['var']) ist es nach einem F5 auch noch true, ergo wird wieder versendet.

Baue die WEiterleitung ein. Du kannst auch den header auf das mailscript selbst setzen -> danach ist das $_POST gelöscht.

alex

[Wolfgang G]

Manchmal hab ich echt nen Knoten im Hirn!

Ich habe die ganze Zeit die mail-Funkton außerhalb meiner if/else-Konstruktion aufgerufen!!!

Da ist es ja nur logisch, dass versendet wird - unabhängig von der Abfrage davor.

Jetzt steht der mail-Aufruf innerhalb der else-Anweisung und das Script verhält sich wie erwartet. Bei leeren Feldern kommt der Hinweistext und es wird keine Mail versandt. HALLELUJA!!!

Jetzt kann ich mich der Weiterleitung zuwenden ...

Wolfgang