ereg_replace bug?

[spider0176]

hello,
i use php 4.3.3 under windows,

the following code seems doesn't give correct output when array contains more than one key:

$array = explode("\n", $emails);
$nummails = count($array);

foreach($array as $key=>$email){
if(eregi("^{[a-z0-9]+([_\.-][a-z0-9]+)@([a-z0-9]+([.-][a-z0-9]+))+\.[a-z]{2,4}$",} $email, $regs))
{
print $email." - Valid<br>";
} else {
print "<font color='FF0000'>".$email." - Invalid</font><br>";
}
}

is this a bug? 😕
if not pls. help anyone to solve this problem,

tnx

p.s.
data posted from the folowing form:
<form method="POST" action="maillist.php" name="form">
<p align="center"><textarea rows="15" name="emails" cols="40"></textarea><br>
<br>
<br>
<input type="submit" value="Submit" name="submit"><input type="reset" value="Reset" name="B2"></p>
</form>

[diego25]

I tried this code:

<?
$emails = "[e-mail]\[e-mail]";
$array = explode("\n", $emails);
$nummails = count($array);

foreach($array as $key=>$email){
if(eregi("^[a-z0-9]+([_\\.-][a-z0-9]+)*@([a-z0-9]+([\.-][a-z0-9]+)*)+\\.[a-z]{2,4}$", $email, $regs))
{
print $email." - Valid\n";
} else {
print $email." - Invalid\n";
}
} 
?>

and it works fine I think. Could you post your input data? Also, try printing out $emails to make sure it is in the format you expect.

Diego

[spider0176]

hi Diego,

tnx for reply, you know the problem is still the same, although i replaced the code with yours,

output looks like this:

[e-mail] - Invalid
[e-mail] - Invalid
xxx3@hotmail - Invalid
[e-mail] - Invalid
[e-mail] - Invalid
[e-mail] - Invalid
[e-mail] - Valid

it's okay when i post 1 email, but in the multiple case it just fails to give correct output. 😕

regards

[xblue]

if your input data is a flat file, there's a good chance that lines are separated by \r\n, so you'd have that nasty \r left after exploding on \n on each line but the last one.

if this should be the problem, try using file() to read the data into an array of lines (I think this handles different types of line breaks fine) or trim each array element.

[spider0176]

nops 🙁

still does not work...

any other ideas???

[Weedpacket]

Am I right in guessing that it's only ever the last email in the list that's valid and never any of the earlier ones?

[spider0176]

yeap, exactly...

[Weedpacket]

In that case there's something about all of the lines except the last that prevents your expression matching. It still sounds like a linebreak problem as xblue suggested - as a trial, drop the ^$ anchors and see if that changes anything. If it does, put the ^ back on and try again. If it still works, the stray characters are on the end of the line, and if it stops working they're on the beginning. Since there's nothing visible, they're probably whitespace, which trimming would take care of.