preg_match Überprüfung funktioniert nur mit bestimmten Formularfeldern

[TDR]

Hallo zusammen,

ich habe ein einfaches kleines Formular, bei denen die Felder in eine CSV-Datei geschrieben werden sollen. Das funktioniert auch alles super. Bestimmte Felder des Formulars möchte ich mit preg_match() auf ihre Gültigkeit überprüfen. Dabei soll, nach meiner Logik, Feld für Feld überprüft werden, ob das jeweilige Feld entweder korrekte Zeichen enhält oder leer ist.

Nun hab ich aber einige Probleme. Es kommt immer eine Fehlermeldung, wenn:

- nur "prename" leer ist
- nur "name" leer ist
- nur "prename" und "name" leer sind
- nur "prename" und "email" leer sind
- nur "prename" und "email" leer sind
- nur "prename" und "address" leer sind
- nur "name" und "email" leer sind
- nur "name" und "address" leer sind

- nur "title" und ein sonstiges Feld ausgefüllt sind
- nur "title" ausgefüllt ist

Bei den restlichen Fällen funktioniert alles wie es soll (Ich denke jedenfalls, dass ich jeden Fall ausprobiert habe).

Ich persönlich nehme an, dass die Zeile "title" Probleme macht, aber selbst beim Ausklammern eben dieser Zeile, treten die selben Probleme auf.

Hat jemand eine Idee?

Hier der entsprechende Code:

if (isset($_POST['save'])) {
    if (preg_match("#^[a-zA-ZäÄöÖüÜß]+$#",$_POST["address"]) or $_POST["address"]=="") {
        if (preg_match("#^[a-zA-ZäÄöÖüÜß]+$#",$_POST["title"]) or $_POST["title"]=="") {
            if (preg_match("#^[a-zA-ZäÄöÖüÜß]+$#",$_POST["name"]) or $_POST["title"]=="") {
                if (preg_match("#^[a-zA-ZäÄöÖüÜß]+$#",$_POST["prename"]) or $_POST["title"]=="") {
                    if (preg_match("#^[a-zA-Z0-9-_.]+@[a-zA-Z0-9-_.]+\.[a-zA-Z]{2,4}$#", $_POST["email"]) or $_POST["email"]=="") {
                        save_data();}

                    else {echo $error;}
                }
                else {echo $error;}
            }
            else {echo $error;}
        }
        else {echo $error;}
    }
    else {echo $error;}
} 

Danke

[hausl]

Puuh, schau dir dringend mal das Wächter Prinzip an und/oder mach eine oder mehr Funktionen draus.
http://php-de.github.io/jumpto/waechter/

Oder Arbeite mit Arrays etc... Ich glaube diese Variante ist der worst Case.

[TDR]

Hey,

danke für den Link. Sehr interessant.

Aber gibt er mir dann nicht mehrere Ausgaben aus, wenn ich den Code nicht so verschachtelt schreibe? Also wenn mehrere Angaben ungültig sind, würde doch dann für jedes Mal der error-String ausgeworfen werden oder verstehe ich da jetzt was falsch?

LG


EDIT: Wenn ich alles in Funktionen aufteilen, bleiben die Probleme. Für mich irgendwie logisch, weil der Ablauf im Grund ja nicht geändert wird.

EDIT2: Ah Stichwort ist in den Funktionen return. So kann ich die Funktionien unterbrechen, wenn ich möchte.
Darüber hinaus habe ich im verschachtelten Code meinen Fehler gefunden. Bei der Abfrage, ob die Felder leer sind, habe ich drei mal $_POST["title"]. Daher die Fehler. Zu lange auf Code starren macht echt blind. Kann geschlossen oder erledigt werden. Danke.

[VPh]

Kommt drauf an wie dein Anwendungsfall aussehen soll.
Wenn nach dem ersten Fehler Ende sein soll, musst du eben die entsprechende Meldung ausgeben und danach abbrechen / Verhindern dass die anderen Prüfungen durchgeführt werden.

[jspit]

Beim ersten Fehler zu beenden ist nicht so nutzerfreundlich. Besser ist alles zu prüfen und die Fehler zu sammeln (z.B. in einem Array).

Allgemein kann ich nur empfehlen mal nach 'Form Validation class php' zu suchen um sich dort einige Anregungen zu holen.

LG jspit

[mermshaus]

Am Rande: Ich würde auch mal darüber nachdenken, auf UTF-8 umzusteigen.

(Witzig, was das derzeit wieder für ein riesiges Thema hier im Forum ist. Eigentlich sollte man annehmen, dass sich derlei Dinge irgendwann erledigt haben.)