PHP Fehlermeldungen Anzeigen, um-und re-display die richtigen Felder

Habe ich eine E-Mail-Formular, das überprüft, drei Felder, Namen, gültige E-Mail und Kommentare. Aber so, wie es jetzt eingerichtet, da die Namen und Kommentare sind in einer Funktion, die zunächst prüft, Namen und Kommentare, auch wenn E-Mail nicht gültig ist, wie kann ich wieder schreiben Sie so prüft es die Felder in der Reihenfolge. Außerdem würde ich gerne wieder die Felder angezeigt, die keine Fehler aufweisen, so dass die Benutzer nicht haben, um geben Sie wieder. Bitte helfen Sie. Dank

<?php
$myemail = "[email protected]";
$yourname = check_input($_POST['yourname'], "Enter your name!");
$email = check_input($_POST['email']);
$phone = check_input($_POST['phone']);
$subject = check_input($_POST['subject']);
$comments = check_input($_POST['comments'], "Write your comments!");

if (!preg_match("/([\w\-]+\@[\w\-]+\.[\w\-]+)/", $email))
  {
    show_error("Enter a valid E-mail address!");
  }

exit();

function check_input($data, $problem='')
 {
$data = trim($data);
$data = stripslashes($data);
$data = htmlspecialchars($data);
if ($problem && strlen($data) == 0)
{
    show_error($problem);
}
return $data;
}

function show_error($myError)
{
?>
<!doctype html>
<html>
<body>
<form action="myform.php" method="post">
 <p style="color: red;"><b>Please correct the following error:</b><br />
 <?php echo $myError; ?></p>
 <p>Name: <input type="text" name="yourname" /></P>
 <P>Email: <input type="text" name="email" /></p>
 <P>Phone: <input type="text" name="phone" /></p><br />
 <P>Subject: <input type="text" style="width:75%;" name="subject" /></p>
 <p>Comments:<br />
 <textarea name="comments" rows="10" cols="50" style="width: 100%;"></textarea></p>
 <p><input type="submit" value="Submit"></p>
</form>
</body>
</html>
<?php
exit();
}
?>

Ich versuchte, um die regex in der Funktion check_input aber dann die E-Mail-Fehler zeigten unabhängig.

InformationsquelleAutor Mosaic | 2012-04-18

forms php

5

Zunächst, ich würde vorschlagen, Sie überprüfen ALLE Felder auf einmal, und alle anzeigen werden entsprechende Fehlermeldungen auf dem Formular. Der primäre Grund dafür ist, dass es kann schlecht sein, user experience, wenn Sie das Formular Absenden, eine ganze Reihe von Zeiten, denn Sie müssen Adresse einen Fehler. Ich würde lieber meine richtige E-Mail-Adresse, Passwort, Kommentare und Auswahl in einem versuchen, statt der Fixierung eines in einer Zeit, nur um zu zeigen, was der nächste Fehler ist.

Gesagt, hier sind einige Hinweise auf die Validierung der form, wie Sie wollen. Dies ist in der Regel, wie ich Ansatz einer form zu tun, was Sie tun möchten. Dieser nimmt Ihr HTML-Formular und Formular-Prozessor (PHP) sind zusammen in die gleiche Datei (das ist, was Sie jetzt haben). Sie können den split der beiden, aber die Methoden, dies zu tun, kann ein bisschen anders.
- Haben eine Funktion oder code-block, der die Ausgänge der form und ist sich bewusst, Ihre Fehlermeldungen und hat Zugang zu der bisherigen form-Eingang (wenn vorhanden). Dies kann in der Regel belassen werden außerhalb einer Funktion und kann den letzten code-block, der in Ihrem PHP-Skript.
- Richten Sie ein array für die Fehlermeldungen (z.B. $errors = array()). Wenn das array leer ist, wissen Sie, es wurden keine Fehler mit der Vorlage
- Überprüfen Sie, ob das Formular abgesendet wurde in der Nähe des oben in Ihrem script, bevor das Formular ausgegeben wird.
- Wenn das Formular abgesendet wurde, zu validieren jedes Feld eine zu einem Zeitpunkt, wenn ein Feld enthielt einen Fehler, fügen Sie die Fehlermeldung, um die $errors array (z.B. $errors['password'] = 'Passwords must be at least 8 characters long';)
- Zu re-füllen Sie die Formularfelder mit der vorherigen Werte, speichert man die eingegebenen Werte irgendwo (Sie können entweder verwenden Sie einfach die $_POST array, oder desinfizieren Sie und ordnen Sie die $_POST Werte zu einzelnen Variablen oder ein array.
- Nachdem alle die Bearbeitung abgeschlossen ist, können Sie überprüfen Sie für jeden Fehler, um zu entscheiden, ob das Formular verarbeitet werden kann, an diesem Punkt, oder braucht neue Eingabe durch den Benutzer.
- Um dies zu tun, die ich normalerweise tun, so etwas wie if (sizeof($errors) > 0) { //show messages } else { //process form }
- Wenn Sie das erneute anzeigen des Formulars, müssen Sie einfach fügen Sie eine value="" Attribut für jedes form-element und liefert den Wert, der gesendet wurde, durch den Benutzer. Es ist sehr wichtig, um die Flucht der Ausgabe mit htmlspecialchars() oder ähnliche Funktionen
Mit diesen Dingen im Ort, hier ist einige Nacharbeit Ihrer form zu tun:
```
<?php
$myemail = "[email protected]";
$errors  = array();
$values  = array();
$errmsg  = '';

if ($_SERVER['REQUEST_METHOD'] == 'POST') {
    foreach($_POST as $key => $value) {
        $values[$key] = trim(stripslashes($value)); //basic input filter
    }

    if (check_input($values['yourname']) == false) { 
        $errors['yourname'] = 'Enter your name!';
    }

    if (check_input($values['email']) == false) {
        $errors['email'] = 'Please enter your email address.';
    } else if (!preg_match('/([\w\-]+\@[\w\-]+\.[\w\-]+)/', $values['email'])) {
        $errors['email'] = 'Invalid email address format.';
    }

    if (check_input($values['comments']) == false) {
        $errors['comments'] = 'Write your comments!';
    }

    if (sizeof($errors) == 0) {
        //you can process your for here and redirect or show a success message
        $values = array(); //empty values array
        echo "Form was OK!  Good to process...<br />";
    } else {
        //one or more errors
        foreach($errors as $error) {
            $errmsg .= $error . '<br />';
        }
    }
}

function check_input($input) {
    if (strlen($input) == 0) {
        return false;
    } else {
        //TODO: other checks?

        return true;
    }
}
?>
<!doctype html>
<html>
<body>
<form action="<?php echo $_SERVER['PHP_SELF'] ?>" method="post">
 <?php if ($errmsg != ''): ?>
 <p style="color: red;"><b>Please correct the following errors:</b><br />
 <?php echo $errmsg; ?>
 </p>
 <?php endif; ?>

 <p>Name: <input type="text" name="yourname" value="<?php echo htmlspecialchars(@$values['yourname']) ?>" /></P>
 <P>Email: <input type="text" name="email" value="<?php echo htmlspecialchars(@$values['email']) ?>" /></p>
 <P>Phone: <input type="text" name="phone" value="<?php echo htmlspecialchars(@$values['phone']) ?>"/></p><br />
 <P>Subject: <input type="text" style="width:75%;" name="subject" value="<?php echo htmlspecialchars(@$values['subject']) ?>" /></p>
 <p>Comments:<br />
 <textarea name="comments" rows="10" cols="50" style="width: 100%;"><?php echo htmlspecialchars(@$values['comments']) ?></textarea></p>
 <p><input type="submit" value="Submit"></p>
</form>
</body>
</html>
```
Habe ich eine erweiterte Beispiel, die Sie sehen können hier, dass kann Ihnen einige Hinweise, wie gut.

Hoffe, das hilft.

Funktioniert perfekt! und danke für die ausführliche Erklärung.
Netter avatar ;). Zum Glück verwenden wir häufiger verschiedene Zimmer auf stackoverflow.
@terencehill thumbsup 🙂

InformationsquelleAutor drew010
1

Die einfachste Möglichkeit ist die Verwendung eines form validation-Bibliothek. PHP ist filter - Erweiterung, zum Beispiel, bietet Validierung und-Bereinigung für einige Arten, aber es ist nicht eine komplette Lösung.

Wenn Sie darauf bestehen, über die Umsetzung, es selbst, ein Problem, das Sie haben zu prüfen, ist das, was zählt als Reihenfolge: die Reihenfolge der Elemente in die form oder die Reihenfolge der Benutzereingabe in $_POST. Auf die meisten Browser, diese sollte die gleiche sein, aber es gibt keinen standard, der erzwingt das. Wenn Sie wollen, um zu gehen aus der form, müssen Sie zum definieren der form der Struktur an einer Stelle, und diese Informationen verwenden, um Dinge zu tun wie die Generierung oder Validierung der form (eine Folge der Don ' T Repeat Yourself (TROCKEN) - Prinzip). Iteration über die entsprechende Struktur geben dir den Auftrag, den Sie wünschen: looping über die form gibt, bilden Sie um, während looping über $_POST gibt Sie Benutzereingaben um.

Wie es aussieht, wollen mehr, als nur die Validierung der Daten; Sie wollen auch, um es für die Verwendung vorbereitet, ein Prozess, der als "Bereinigung".

Wenn es um die Bereinigung, definieren verschiedene Arten von Desinfektionsmittel, sondern als eine einzige check_input Funktion. Spezifischen Desinfektionsmittel werden könnte, Funktionen oder Objekte mit einem __invoke Methode. Eine Karte zu erstellen von Formularfeldern für Desinfektionsmittel (zum Beispiel ein array von input name sanitizer Rückrufe). Die Reihenfolge der Elemente in der Zuordnung wird die Anordnung der Bereinigung; wenn Sie eine einzelne Struktur zu definieren, die die form von Informationen, der Anzeige-Reihenfolge und-Bereinigung, um somit die gleiche sein.

Hier ist eine sehr grobe Gliederung:
```
# $fields could be form structure or user input
foreach ($fields as $name => $data) {
    # sanitize dispatches to the appropriate sanitizer for the given field name
    $form->sanitize($name, $data);
    # or:
    //sanitize($name, $data);
    # or however you choose to structure your sanitization dispatch mechanism
}
```
Als für das setzen eines input-Wert, der vom Benutzer gelieferten Daten, einfach die Ausgabe der Wert des Elements bei der Ausgabe des Elements. Wie bei allen Benutzereingaben (wirklich, alle formatierte Ausgabe), richtig escape, die Daten bei der Ausgabe. Für HTML-Attribute, dies bedeutet, mit (z.B.) htmlspecialchars. Hinweis: Sie sollten nur die Flucht ausgehende Daten. Dies bedeutet, dass Ihre Bereinigung Funktionen sollten nicht nennen htmlspecialchars.

Können Sie die Benutzerfreundlichkeit zu verbessern indem Sie jeden Fehler neben den entsprechenden Eingang, indem eine "Fehler" - Klasse, um das element und das styling der "Fehler" Klasse zu machen, zeichnen sich. Verbesserung der Erreichbarkeit durch die Verpackung <label> Elemente rund um den label-text.

Ich bin neu in PHP und Schätze Eure Informationen sehr viel. Danke!

InformationsquelleAutor outis

Diese Struktur verwenden von Skript:

<?php
$errors = array();

if (isset($_POST['send'])) {
    //check data validity
    if (!mailValid($_POST['email']))
        $errors[] = 'Mail is not valid';
    ...

    //send data by email
    if (!$errors) {
        //send mail and redirect
    }
}
?>
<html>
    ...
    <?php
    if ($errors) {
        //display errors
        foreach ($errors as $error) {
            echo "$error<br />";
        }
    }
    ?>
    <form ...>
        ...
        Email: <input type="text" name="email" value="<?php echo isset($_POST['email']) ? htmlspecialchars($_POST['email']) : '' ?>" />
        ...
    </form>
    ...
</html>

InformationsquelleAutor Fanda

Können Sie immer tun Sie es, wie dieses, filter_var und in_array überprüft:

<?php
$myemail = "[email protected]";

//Pre made errors array
$errors=array('name'=>'Enter Your name',
              'email'=>'Please enter valid email',
              'phone'=>'Please enter valid phone number',
              'subject'=>'Please enter valid subject, more then 10 chars',
              'comment'=>'Please enter valid comment, more then 10 chars');

//Allowed post params and its validation type
$types = array('name'=>'string',
               'email'=>'email',
               'phone'=>'phone',
               'subject'=>'string',
               'comment'=>'string');

//A simple validation function using filter_var
function validate($value,$type){
    switch ($type){
        case "email":
            return ((filter_var($value, FILTER_VALIDATE_EMAIL))?true:false);
            break;
        case "phone":
            return ((preg_match("/^[0-9]{3}-[0-9]{4}-[0-9]{4}$/", $value))?true:false);
            break;
        case "string":
            return ((strlen($value) >=10 )?true:false);
            break;

        default:
            return false;
            break;
    }
}


//If forms been posted
if(!empty($_POST) && $_SERVER['REQUEST_METHOD'] == 'POST'){
    //Assign true, if all is good then this will still be true
    $cont=true;
    $error=array();
    foreach($_POST as $key=>$value){
      //if key is in $types array
      if(in_array($key,$types)){
        //If validation true
        if(validate($value, $types[$key])==true){
            $$key=filter_var($value, FILTER_SANITIZE_STRING);
        }else{
            //Validation failed assign error and swithc cont to false
            $error[$key]=$errors[$key];
            $cont=false;
        }
      }
    }
}

if($cont==true && empty($error)){
    //Send mail /do insert ect
}else{
    //Default to form
?>
<!doctype html>
<html>
<body>
<form action="" method="post">
 <p>Name: <input type="text" name="name" value="<?=@htmlentities($name);?>"/> <?=@$error['name'];?></P>
 <P>Email: <input type="text" name="email" value="<?=@htmlentities($email);?>" /> <?=@$error['email'];?></p>
 <P>Phone: <input type="text" name="phone" value="<?=@htmlentities($phone);?>"/> <?=@$error['phone'];?></p><br />
 <P>Subject: <input type="text" style="width:75%;" name="subject" /> <?=@$error['subject'];?></p>
 <p>Comments: <?=@$error['comment'];?><br />
 <textarea name="comment" rows="10" cols="50" style="width: 100%;"><?=@htmlentities($comment);?></textarea></p>
 <p><input type="submit" value="Submit"></p>
</form>
</body>
</html>
<?php
}?>

InformationsquelleAutor Lawrence Cherone

Schreibe einen Kommentar

Du musst angemeldet sein, um einen Kommentar abzugeben.