Please help with email script.

Peredy

Here is the code I am using on my site....

<?php
set_time_limit();
$filename = "address.txt"; // File which holds all data

$arrFp = file( $filename ); // Make array of file content lines
$numAdds = count( $arrFp ); // Count no. of elements in the array, count email addresses

$sender = "$_POST[sendername]"; 

$message = "$_POST[block]"; 

for($i=0; $i<$numAdds; $i++)
{   

$emailAddress = trim( $arrFp[$i] ); // Trim email addresses before sending mails
echo "Sending email to $emailAddress...";
$success = 
mail( $emailAddress, $subject, 
        $message, "From: ".$sender." <".$senderemail.">");

if($success) echo "Success<br>";
else echo "Error<br>";
} 

?>

It reads the email addresses from a text filr.

For some reason it has "Error" written next to all emails after 120. It wont send more than that. Is there something in this that I can change? It's not for spamming, its for the people that signed up on my bands email list. There are over 200 of them (I think) but I know there's plenty more than 120 that arent getting the emails.

Thanks a million, it's much appreciated for the help.

Peredy

rupturedtoad

Might be worth putting sleep(1) after the mail command to see if your mailer isn't keeping up.

phphp

maybe it's mail-server limit

Peredy

What does sleep(1) do? I mean does it stop the email or would it fix the problem. I only ask cause I dont want to send out a million emails to everyone on the list just to test it.
Well, I dont have it going through a mail server per say. They are all different email addresses so I dont think that one would limit the rest. I mean, I'm not using an SMPT or anything like that. But The line that mentions counting the email addresses.... would that have something to do with it?

rupturedtoad

sleep(1) will delay execution of your script by one second. It could be that whatever is handling your mail is generating errors because it can't handle the speed at which your script is pumping the mail requests out. By putting in the sleep command it allows things to settle down before the next mail arrives.

This might not be the problem, but at least it eliminates something!

If you don't want to annoy your users by all this testing, mod the script to send everything to your email address until it's sorted out.

Peredy

Well, I added it and it freezes after 51 messages. I changed the list of emails to my own email address (2 different ones) for testing.

*** CORRECTION.. it pauses for a long time after each 50 messages. But it seems to be sending them Problem is that last time I tested it with my own email address's it worked too, then went back to the problem when each one is different. The line about counting the address's wouldnt be the problem? And setting the sleep to a different number... would that maybe speed it up? The delay is grouling long.

thorpe

I'm not using an SMPT or anything like that

well, how are you sending them?

Peredy

By that script alone. I suppose it is actually since it's going through my site, but I never specified anything like that. This script is a snippet I found in a yahoo search. What I want to know though is if that line sleep(1) is making it work, is there a way to speed it up? It takes VERY long breaks in between sending every 50 messages.

rupturedtoad

Try usleep, which delays execution by a number of microseconds - usleep(100000) will introduce a delay of one tenth of a second.

Peredy

Now the error came back after 120 emails again. I wonder why it knows that many as a count each time. I guess it does have to do with the server. Is there anyway around that?

Anyway, I changed it back to: sleep(1); and it worked. I guess it will just have to be slow, unless someone has another idea.