PHP masowa zmiana rekordow/aktywacja kodow. Opcje

[kurczak610]

Mam skrypt ktory po wpisaniu kilku kodow zmienia je i dodaje do konta 1 partie:

[PHP] pobierz, plaintext 
<form method="POST">
<table>
<tr><td>Kody:</td><td><textarea name="codes" cols="30" rows="10"></textarea></td></tr>
<tr><td></td><td><input type="submit" name="submit" value="Aktywuj"></td></tr>
</table>
</form><br>
<?php
if(isset($_POST['submit']))
{
  $explode=explode("\n",$_POST['codes']);
  $code=count($explode);
  $poprawne=0;
  $wpisane=0;
  $zle=0;
  
  for($i=0;$i<$code;$i++)
     {
        $kod = trim ($explode[$i]);
        $query=mysql_query("SELECT * FROM `kody` WHERE `kod`='$kod' AND `aktywowany`='0'") or die(mysql_error());
        if(mysql_num_rows($query)>0)
        {
           echo "<font color=\"GREEN\"><b>$kod</b> - Kod poprawny</font><br />";
           $poprawne=$poprawne+1;
           $wpisane=$wpisane+1;
           mysql_query("UPDATE users1 SET partie=partie+$poprawne WHERE id=$_SESSION[userid]") or die(mysql_error());
           mysql_query("UPDATE kody SET aktywowany=aktywowany+1, kto='".$_SESSION['username']."' WHERE kod='$kod'") or die(mysql_error());
        }
        else
        {
           echo "<font color=\"RED\"><b>$kod</b> - Kod niepoprawy</font><br />";
           $zle=$zle+1;
           $wpisane=$wpisane+1;
        }
     }
  echo "Wpisane kody: $wpisane <br>Poprawne: $poprawne <br> Niepoprawne: $zle<br>";
}
?>
[PHP] pobierz, plaintext 

Wszystko pisze ok, ale jak dodaje partie do dodaje ich za duzo.
Nizej pisze: Poprawne: 2
A do konta dodaje 3 partie.
Dlaczego tak jest?

[peter_zan]

Błąd jest tutaj, dodajesz zmienną $poprawne, która została wielokrotnie zwiększona w pętli w lini 23. I dodajesz ją wielokrotnie: raz jako '1' a potem jako '2'.

[SQL] pobierz, plaintext 
UPDATE users1 SET partie=partie+$poprawne WHERE id=$_SESSION[userid]
[SQL] pobierz, plaintext 

Dlatego masz 3 partie zamiast dwóch.

Ogólnie bym trochę pozmieniał kod, ale wystarczy jak zamiast +$poprawne w zapytaniu SQL i dasz +1, tak jak w zpytaniu aktywowane.


pzdr
Piotr Zaniewicz