Zitat:
|
Fällt euch noch etwas auf oder würdet Ihr diese Funktion auch bei euch bedenkenlos einsetzen?
|
Auf keinen Fall.
1.
Zitat:
|
if (realpath($_SERVER['SCRIPT_FILENAME'])
|
hat weder mit den Belangen der Funktion zu tun, noch muss mehr als einmal pro Script vorgenommen werden. Ein die ist auch unpraktisch.
2.
Zitat:
|
if(get_magic_quotes_gpc()
|
Selbe Argumentation. Ist nur einmal noetig, wobei man die Funktion mehrfach für dasselbe Feld aufrufen koennte. Hat auch nichts mit "protect" oder "real_escape" zu tun.
3.
Zitat:
|
$var = stripslashes($var);
|
stripslashes funktioniert so nicht für Arrays.
4.
Ist unnütz für Hashes und ohne Datenbankverbindung wertlos. Die ist aber kein Parameter der Funktion
5.
ist keine lange Zahl, sondern eine Gleitkommazahl. Lerne Grundlagen!
6.
nichtssagender Bezeichner, kein Default für Typ
Usw. usf. Hättest Du die Links oben mal vollständig gelesen, hättest Du einen Grossteil der Kritiken schon gekannt.