Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: PHP masowa zmiana rekordow/aktywacja kodow.
Forum PHP.pl > Forum > PHP
kurczak610
Mam skrypt ktory po wpisaniu kilku kodow zmienia je i dodaje do konta 1 partie:
  1. <form method="POST">
  2. <table>
  3. <tr><td>Kody:</td><td><textarea name="codes" cols="30" rows="10"></textarea></td></tr>
  4. <tr><td></td><td><input type="submit" name="submit" value="Aktywuj"></td></tr>
  5. </table>
  6. </form><br>
  7. <?php
  8. if(isset($_POST['submit']))
  9. {
  10.  $explode=explode("\n",$_POST['codes']);
  11.  $code=count($explode);
  12.  $poprawne=0;
  13.  $wpisane=0;
  14.  $zle=0;
  15.  
  16.  for($i=0;$i<$code;$i++)
  17.     {
  18.        $kod = trim ($explode[$i]);
  19.        $query=mysql_query("SELECT * FROM `kody` WHERE `kod`='$kod' AND `aktywowany`='0'") or die(mysql_error());
  20.        if(mysql_num_rows($query)>0)
  21.        {
  22.           echo "<font color=\"GREEN\"><b>$kod</b> - Kod poprawny</font><br />";
  23.           $poprawne=$poprawne+1;
  24.           $wpisane=$wpisane+1;
  25.           mysql_query("UPDATE users1 SET partie=partie+$poprawne WHERE id=$_SESSION[userid]") or die(mysql_error());
  26.           mysql_query("UPDATE kody SET aktywowany=aktywowany+1, kto='".$_SESSION['username']."' WHERE kod='$kod'") or die(mysql_error());
  27.        }
  28.        else
  29.        {
  30.           echo "<font color=\"RED\"><b>$kod</b> - Kod niepoprawy</font><br />";
  31.           $zle=$zle+1;
  32.           $wpisane=$wpisane+1;
  33.        }
  34.     }
  35.  echo "Wpisane kody: $wpisane <br>Poprawne: $poprawne <br> Niepoprawne: $zle<br>";
  36. }
  37. ?>


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'.

  1. UPDATE users1 SET partie=partie+$poprawne WHERE id=$_SESSION[userid]


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? smile.gif
peter_zan
O na przykład tak:

  1. <?php
  2. if(isset($_POST['submit'])) {
  3.    $codes=explode("\n",$_POST['codes']);
  4.    $poprawne=0;
  5.    $wpisane=0;
  6.    $zle=0;
  7.    foreach($codes as $i => $code) { //usuwa puste wpisy i wszystkie znaki inne niż alfanumeryczne (np. jeśli ktoś zostawi pustą linię)
  8.        $code = preg_replace('/[^a-zA-Z0-9]/i', '', $code);
  9.        $codes[$i] = $code;
  10.        if (is_null($code) || $code == "") {
  11.            unset($codes[$i]);
  12.        }
  13.    }
  14.    $codes = array_values($codes); //sortuje ponownie tablicę
  15.  
  16.    foreach($codes as $code){
  17.        $query=mysql_query("SELECT * FROM `kody` WHERE `kod`='$code' AND `aktywowany`='0'") or die(mysql_error());
  18.        if(mysql_num_rows($query)>0) {
  19.            echo "<font color=\"GREEN\"><b>$code</b> - Kod poprawny</font><br />";
  20.            mysql_query("UPDATE users1 SET partie=partie+1 WHERE id='".$_SESSION[userid]."'") or die(mysql_error());
  21.            mysql_query("UPDATE kody SET aktywowany=aktywowany+1, kto='".$_SESSION['username']."' WHERE kod='$code'") or die(mysql_error());
  22.            $poprawne++;
  23.        }
  24.        else {
  25.            echo "<font color=\"RED\"><b>$code</b> - Kod niepoprawy</font><br />";
  26.            $zle++;
  27.        }
  28.  
  29.    $wpisane++;
  30.    } //foreach end
  31.  
  32.    echo "Wpisane kody: $wpisane <br>Poprawne: $poprawne <br> Niepoprawne: $zle<br>";
  33. }//if submit end
  34. ?>


Powodzenia,
PZ
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? tongue.gif
Savage.Mephisto
Umknął Ci nawias klamrowy. Powinno być:

  1. <?php
  2. if(mysql_num_rows($query)>0) {
  3. ?>
peter_zan
No tak, korzystałem z innego ifa, a tego skopiowałem...

pzdr
PZ
To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.
Invision Power Board © 2001-2025 Invision Power Services, Inc.