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

[kurczak610]

Dzięki, dalem pomogł, jezeli moge spytac co zmieniłbyś w kodzie? (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[peter_zan]

O na przykład tak:

[PHP] pobierz, plaintext 
<?php
if(isset($_POST['submit'])) {
    $codes=explode("\n",$_POST['codes']);
    $poprawne=0;
    $wpisane=0;
    $zle=0;
    foreach($codes as $i => $code) { //usuwa puste wpisy i wszystkie znaki inne niż alfanumeryczne (np. jeśli ktoś zostawi pustą linię)
        $code = preg_replace('/[^a-zA-Z0-9]/i', '', $code);
        $codes[$i] = $code;
        if (is_null($code) || $code == "") {
            unset($codes[$i]);
        }
    }
    $codes = array_values($codes); //sortuje ponownie tablicę
 
    foreach($codes as $code){
        $query=mysql_query("SELECT * FROM `kody` WHERE `kod`='$code' AND `aktywowany`='0'") or die(mysql_error());
        if(mysql_num_rows($query)>0) {
            echo "<font color=\"GREEN\"><b>$code</b> - Kod poprawny</font><br />";
            mysql_query("UPDATE users1 SET partie=partie+1 WHERE id='".$_SESSION[userid]."'") or die(mysql_error());
            mysql_query("UPDATE kody SET aktywowany=aktywowany+1, kto='".$_SESSION['username']."' WHERE kod='$code'") or die(mysql_error());
            $poprawne++;
        }
        else {
            echo "<font color=\"RED\"><b>$code</b> - Kod niepoprawy</font><br />";
            $zle++;
        }
 
    $wpisane++;
    } //foreach end
 
    echo "Wpisane kody: $wpisane <br>Poprawne: $poprawne <br> Niepoprawne: $zle<br>";
}//if submit end
?>
[PHP] pobierz, plaintext 

Powodzenia,
PZ

Ten post edytował peter_zan 4.06.2009, 14:26:21

[kurczak610]

Jest blad:

Parse error: syntax error, unexpected T_ELSE in /home/kamil610/domains/testd-k.unl.pl/public_html/Lay/aktywuj21.php on line 27

Co jest zle? (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)

[Savage.Mephisto]

Umknął Ci nawias klamrowy. Powinno być:

[PHP] pobierz, plaintext 
<?php
if(mysql_num_rows($query)>0) {
?>
[PHP] pobierz, plaintext 

[peter_zan]

No tak, korzystałem z innego ifa, a tego skopiowałem...

pzdr
PZ