Ansicht umschalten
Avatar von
  • unbekannter Benutzer

667 Beiträge seit 22.07.2004

Re: Das ist doch total übertrieben

lev_arris schrieb am 5. Januar 2011 22:52


> <?php

> $strEmail = (isset($_GET["email"]) && isValidEMail($_GET["email"]) ?
> strtolower($_GET["email"]) : null);
> // Exclude webmaster users for some reason
> if ($strEmail !== null) {
>     $arrTmp = explode('@', $strEmail, 2);
>     if ($arrTmp[0] == 'webmaster') { // <- Hier kracht's!

Bevor man $arrTemp[0] verwednet sollte man erstmal überprüfen, ob es
das überhaupt gibt. (ist vielleicht anzunehmen, wegen isValidEmail())
Und dann würde man eh den === Operator verwenden, und somit hätte PHP
garkeinen Grund, den String als Zahl, float oder sonstwas, zu
interpretieren.

Bewerten
- +
Ansicht umschalten