dedlfix: Doppelte Daten, fehlende Programmlogik?

Beitrag lesen

echo $begrüßung;

$var1 = substr(htmlspecialchars($_POST['comment_titel'],ENT_QUOTES),0,150);
Noch was: Es ist keine gute Idee, diese "Importvariablen" lokalen Variablen zuzuweisen. Das verschleiert die Herkunft.

Vorsicht! In dem Fall ist das ja keine Import-Variable mehr, sondern ein (wenn auch nicht sehr sinnvoll) bearbeiteter Wert. Und diesen in einer neuen Variable abzulegen, spricht nichts dagegen.

Ansonsten:
htmlspecialchars() ist, wie schon erwähnt, eine für SQL-Statements unpassende Funktion.
Und auch die Reihenfolge - erst htmlspecialchars(), dann mit substr() beschneiden - ist alles andere als sinnvoll, da man dabei Entities treffen und verstümmeln kann.

Wenn an einer Stelle nur Integer-Werte, wie beispielsweise in $idd, vorkommen sollen, wäre es sinnvoll, das mit intval() sicherzustellen.

echo "$verabschiedung $name";