Jestem w trakcie tworzenia gry via www, obecnie pracuje nad przypominaczem który wysyła prywatną wiadomość na pocztę (wewnętrzną w grze) usera gdy pozostał 1 dzień konta premium.


Wygląda to tak:

[PHP] pobierz, plaintext 
 
$nump = @mysql_num_rows(mysql_query("select * from players"));
for($index=0;$index<$nump;$index++){
 
//powinno wysyłać do graczy którym pozostał 1 dzien konta premium
 
$gracz = mysql_fetch_array(mysql_query("select * from players where id='$index' and premium='1'"));
 
$temat= "Został 1 dzień konta premium";
$list="Cześć<br><br>Pozostał Ci tylko jeden dzień konta premium.<br> Aby przedłużyć ten okres przejdź do zakładki Premium.";
$data = date("d.m.y");
$sender = "Missy";
$senderid = "2";
 
mysql_query("insert into poczta (ownerid, subject, body, sender, senderid, data) values('$gracz[id]','$temat','$list','$sender','$senderid', '$data')") or die("Nie moża wysłać;
}
 
[PHP] pobierz, plaintext 

Jakie są efekty skryptu?

Co prawda przypomnienie dochodzi gdzie ma dojść ale oprócz tego wysyła (tyle wiadomości ile wynosi $nump) pod ID = 0

Czyli zapycha baze danych na darmo.

Co jest źle w skrypcie?

pętla wykonuje się tyle razy ilu masz userów w bazie danych, a powinna się wykonywać tyle razy ile masz userów w bazie, którym pozostał 1 dzień konta premium

[PHP] pobierz, plaintext 
$result = mysql_query("select * from players where premium='1'"); 
$numrows = mysql_num_rows($result);
if($numrows > 0) {
     while ($row = mysql_fetch_array($result)) {
           $temat= "Został 1 dzień konta premium";
           $list="Cześć<br><br>Pozostał Ci tylko jeden dzień konta premium.<br> Aby przedłużyć ten okres przejdź do zakładki Premium.";
           $data = date("d.m.y");
           $sender = "Missy";
           $senderid = "2";
 
           mysql_query("insert into poczta (ownerid, subject, body, sender, senderid, data) values('$row['id']','$temat','$list','$sender','$senderid', '$data')") or die("Nie moża wysłać;
     }
}
 
[PHP] pobierz, plaintext