Du befindest Dich im Archiv vom ABAKUS Online Marketing Forum. Hier kannst Du Dich für das Forum mit den aktuellen Beiträgen registrieren.

Code optimieren kürzen

Ajax, Hijax, Microformats, RDF, Markup, HTML, PHP, CSS, MySQL, htaccess, robots.txt, CGI, Java, Javascript usw.
Mork vom Ork
PostRank 9
PostRank 9
Beiträge: 2557
Registriert: 08.07.2008, 11:07
Wohnort: Aufm Friedhof.

Beitrag von Mork vom Ork » 02.06.2009, 15:48

net(t)worker hat geschrieben:wenn du schon phpcode von mir zitierst, dann bitteschön auch richtig...
Sorry, das kommt davon, wenn man Zitiertes hin- und herschiebt …

Nichtsdestrotz:
Mork vom Ork hat geschrieben: - Das Grundproblem löst es nicht (wozu die ganzen Variablen, wenn alles fein säuberlich in $_POST steht).
wie ich bereits weiter oben ausführte wäre es immer sinnvoll alle externen Variablen auch zu validieren, also auf gültige Werte zu prüfen...
Das habe ich nicht bestritten. Aber die Gültigkeit kann man auch in $_POST prüfen und ein leicht korrigierter Wert lässt sich auch in $_POST ablegen (alles andere sollte eh gleich wieder ausgespuckt werden) - weder für das eine, noch für das andere braucht es eine Doppelgängervariable.
vor allem wenn später irgendjemand anderes das script erweitern muss...
Gerade Daten, die aus unsicheren Quellen stammen (wie eben $_POST), sind immer mit Vorsicht zu behandeln, auch nach einer allgemeinen Gültigkeitsprüfung, weshalb ein praktisch unveränderter Wert meiner Meinung nach auch da gelassen werden sollte, wo er herkommt. Mit dieser Von-einem-Eimerchen-ins-andere-Methode wird nur die Herkunft verschleiert. Wenn ich ein fremdes Skript sehe, sagt mir $s1 erstmal gar nix, bei $_POST["s1"] leuchten hingegen sofort, ohne weitere Nachforschungen, alle Warnlampen von htmlspecialchars() bis mysql_real_escape_string().
Aber das führt jetzt ins Philosophische …