Hi!
Einige Ergänzungen und Anmerkungen zu
http://aktuell.de.selfhtml.org/artikel/php/form-mail-advanced/
Man sollte vielleicht noch das Stichwort Fluent Interface als Kommentar in den Code stellen, damit interessierte Leser, die diese Art der Verkettung nicht kennen, sich mit passendem Stichwort auf die Suche begeben können. Oder man erläutert es kurz und dass weitere Methodenaufrufe an beliebigen Stellen eingefügt oder angehängt werden können.
Warum ist ($_SERVER['REQUEST_METHOD'] === "POST") ein typidentischer Vergleich? Was für andere Typen erwartest du denn in diesem Feld? Und welchen Nachteil/Vorteil hat es, wenn ein solcher kommt gegenüber einem normalen Vergleich?
Warum ist da eine Definitionsliste für die Felder des Formulars verwendet worden? Eine einfache Liste mit Label-Elementen wäre semantisch besser und hätte auch noch einen Nutzen für den Anwender. Ein Untereinander statt Nebeneinander (bei Checkboxen, Radios und Select) verlängert zwar die Code-Box, doch die damit erhöhte Übersichtlichkeit wiegt deutlich mehr.
Und die() ist keine Fehlerbehandlung. Das sollte man auch nicht der Einfachheit in Beispielen bringen. Beispiele haben Lehrcharakter und werden immer wieder als "so muss man es machen" angesehen. Deshalb sollte man dort besonders auf die Dinge achten, die man ansonsten immer bemängelt, weil sie schlecht sind.
Man könnte vielleicht auch noch eine Eingabevalidierung andeuten nebst alternativem Programmfluss, der bei Nichtbestehen die Mail nicht sendet. Vermutlich ist da gleich ein Umbau auf Affenformular gescheiter.
Lo!