Artikel ins Angebot übernehmen

[1unitedpower]

Ok, habe ich nun geändert, ist es so richtig?

So langsam macht das Formular einen richtig guten Eindruck.

3. Die Fehler, die von deiner PHP-Überprüfung festgellt werden, sollten direkt an den betroffenen Feldern ausgegeben werden, und nicht irgendwo ihres Kontextes entrissen am Anfang oder Ende des Formulars ausgegeben werden.

Ja, das würde ich auch viel lieber machen, das würde dann bedeuten, ich muss diesen Block nach jedem Formular einfügen?

So ähnlich könntest du vorgehen, allerdings würde das jetzt noch dazu führen, dass die Fehlermeldung bei jedem Formular angezeigt werden würde, unabhängig davon, welches Formular überhaupt ausgefüllt und versendet wurde. Du müsstest also vorher noch prüfen, auf welchen Artikel sich die Fehlermeldung eigentlich bezieht. Dazu kannst du Artikel-ID benutzen, die du ja von dem Hiddenfeld übermittelt bekommst.

Kopieren von Codeausschnitten ist außerdem ein Indiz dafür, dass man den Code besser organisieren könnte, zum Beispiel indem du den Codeausschnitt in eine wiederverwendbare Funktion packst und die Stellen, die sich unterscheiden können, durch Parameter generalisierst. Viele Entwickler versuchen sogar, sich möglichst nie zu wiederholen, das Prinzip heißt "Don't repeat yourself" kurz DRY, also "Wiederhole dich nicht selbst".