Auf Thema antworten 2026

Beitrag

<blockquote data-quote="Duddle" data-source="post: 2250440" data-attributes="member: 17775">AW: PHP-Kontaktformular, Sicherheit und FragenDie Mail wird durch mail() abgeschickt (bzw. versucht), der Rückgabewert der Funktion wird in $send_mail gespeichert. Das ist das gleiche wie wenn du beim Pizzaboten anrufst und bestellst: er startet den internen Prozess des Pizza-Bestellens und gibt dir am Ende Bescheid / zurück, ob alles okay war. Auch wenn es fehl schlägt, wurde der interne Prozess dennoch gestartet.Du solltest prüfen, ob eine Variable gesetzt und nicht-leer ist / die Anforderungen erfüllt. Wenn du aber sowieso z.B. $email auf bestimmte Zeichen prüfst, könntest du darin auch gleich prüfen ob sie leer ist. Eine leere Mail-Adresse ist auch nicht valide. Ich hatte mich da etwas inkorrekt ausgedrückt.Dabei fällt mir jetzt im Code auch noch mehr auf:[PHP]if&nbsp; &nbsp; (isset ($_POST[&#039;submit&#039;]) &amp;&amp; ($_POST[&#039;abs_name&#039;]) &amp;&amp; ($_POST[&#039;abs_email&#039;]) &amp;&amp; ($_POST[&#039;abs_nachricht&#039;])) {[/PHP]Warum hast du dort nur ein Mal isset()?Auch hast du einen Logik-Fehler in der Fehlerabfrage: du prüfst in Zeile 50 mit !empty($name) usw., springst dann in den Block mit den check-Funktionen, hast aber nach diesem Block ein mail(). Erstens ist das ein doppeltes mail(), zweitens wird dann die Mail unabhängig von dem !empty($name) usw. geschickt.Ich empfehle dir, das ganze drastisch zu entschlacken. Du brauchst die ganzen check-Funktionen nicht wirklich. Abgesehen davon dass die Regex-Muster sowieso nicht ausreichen, gibt es wie erwähnt eine fertige und durchdachte Funktionssammlung in <a href="http://www.php.net/manual/de/ref.filter.php" target="_blank">Filter</a>. Auch könntest du in folgendem Abschnitt die Doppelungen vermeiden: [PHP]if (empty($name)) {&nbsp; &nbsp; echo&nbsp; &nbsp; &#039;&lt;div class=&quot;fehler_name&quot;&gt;&lt;p&gt;How may I call you?&lt;/p&gt;&lt;p&gt;Please fill in a name.&lt;/p&gt;&lt;/div&gt;&#039;;&nbsp; &nbsp; echo&nbsp; &nbsp; &#039;&lt;input class=&quot;button&quot; type=&quot;submit&quot; name=&quot;submit&quot; value=&quot;again&quot;&gt;&#039;;}if (empty($email)) {&nbsp; &nbsp; echo&nbsp; &nbsp; &#039;&lt;div class=&quot;fehler_email&quot;&gt;&lt;p&gt;How may I reply?&lt;/p&gt;&lt;p&gt;Please fill in an email adress.&lt;/p&gt;&lt;/div&gt;&#039;;&nbsp; &nbsp; echo&nbsp; &nbsp; &#039;&lt;input class=&quot;button&quot; type=&quot;submit&quot; name=&quot;submit&quot; value=&quot;again&quot;&gt;&#039;;}if (empty($nachricht)) {&nbsp; &nbsp; echo&nbsp; &nbsp; &#039;&lt;div class=&quot;fehler_nachricht&quot;&gt;&lt;p&gt;How may I help you?&lt;/p&gt;&lt;p&gt;Please fill in your question.&lt;/p&gt;&lt;/div&gt;&#039;;&nbsp; &nbsp; echo&nbsp; &nbsp; &#039;&lt;input class=&quot;button&quot; type=&quot;submit&quot; name=&quot;submit&quot; value=&quot;again&quot;&gt;&#039;;}[/PHP]Beispiel:[PHP]$errormsg = &quot;&quot;;if(empty($name)) $errormsg .= &quot;&lt;p&gt;Please fill in a name.&lt;/p&gt;&quot;;if(empty($email)) $errormsg .= &quot;&lt;p&gt;Please fill in an email address.&lt;/p&gt;&quot;; //übrigens: &quot;address&quot;if(empty($nachricht)) $errormsg .= &quot;&lt;p&gt;Please fill in your question.&lt;/p&gt;&quot;;if($errormsg != &quot;&quot;) {&nbsp; &nbsp; echo&nbsp; &nbsp; &#039;&lt;div class=&quot;fehler&quot;&gt;&#039;.$errormsg.&#039;&lt;/div&gt;&#039;;&nbsp; &nbsp; echo&nbsp; &nbsp; &#039;&lt;input class=&quot;button&quot; type=&quot;submit&quot; name=&quot;submit&quot; value=&quot;again&quot;&gt;&#039;;}[/PHP]Du brauchst sicher keine zusätzliche Klasse pro Fehler. Wenn du die &quot;how may I&quot;-Fragen brauchst, wirf sie in $errormsg.Duddle</blockquote>

[QUOTE="Duddle, post: 2250440, member: 17775"] [b]AW: PHP-Kontaktformular, Sicherheit und Fragen[/b] Die Mail wird durch mail() abgeschickt (bzw. versucht), der Rückgabewert der Funktion wird in $send_mail gespeichert. Das ist das gleiche wie wenn du beim Pizzaboten anrufst und bestellst: er startet den internen Prozess des Pizza-Bestellens und gibt dir am Ende Bescheid / zurück, ob alles okay war. Auch wenn es fehl schlägt, wurde der interne Prozess dennoch gestartet. Du solltest prüfen, ob eine Variable gesetzt und nicht-leer ist / die Anforderungen erfüllt. Wenn du aber sowieso z.B. $email auf bestimmte Zeichen prüfst, könntest du darin auch gleich prüfen ob sie leer ist. Eine leere Mail-Adresse ist auch nicht valide. Ich hatte mich da etwas inkorrekt ausgedrückt. Dabei fällt mir jetzt im Code auch noch mehr auf: [PHP] if (isset ($_POST['submit']) && ($_POST['abs_name']) && ($_POST['abs_email']) && ($_POST['abs_nachricht'])) { [/PHP] Warum hast du dort nur ein Mal isset()? Auch hast du einen Logik-Fehler in der Fehlerabfrage: du prüfst in Zeile 50 mit !empty($name) usw., springst dann in den Block mit den check-Funktionen, hast aber [i]nach[/i] diesem Block ein mail(). Erstens ist das ein doppeltes mail(), zweitens wird dann die Mail unabhängig von dem !empty($name) usw. geschickt. Ich empfehle dir, das ganze drastisch zu entschlacken. Du brauchst die ganzen check-Funktionen nicht wirklich. Abgesehen davon dass die Regex-Muster sowieso nicht ausreichen, gibt es wie erwähnt eine fertige und durchdachte Funktionssammlung in [URL=http://www.php.net/manual/de/ref.filter.php]Filter[/URL]. Auch könntest du in folgendem Abschnitt die Doppelungen vermeiden: [PHP] if (empty($name)) { echo '<div class="fehler_name">How may I call you?Please fill in a name.</div>'; echo '<input class="button" type="submit" name="submit" value="again">';} if (empty($email)) { echo '<div class="fehler_email">How may I reply?Please fill in an email adress.</div>'; echo '<input class="button" type="submit" name="submit" value="again">';} if (empty($nachricht)) { echo '<div class="fehler_nachricht">How may I help you?Please fill in your question.</div>'; echo '<input class="button" type="submit" name="submit" value="again">';} [/PHP] Beispiel: [PHP] $errormsg = ""; if(empty($name)) $errormsg .= "Please fill in a name."; if(empty($email)) $errormsg .= "Please fill in an email address."; //übrigens: "address" if(empty($nachricht)) $errormsg .= "Please fill in your question."; if($errormsg != "") { echo '<div class="fehler">'.$errormsg.'</div>'; echo '<input class="button" type="submit" name="submit" value="again">'; } [/PHP] Du brauchst sicher keine zusätzliche Klasse pro Fehler. Wenn du die "how may I"-Fragen brauchst, wirf sie in $errormsg. 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?