Kontaktformular (Fehlermeldung)

Stef

Hey zusammen,

unzwar ich habe mir ein kleines Kontaktformular gebaut.

Und das meiste funktioniert auch, bis auf 2 Fehlermeldungen. Unzwar ich habe zum spam schutz 3 radio-felder wovon nur 1ns richtig ist. Nun habe ich ne Bedingung aufgestellt wenn nichts angekreuzt ist dann soll es dies ausgeben und wenn das falsche angekreuzt wurde wird das ausgegeben. Nun habe ich noch dazu ein hidden feld erstellt um abzufragen ob das Formular schon abgesendet wurde und falls dies zutrifft werden die fehler ausgegeben.

Nun aber werden gleich beide Fehler ($fehler5 & $fehler 6) ausgegeben. Ich denke weil ich ja da überprüfe ob es leer ist und dann nochmal überprüfe ob das richtige kreuz angekreuzt wurde aber ich weiß nicht wie ichs lösen soll..

Hier mein php script:

PHP

<?php


    // Kurzschreibweise : Variable = WENN ? DANN : SONST;
    $name = isset ($_POST['name']) ? $_POST['name'] : null; 
    $email = isset($_POST['email']) ? $_POST['email'] : null;
    $betreff = isset($_POST['betreff']) ? $_POST['betreff'] : null;
    $nachricht = isset($_POST['nachricht']) ? $_POST['nachricht'] : null;


    //zum Spamschutz
    $eingabeUser = isset($_POST['spamschutz']) ? $_POST['spamschutz'] : null;

    $fehlerausgeben = isset ($_POST['schonabgesendet']) ? $_POST['schonabgesendet'] : null ;


    if(empty($name)){
        $fehler1 = "<p class='php_errors'>Bitte geben sie ihren Namen an!</p>";
    } 
    if(empty($email))
    {
        $fehler2 = "<p class='php_errors'>Bitte geben sie ihre E-Mail an!</p>";
    } 
    if(empty($betreff))
    {
        $fehler3 = "<p class='php_errors'>Bitte geben sie ihren Betreff an!</p>";
    } 
    if(empty($nachricht))
    {
        $fehler4 = "<p class='php_errors'>Bitte geben sie ihre Nachricht an!</p>";
    } 
    if(empty($eingabeUser))
    {
        $fehler5 = "<p class='php_errors'>Bitte kreuzen sie eines der Felder an!</p>";
    } else{
        $fehler5 = null;
    }
    if ($eingabeUser !== "Pferd"){
        $fehler6 = "<p class='php_errors'> Sie haben das falsche Kreuz angekreuzt!</p>";
    }  else{
        $fehler6 = null;
    }


    $fehlerarr = array($fehler1,$fehler2,$fehler3,$fehler4,$fehler5,$fehler6);
    ?>


    <head>
    <meta charset="utf-8">


    <title>Kontaktformular</title>


    <link rel="stylesheet" href="kontakformular.css">
</head>
<body>
    <div id="kontakt">

        <form method="POST" action="kontaktformular.php">
            <h2>Kontaktformular</h2>
            <label for="name">Ihr Name: </label>
                <input type="text" name="name" id="name" value="<?php echo $name ?>"><br>
             <label for="email">Ihre E-Mail-Adresse:</label>
                <input type="email" name="email" id="email" value="<?php echo $email ?>"><br>
            <label for="betreff">Betreff: </label>
                <input type="text" name="betreff" id="betreff" value="<?php echo $betreff ?>"><br>
            <label for="nachricht">Ihre Nachricht:</label><br>
                <textarea name="nachricht" id="nachricht" cols="50" rows="10" value="Hans"></textarea>
            <p>Kreuzen sie Bitte den Button an welches ein Tier ist!</p>
            <label for="spamschutz">Stein</label>
                <input type="radio" name="spamschutz" id="spamschutz" value="Stein">
            <br>
            <input type="hidden" name="schonabgesendet" value="ja">
            <label for="spamschutz">Haare</label>
                <input type="radio" name="spamschutz" id="spamschutz" value="Haare">  <br>
            <label for="spamschutz">Pferd</label>
                <input type="radio" name="spamschutz" id="spamschutz" value="Pferd">
          <br><input type="submit" value="Absenden!" name="submitted">
          <?php
              if(isset($fehlerarr)){
                  foreach ($fehlerarr as $ausgabe) {
                      if($fehlerausgeben == "ja"){
                      echo $ausgabe;
                  }
              }
              }
          ?>
        </form>
    </div>
</body>






    <?php






     
/*if(empty($name) or empty($email) or empty($betreff) or empty($nachricht) or $eingabeUser !== "Pferd")
    {
    }
    else {
        echo "Vielen Dank!";


    }*/


?>

Alles anzeigen

Kann man denn bei der textarea die value nicht speichern ?

Gruß, Stef

Theas1965

Das Wort "unzwar" findet man in keinem Duden
Ein else vor if ($eingabeUser !== "Pferd") sollte das Problem beheben
Warum bastelst du mehrere Fehler zusammen, um sie dann in ein Array zu packen? Nimm doch gleich ein Array und spare dir $fehler1 bis $fehler6
$fehlerarr = array($fehler1,$fehler2,$fehler3,$fehler4,$fehler5,$fehler6); sollte dir bei aktiviertem Errorreporting einiges um die Ohren hauen, wenn es keine Fehler gibt

Getestet ist das nicht:

PHP

<?php
     $name = $email = $betreff = $nachricht = '';


     $errors = array();

     if (isset ($_POST['name']))
     {
         $name = trim($_POST['name']); 
         $email = trim($_POST['email']);
         $betreff = trim($_POST['betreff']);
         $nachricht = trim($_POST['nachricht']);


         //zum Spamschutz
         $eingabeUser = isset($_POST['spamschutz']) ? $_POST['spamschutz'] : '';

        if(empty($name))
           $errors[] = "<p class='php_errors'>Bitte geben sie ihren Namen an!</p>";

        if(empty($email))
           $errors[] = "<p class='php_errors'>Bitte geben sie ihre E-Mail an!</p>";
        else if (filter_var($email, FILTER_VALIDATE_EMAIL) === false)
           $errors[] = "<p class='php_errors'>eMail-Adresse ist ungültig!</p>";

        if(empty($betreff))
           $errors[] = "<p class='php_errors'>Bitte geben sie ihren Betreff an!</p>";


        if(empty($nachricht))
           $errors[] = "<p class='php_errors'>Bitte geben sie ihre Nachricht an!</p>";


        if(empty($eingabeUser))
           $errors[] = "<p class='php_errors'>Bitte kreuzen sie eines der Felder an!</p>";
        else if ($eingabeUser !== "Pferd")
           $errors[] = "<p class='php_errors'> Sie haben das falsche Kreuz angekreuzt!</p>";
    }

    ?>
<html>
<head>
<meta charset="utf-8">
<title>Kontaktformular</title>
<link rel="stylesheet" href="kontakformular.css">
</head>
<body>
  <div id="kontakt">
   <form method="POST" action="">
     <h2>Kontaktformular</h2>
     <label for="name">Ihr Name: </label>
       <input type="text" name="name" id="name" value="<?php echo $name ?>"><br>
     <label for="email">Ihre E-Mail-Adresse:</label>
       <input type="email" name="email" id="email" value="<?php echo $email ?>"><br>
     <label for="betreff">Betreff: </label>
       <input type="text" name="betreff" id="betreff" value="<?php echo $betreff ?>"><br>
     <label for="nachricht">Ihre Nachricht:</label><br>
       <textarea name="nachricht" id="nachricht" cols="50" rows="10" value="Hans"></textarea>
     <p>Kreuzen sie Bitte den Button an welches ein Tier ist!</p>
     <label for="spamschutz">Stein</label>
       <input type="radio" name="spamschutz" id="spamschutz" value="Stein">
     <br>
     <input type="hidden" name="schonabgesendet" value="ja">
     <label for="spamschutz">Haare</label>
       <input type="radio" name="spamschutz" id="spamschutz" value="Haare">  <br>
     <label for="spamschutz">Pferd</label>
       <input type="radio" name="spamschutz" id="spamschutz" value="Pferd">
     <br><input type="submit" value="Absenden!" name="submitted">
<?php
     if(count($errors))
       echo implode("\n", $errors);
?>
   </form>
  </div>
</body>
</html>

Alles anzeigen

Arne Drews

PHP

<?php

$name = isset ($_POST['name']) ? $_POST['name'] : null; 
$email = isset($_POST['email']) ? $_POST['email'] : null;
$betreff = isset($_POST['betreff']) ? $_POST['betreff'] : null;
$nachricht = isset($_POST['nachricht']) ? $_POST['nachricht'] : null;
$eingabeUser = isset($_POST['spamschutz']) ? $_POST['spamschutz'] : null;


if ( isset($_POST['submitted']) ) {


    $aErrors = array_filter(
            [
                [$name, 'Bitte geben Sie Ihren Namen an!'],
                [$email, 'Bitte geben Sie Ihre E-Mail an!'],
                [$betreff, 'Bitte geben Sie Ihren Betreff an!'],
                [$nachricht, 'Bitte geben Sie Ihre Nachricht an!'],
                [$eingabeuser, 'Fehler bei Angabe des Kreuzes!', 'Pferd']
            ]
            , function ( $data ) {
                $bIsError = isset( $data[2] )? ( empty($data[0] && $data[0] !== $data[2]) ): ( empty($data[0]) );
                return $bIsError? $data[1]: null;
            }


        );


}
?>


<head>
<meta charset="utf-8">
<title>Kontaktformular</title>
<link rel="stylesheet" href="kontakformular.css">
</head>
<body>
    <div id="kontakt">

        <form method="POST" action="kontaktformular.php">
            <h2>Kontaktformular</h2>
            <label for="name">Ihr Name: </label>
                <input type="text" name="name" id="name" value="<?php echo $name ?>"><br>
             <label for="email">Ihre E-Mail-Adresse:</label>
                <input type="email" name="email" id="email" value="<?php echo $email ?>"><br>
            <label for="betreff">Betreff: </label>
                <input type="text" name="betreff" id="betreff" value="<?php echo $betreff ?>"><br>
            <label for="nachricht">Ihre Nachricht:</label><br>
                <textarea name="nachricht" id="nachricht" cols="50" rows="10" value="Hans"></textarea>
            <p>Kreuzen sie Bitte den Button an welches ein Tier ist!</p>
            <label for="spamschutz">Stein</label>
                <input type="radio" name="spamschutz" id="spamschutz" value="Stein">
            <br>
            <input type="hidden" name="schonabgesendet" value="ja">
            <label for="spamschutz">Haare</label>
                <input type="radio" name="spamschutz" id="spamschutz" value="Haare">  <br>
            <label for="spamschutz">Pferd</label>
                <input type="radio" name="spamschutz" id="spamschutz" value="Pferd">
          <br><input type="submit" value="Absenden!" name="submitted">
          <?php
            if( isset($aErrors) && count($aErrors) > 0 ) {


                echo implode( '<br>', $aErrors );


            }
          ?>
        </form>
    </div>
</body>

Alles anzeigen

Theas1965

Arne Drews
Das ist klar die kürze Version, aber damit wirst du höchst wahrscheinlich Stef überfordern

Arne Drews

Mit dem jetzigen Wissenstand mag das sein, aber eigentlich ist er immer aufgeschlossen, neues zu lernen, sonst hätte ich mir die Mühe nicht gemacht...

Ist allerdings ein Logikfehler drin, musste ich gerade feststellen, es muß über array_map laufen, da array_filter nur auf Basis von true und false auswertet:

PHP

$aErrors = array_filter(
            array_map(
                function ( $data ) {
                    $bIsError = isset( $data[2] )? ( empty($data[0] || $data[0] !== $data[2]) ): ( empty($data[0]) );
                    return $bIsError? $data[1]: null;
                }
                , [
                    [$name, 'Bitte geben Sie Ihren Namen an!'],
                    [$email, 'Bitte geben Sie Ihre E-Mail an!'],
                    [$betreff, 'Bitte geben Sie Ihren Betreff an!'],
                    [$nachricht, 'Bitte geben Sie Ihre Nachricht an!'],
                    [$eingabeuser, 'Fehler bei Angabe des Kreuzes!', 'Pferd']
                ]
            )
        );

Alles anzeigen

Stef

Hi,

oh... habe manches vergessen...

@Arne Drews : Warum setzt du denn vor dem überprüfen ob index submitted abgesendet wurde die variablen ? Und habe nun mal dein script ausprobiert. Funktioniert auch. Bis auf, das das Wort welches richtig ist auch ausgegeben wird und wenn ich keine foreach schleife beim ausgeben im html teil mache gibt er mie die fehlermeldung : Array to string conversion.

Theas1965 : Habe deins auch mal ausprobiert. Funktioniert auch gut.

Habs nun so gemacht:

PHP

<?php


    error_reporting(E_ALL);


    // Kurzschreibweise : Variable = WENN ? DANN : SONST;
    $name = isset ($_POST['name']) ? $_POST['name'] : null; 
    $email = isset($_POST['email']) ? $_POST['email'] : null;
    $betreff = isset($_POST['betreff']) ? $_POST['betreff'] : null;
    $nachricht = isset($_POST['nachricht']) ? $_POST['nachricht'] : null;


    //zum Spamschutz
    $eingabeUser = isset($_POST['spamschutz']) ? $_POST['spamschutz'] : null;

    //zum checken ob formular das 2.te mal aufgerufen worden ist
    $fehlerausgeben = isset ($_POST['schonabgesendet']) ? $_POST['schonabgesendet'] : null ;




    trim($name);
    trim($email);
    trim($betreff);
    trim($nachricht);


    $fehler = array ();


    if(isset($_POST['submitted']))
    {


    if(empty($name))
    {
        $fehler[] = "<p class='php_errors'>Bitte geben sie ihren Namen an!</p>";
    } 


    if(empty($email))
    {
        $fehler[] = "<p class='php_errors'>Bitte geben sie ihre E-Mail an!</p>";
    } else if ((filter_var($email, FILTER_VALIDATE_EMAIL) === false))
    {
        $fehler[] = "<p class='php_errors'>Bitte geben sie eine gültige E-Mail-Adresse an!</p>";    
    }
    if(empty($betreff))
    {
        $fehler[] = "<p class='php_errors'>Bitte geben sie ihren Betreff an!</p>";
    } 
    if(empty($nachricht))
    {
        $fehler[]= "<p class='php_errors'>Bitte geben sie ihre Nachricht an!</p>";
    }


    if(empty($eingabeUser))
    {
        $fehler[] = "<p class='php_errors'>Bitte kreuzen sie eines der Felder an!</p>";
    } else if ($eingabeUser !== "Pferd")
    {
        $fehler[] = "<p class='php_errors'> Sie haben das falsche Kreuz angekreuzt!</p>";
    }


    //var_dump(filter_var($email, FILTER_VALIDATE_EMAIL));


}
    ?>
    <!DOCTYPE html>
<html>
    <head>
    <meta charset="utf-8">


    <title>Kontaktformular</title>


    <link rel="stylesheet" href="kontakformular.css">
</head>
<body>
    <div id="kontakt">

        <form method="POST" action="kontaktformular.php">
            <h2>Kontaktformular</h2>
            <label for="name">Ihr Name: </label>
                <input type="text" name="name" id="name" value="<?php echo $name ?>"><br>
             <label for="email">Ihre E-Mail-Adresse:</label>
                <input type="text" name="email" id="email" value="<?php echo $email ?>"><br>
            <label for="betreff">Betreff: </label>
                <input type="text" name="betreff" id="betreff" value="<?php echo $betreff ?>"><br>
            <label for="nachricht">Ihre Nachricht:</label><br>
                <textarea name="nachricht" id="nachricht" cols="50" rows="10" value="Hans"></textarea>
            <p>Kreuzen sie Bitte den Button an welches ein Tier ist!</p>
            <label for="spamschutz">Stein</label>
                <input type="radio" name="spamschutz" id="spamschutz" value="Stein">
            <br>
            <input type="hidden" name="schonabgesendet" value="ja">
            <label for="spamschutz">Haare</label>
                <input type="radio" name="spamschutz" id="spamschutz" value="Haare">  <br>
            <label for="spamschutz">Pferd</label>
                <input type="radio" name="spamschutz" id="spamschutz" value="Pferd">
          <br><input type="submit" value="Absenden!" name="submitted">
          <?php
              if(count($fehler))
              echo implode("\n", $fehler);

          ?>
        </form>
    </div>
</body>
</html>

Alles anzeigen

Stef

Arne Drews

Zitat

Warum setzt du denn vor dem überprüfen ob index submitted abgesendet wurde die variablen ?

Du kannst die auch leer definieren und dann innerhalb des DIV aus den POST-Daten setzen, wie Theas1965 es gemacht hat. Ist vom Ergebnis dasselbe.

Zitat

Bis auf, das das Wort welches richtig ist auch ausgegeben wird und wenn ich keine foreach schleife beim ausgeben im html teil mache gibt er mie die fehlermeldung : Array to string conversion.

Dann hast Du vermutlich meine Korrektur aus #5 noch nicht gehabt.

Stef

Hey,

eure 2 Kommentare waren als ich gepostet habe noch nicht da.

Ok. Die stehen ja außerhalb, damit man auch im html teil auf diese zugreifen kann ?

Dein code überfordert mich grad schon etwas, aber man möchte ja was lernen .

PHP

$aErrors = array_filter //mit dem filtert man ja den array mit ner callback funktion
            array_map //was macht array map ?
(
                function ( $data ) //warum gibts du der funktion keinen namen ? {
                    $bIsError = isset( $data[2] )? ( empty($data[0] || $data[0] !== $data[2]) ): ( empty($data[0]) ); // hier versteh ich grad nichts. ich glaube, da ich ja den array_filter vorhin schonmal mit var_dump ausgegeben habe um zu schauen was drin steckt, wenn $data[2] in diesem Fall das Pferd (glaub ich) existiert dann  ist data[0] leer oder data[0] nicht gleich $data[2] sonst data[0] ist leer , data[0] ist dann sozusagen die eingabe des Benutzers ? 
                    return $bIsError? $data[1]: null; //hier den wert $data [1] wenn $bIsError exsistiert zuordnen sonst ist data[1] null (leer) und es returnen um es außerhalb nutzen zu können
                }
                , [//warum sind diese klammern ([]) um den anderen [] mit dem Text ? hier tust du dann pro zeile jeder variable die fehlermeldung zuordnen ?  Da lieg ich bestimmt falsch.
                    [$name, 'Bitte geben Sie Ihren Namen an!'],
                    [$email, 'Bitte geben Sie Ihre E-Mail an!'],
                    [$betreff, 'Bitte geben Sie Ihren Betreff an!'],
                    [$nachricht, 'Bitte geben Sie Ihre Nachricht an!'],
                    [$eingabeuser, 'Fehler bei Angabe des Kreuzes!', 'Pferd']
                ]
            )
        );

Alles anzeigen

Zu implode. Mann muss doch beim if(...) eine { } machen und da kommt dann das implode rein oder ?

Zu meinem Spam-Schutz:

Ich habe ja 3 sachen zur auswahl (Haare Pferd Stein) ist es nicht etwas zu leicht für ein spambot oder schafft er nicht das richtige auszuwählen ?

Jetzt funktioniert auch deins

Arne Drews : Was sind denn für dich die wichtigsten array funktionen ?

Vielen Dank,

Gruß,Stef

Arne Drews

Hi,

Also array_map() und array_filter() machen vom Grunde her das gleiche.
Sie führen für jedes Element des übergebenen Array eine Callback-Funktion aus.
Der Unterschied liegt ganz einfach darin, dass Du bei array_map() für jedes Element den Rückgabewert über return selbst definieren kannst, während array_filter die Rückgabe auf bool castet und alle Elemente, bei denen ein false aus dem Callback kommen quasi aus dem Array entfernt.

Daher nutze ich erst array_map(), um die Fehlermeldungen oder null für keinen Fehler in das Array $aErrors zu schreiben und lasse das ganze dann noch einmal durch array_filter() laufen, um die null ( hier äquivalent zu false ) Werte zu entfernen. Übrig bleiben dann nur noch die Fehler.

Stef

Hey,

aso ok.

Und wie kommst du auf data[0], data[1] und data[2] ?

Hier:

PHP

[                    [$name, 'Bitte geben Sie Ihren Namen an!'],                    [$email, 'Bitte geben Sie Ihre E-Mail an!'],                    [$betreff, 'Bitte geben Sie Ihren Betreff an!'],                    [$nachricht, 'Bitte geben Sie Ihre Nachricht an!'],                    [$eingabeuser, 'Fehler bei Angabe des Kreuzes!', 'Pferd']                ]

du hast ja da nach der funktion diese [] klammern. Warum machst du diese umrum der anderen [] wo die var und die fehlermeldung drinsteht. Ich denke du machst diese um diese in ein array zu packen ?
Dazu: Tust du indem die die va und die fehlermeldung in einem Array schreibst, sagen das die fehlermeldung nur erscheinen soll wenn z.b. $name fehlt?

Stef

Arne Drews

Hallo,

Ich übergebe der Callback-Funktion ja jedes Array-Element als $data, das ist bspw. im ersten Durchlauf das hier:

PHP

[$name, 'Bitte geben Sie Ihren Namen an!']

...was ja ebenfalls ein Array ist. In diesem Fall mit zwei Elementen, also $data[0] und $data[1].
Im Falle Deines Hakens willst Du aber eine "erweiterte" Prüfung auf den Wert haben. Für den Durchlauf bekommt die Callback-Funktion im Gegensatz zu den anderen Durchläufen ein Array mit drei Elementen:

PHP

[$eingabeuser, 'Fehler bei Angabe des Kreuzes!', 'Pferd']

Also prüfe ich einfach, ob $data[2], also ein drittes Array-Elelment existiert und prüfe entsprechend auf leer und Wert oder halt nur auf leer.

Gruß Arne

Stef

Hey,

aso ok. So ist das. Ich glaube ich habs verstanden.

Ich habe jetzt die funktion an meiner registrierform angewandt,
die funktioniert auch soweit.

Aber ne frage hab ich dazu doch noch:

PHP

$errors = array_filter (
                //filterung der arrays mit callback funktion (bool) = true/false
                array_map (
                    //ebenso soart durchlaufen/filtern der arrays mit callback funktion welche man returnen kann und außerhalb nutzen kann
                    function($daten){
                        // wenn $daten[0] nicht exsestiert , dann $daten[0] ist  leer, sonst daten[0] ist nicht leer
                         $fehler = !isset($daten[0]) ? (empty($daten[0])) : (!empty($daten[0])); 
                            //hier wenn $fehler da sind sind diese $daten[1] (die strings) sonst null 
                            return $fehler ? $daten[1] : null;
            },

        [
            [$vorname , 'Bitte geben sie ihren Vornamen an!'],
            [$nachname , 'Bitte geben sie ihren Nachnamen an!'],
            [$email , 'Bitte geben sie ihre E-Mail-Adresse an!'],
            [$benutzername , 'Bitte geben sie ihren Benutzernamen an!'],
            [$password , 'Bitte geben sie ihr Password an!'],
            [$password_wiederholung , 'Bitte wiederholen sie die Angabe ihres Passwords!']
            //hier hat man daten[0] & daten[1]
        ]

    )
);

Alles anzeigen

PHP

$fehler = !isset($daten[0]) ? (empty($daten[0])) : (!empty($daten[0]));

Warum gibt es nur was aus wenn ich nach dem : empty und nicht !empty schreib? Ich sage doch: wenn $daten[0] nicht exsestiert , dann $daten[0] ist leer, sonst daten[0] ist nicht leer ?

Gruß, Stef

Arne Drews

Hallo,

Nicht ganz...
Aber Du machst jetzt ja auch was anderes... Das ist nicht die gleiche Logik, die ich Dir gegeben habe. Soll es denn so sein oder hast Du Dich vertan?

Bei Dir passiert folgendes:

PHP

if ( !isset($data[0]) ) { // wenn der Index 0 im Array $data nicht existiert...


    $fehler = empty( $data[0] ); //...dann prüfe, ob das Element mit dem Index 0 leer ist


} else {


    $fehler = !empty( $data[0] ); //...sonst prüfe, ob das Element mit dem Index 0 nicht leer ist


}

Alles anzeigen

Das macht so eigentlich keinen Sinn, denn das erste Element existiert immer bei Deiner Grundlage. Damit würde er eh nie das empty() zwischen dem Fragezeichen und dem Doppelpunkt anwenden.

Bei mir war die Logik anders:

PHP

if ( isset($data[2]) ) { // wenn ein Element mit dem Index 2 ( also ein drittes Element! ) existiert


    $fehler = empty( $data[0] ) || $data[0] != $data[2]; // prüfe ob Element mit Index 0 leer ist ODER Element mit Index 0 ungleich Element mit Index 2 ist


} else {


    $fehler = empty( $data[0] ); // prüfe nur ob Element mit Index 0 leer ist.


}

Alles anzeigen

Wenn Du die Logik mit dem Vergleich der Eingabe nicht mehr benötigst, ist das auch kürzer und einfacher abbildbar:

PHP

if ( empty($data[0]) ) {


    $fehler = $data[1]; // Fehlermeldung in $fehler schreiben


} else {


    $fehler = null; // kein Fehler definiert


}

Alles anzeigen

was das gleiche ist, wie:

PHP

$fehler = empty( $data[0] )? $data[1]: null;

Dadurch hast Du in $fehler ein Array, in dem jedes Element entweder eine Fehlermeldung oder null ist.
array_filter() "schmeisst" nun jedes Element, das auf bool gecastet nicht true ergibt aus dem Array raus. Und da man bei array_filter die Callback-Funktion auch weglassen kann, sieht das ganze in Kurzform so aus:

PHP

$fehler = array_filter( $fehler );

Das einzige, was ich gemacht habe ist, dass ich anstelle von $fehler die komplette array_map()-Verarbeitung als Parameter an array_filter() übergeben habe, was am Ende nur das Ergebnis von array_map() ist, nämlich das Array $fehler mit den Fehlermeldungen oder eben dem Wert null in den Elementen.

Stef

Hey,

Wollte es mal testen und habe mich anscheinend vertan.

Aso ok. So weit habe ich nicht gedacht.

Meinst du das man erstmal mit array map dies durchläuft und darin die ganzen vars mit fehlermeldung speichert und dann die function returnt mit z.b. folgendem code:

PHP

$errors = array_map (
                    //ebenso soart durchlaufen/filtern der arrays mit callback funktion welche man returnen kann und außerhalb nutzen kann
                    function($daten){
                        // wenn $daten[0] leer ist Dann ist $fehler data[1] die fehlermeldung , sonst ist die var leer
                         $fehler = empty($daten[0]) ? $daten[1] : null;
                            //Dann muss man sozusagen hier dann nur die var $fehler returnen ?
                            return $fehler;
            },

        [
            [$vorname , 'Bitte geben sie ihren Vornamen an!'],
            [$nachname , 'Bitte geben sie ihren Nachnamen an!'],
            [$email , 'Bitte geben sie ihre E-Mail-Adresse an!'],
            [$benutzername , 'Bitte geben sie ihren Benutzernamen an!'],
            [$password , 'Bitte geben sie ihr Password an!'],
            [$password_wiederholung , 'Bitte wiederholen sie die Angabe ihres Passwords!']
            //hier hat man daten[0] & daten[1]
        ]

    );

Alles anzeigen

und dann außerhalb dieser durch den array_filter jagen zu lassen?

PHP

$fehler = array_filter($errors);

Und dann z.b. im html teil so ausgeben:

PHP

if(isset($fehler) && count($fehler) > 1) //wenn $fehler exsistieren und $fehleranzhal ist größer als 1 
{
     implode(' , ', $fehler); //gebe diese aus
}

Gruß,

Stef

Arne Drews

Jap, so ist es gedacht...
Wie gesagt, Eingangs hattest Du noch die Anforderung, dass in einem bestimmten Fall zusätzlich zu empty() noch auf einen Wert geprüft wird.

Stef

Ok. Gut.

Tags