PHP Server loop - php

Question

I have problem with my script
the problem with loop if someone can help me please
Problems:
I am starting loop perfectly without any problem with first loop only.
When result = 0 next loop start it gives me -1 -2 -3 etc in first account only.
and the correct way :
when next loop start its add +2 in smtp $server_index++; and start from first without sticky in first account and get minues numbers in same account .
$server_index = 0;
while($customer = mysql_fetch_assoc($result)){
//// Start Server Switch
$available_server_limit = $servers[$server_index]["per_day_limit"] - getEmailUsage($servers[$server_index]["id"],date("d/m/Y",time()));
if($available_server_limit==0){
if($server_index==$servers_count-1){
//exit(showError("Sorry! We don't have limit to send more emails"));
mysql_query("UPDATE `smtp_servers` SET `per_day_limit`=`per_day_limit`+2");
$server_index=0;
}else{
$server_index++;
}
}

Answer 1

It's really difficult to see exactly what you want from only this much info. However, I should point out that you're only going to hit $server_index++; IF $available_server_limit==0
If $available_server_limit does not equal 0, you'll get stuck.
The loop -- by the way -- is in a while statement relying on variables you're not using. May I suggest a foreach, or for statement?
On another note, please do away with mysql_* while you still can. It's depreciated as of recent versions of PHP.
See this for more info
I also notice that you don't show the hole loop. So with that in mind, here's my GUESS at the best way around this for YOU.
$server_index = 0;
foreach( $servers as $key=>$val )
{
$available_server_limit = $servers[$server_index]["per_day_limit"] - getEmailUsage($servers[$server_index]["id"],date("d/m/Y",time()));
if($available_server_limit==0)
{
if($server_index==$servers_count-1)
{
mysql_query("UPDATE `smtp_servers` SET `per_day_limit`=`per_day_limit`+2");
$server_index=0;
}
}
$server_index++;
Note: I didn't close the foreach loop because your while loop isn't closed. I figure you've got code after this that you're using.