[PHP] array check werkt niet

[Saven]

Hellow Tweakers,

Het is weer eens zo ver, ik heb een vreemd probleem welke ik zelf maar niet opgelost krijg. Wellicht kan dit stukje in domme programmeervoorbeelden, maar ik zie de fout? echt niet

Ik heb namelijk een email blacklist array, en ik wil kijken of het ingevoerde email adres inderdaad gebanned is.

Daarvoor gebruik ik dit stukje code:

PHP:

$expl = explode('@', $email);

$banned = explode("\n", $this->core->settings['banned_email']);

if( in_array($expl[0], $banned) || in_array($expl[1], $banned) || in_array($email, $banned) )
{
    $return[] = 'Je e-mail adres staat helaas op de blacklist';
}

$email bevat in dit geval [e-mail] en hij doorstaat de test...
als $email = [e-mail] geeft hij wel mooi de error.

Ik heb natuurlijk een beetje lopen debuggen:

code:

Email adres:
Array ( [0] => test [1] => mailcatch.com )

Banned lijstje:
Array ( [0] => mailcatch.com [1] => tempinbox.com [2] => temporaryinbox.com [3] => [e-mail] )

Exploded adres:
Email[0]: test
Email[1]: mailcatch.com

[e-mail] wordt blijkbaar wel geblokkeerd, maar [e-mail] niet

Kan iemand mij vertellen wat ik fout doe

[Erik Jan]

Er zit nog een verstopt \r karakter aan het einde van je string omdat je waarschijnlijk "windows"-stijl enters in je PHP file gebruikt (\r\n).

Aan het einde van de array heb je daar dus geen last van.

@onder:

[ Voor 3% gewijzigd door Erik Jan op 24-02-2009 23:16 ]

[Patriot]

[e-mail] is de laatste in het lijstje. Als $this->core->settings['banned_email'] vanuit een bestand geladen is, is het mogelijk dat na iedere regel niet \n maar \r\n staat. Ik weet niet hoe strict PHP daarin is, maar technisch gezien is het dan mogelijk dat hij die carriage return (\r) nog na alle elements van de Banned-array heeft staan, behalve de laatste (mits er geen lege regel onderaan het bestand waaruit ze geladen worden staat).

Probeer dit eens:

PHP:

$expl = explode('@', $email);

$banned = explode("\n", $this->core->settings['banned_email']);
$banned = array_map('trim', $banned);

if( in_array($expl[0], $banned) || in_array($expl[1], $banned) || in_array($email, $banned) )
{
    $return[] = 'Je e-mail adres staat helaas op de blacklist';
}

EDIT:

Damn you Erik Jan

[Saven]

Hehehe.. That did the trick
Ik had ze rechtstreeks ingevoerd via PHPMyAdmin. Maar trim er overheen halen werkt inderdaad

Dank u allen :)!