Auf Thema antworten 2026

Beitrag

<blockquote data-quote="Duddle" data-source="post: 2261342" data-attributes="member: 17775">AW: PHP-Kontaktformular, Sicherheit und FragenDein Muster in check_email() beachtet sie derzeit auch nicht. Folgende Zeilen haben keine Funktion:[PHP]//Validierung&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; check_name($name);&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; check_email($email);&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; check_nachricht($nachricht);}[/PHP]In [PHP]if (check_name($name) &amp;&amp; check_email($email) &amp;&amp; check_nachricht($nachricht) ==true) {[/PHP]muss das &quot;==true&quot; nicht sein, da du die boolschen Rückgabewerte mit UND verknüpfst. Ist eines davon false, wird der gesamte Ausdruck false.Bei sowas wie [PHP]// Abfrage ob alles gesetzt und ausggefüllt wurde&nbsp; &nbsp; if&nbsp; &nbsp; (isset($_POST[&#039;abs_name&#039;]) &amp;&amp; isset($_POST[&#039;abs_email&#039;]) &amp;&amp; isset($_POST[&#039;abs_nachricht&#039;]) &amp;&amp;&nbsp; &nbsp; &nbsp; &nbsp; !empty($name) &amp;&amp; !empty($email) &amp;&amp; !empty($nachricht)) { [/PHP]könntest du eine Hilfsfunktion einführen, damit du am Ende sowas schreiben kannst wie [PHP] if(isDefined($name) &amp;&amp; isDefined($email) ...[/PHP] Ich sehe keinen Mehrwert darin, an diesem Punkt noch zwischen &quot;$_POST[&#039;abs_name&#039;]&quot; und &quot;$name&quot; zu unterscheiden.Weiterhin würde ich wohl die Fehlerabfragen defensiver strukturieren. Das heisst, erst auf Fehler prüfen und entsprechend handeln, danach ist es per Definition fehlerfrei. Grob gesagt, statt[PHP]if(isDefined($foo) &amp;&amp; isDefined($bar)) {&nbsp; mail();} else {&nbsp; $error = &quot;&quot;;&nbsp; if(!isDefined($foo)) $error .= $foo;&nbsp; if(!isDefined($bar)) $error .= $bar;}[/PHP]lieber sowas[PHP]$error = &quot;&quot;;if(!isDefined($foo)) $error .= $foo;if(!isDefined($bar)) $error .= $bar;if($error == &quot;&quot;) {&nbsp; mail();}[/PHP]Das Ergebnis ist das selbe, der Code aber sauberer und einfacher zu erweitern.Eines noch:[PHP]foreach ($abs_eingaben as $key =&gt; $value)&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; $abs_eingaben[$key]=entfernen($value);[/PHP]Schau dir <a href="http://us1.php.net/manual/en/function.array-map.php" target="_blank">array_map()</a> an.Duddle</blockquote>

[QUOTE="Duddle, post: 2261342, member: 17775"] [b]AW: PHP-Kontaktformular, Sicherheit und Fragen[/b] Dein Muster in check_email() beachtet sie derzeit auch nicht. Folgende Zeilen haben keine Funktion: [PHP] //Validierung check_name($name); check_email($email); check_nachricht($nachricht);} [/PHP] In [PHP]if (check_name($name) && check_email($email) && check_nachricht($nachricht) ==true) {[/PHP] muss das "==true" nicht sein, da du die boolschen Rückgabewerte mit UND verknüpfst. Ist eines davon [I]false[/I], wird der gesamte Ausdruck [I]false[/I]. Bei sowas wie [PHP]// Abfrage ob alles gesetzt und ausggefüllt wurde if (isset($_POST['abs_name']) && isset($_POST['abs_email']) && isset($_POST['abs_nachricht']) && !empty($name) && !empty($email) && !empty($nachricht)) { [/PHP] könntest du eine Hilfsfunktion einführen, damit du am Ende sowas schreiben kannst wie [PHP] if(isDefined($name) && isDefined($email) ...[/PHP] Ich sehe keinen Mehrwert darin, an diesem Punkt noch zwischen "$_POST['abs_name']" und "$name" zu unterscheiden. Weiterhin würde ich wohl die Fehlerabfragen defensiver strukturieren. Das heisst, erst auf Fehler prüfen und entsprechend handeln, danach ist es per Definition fehlerfrei. Grob gesagt, statt [PHP] if(isDefined($foo) && isDefined($bar)) { mail(); } else { $error = ""; if(!isDefined($foo)) $error .= $foo; if(!isDefined($bar)) $error .= $bar; } [/PHP] lieber sowas [PHP] $error = ""; if(!isDefined($foo)) $error .= $foo; if(!isDefined($bar)) $error .= $bar; if($error == "") { mail(); } [/PHP] Das Ergebnis ist das selbe, der Code aber sauberer und einfacher zu erweitern. Eines noch: [PHP] foreach ($abs_eingaben as $key => $value) $abs_eingaben[$key]=entfernen($value); [/PHP] Schau dir [URL=http://us1.php.net/manual/en/function.array-map.php]array_map()[/URL] an. Duddle [/QUOTE]

Bilder bitte hier hochladen und danach über das Bild-Icon (Direktlink vorher kopieren) platzieren.

Authentifizierung

Wenn ▲ = 7, ▼ = 3, ◇ = 2 und die Summe von ▲ und ▼ durch ◇ geteilt wird, was ist das Ergebnis?