[PHP] podwójne odejmowanie rekordu :/, gdzie jest błąd w php?? Opcje

[Kamil90]

Witam.
Mam dziwny i chyba rzadko spotykany problem (IMG:http://forum.php.pl/style_emoticons/default/wstydnis.gif) . Chodzi o to że mam formularz dodawania filmików, wprowadzam dane i wszystko jest ok. Pole 'typ' wynosi 7 tak jak być powinno...
(IMG:http://img159.imageshack.us/img159/9518/83638262bg6.jpg)

Później dane z formularza idą do weryfikacji do admina:

[PHP] pobierz, plaintext 
<?php
if ($admin && $row['typ'] >= 5 && $row['typ'] != 15 && $row['typ'] != 16 && $row['typ'] != 17)
 {
 echo '<p>
 Weryfikacja: <a href="index.php?page=display&amp;id=' . ((int) $_GET['id']) . '&amp;action=zatwierdz" title="ZatwierdĽ"><img src="ikonki/zatwierdz.gif"border="0">zatwierdĽ</a> <a href="index.php?page=display&amp;id=' . ((int) $_GET['id']) . '&amp;action=odrzuc" title="Odrzuć"><img src="ikonki/usun.gif"border="0">odrzuć</a>
 </p>';
?>
[PHP] pobierz, plaintext 

a nastepnie od pola 'typ' ma być odjęte -4 i problem jest w tym, że odejmuje zawsze dwa razy tyle, czyli odejmuje od 7-8 i mam -1 w mysql a powinno być 3 :/, jak wprowadze 2 zamiast 4 to bedzie dejmowało 4 :/ niestety przy kazdej wartości pola 'typ' musi byc odejmowane 4 dlatego nie moge zastapić tego 2. Oto kod w którym przerabiana jest weryfikacja:

[PHP] pobierz, plaintext 
<?php
if ($admin && isset($_GET['action']))
 {
   switch ($_GET['action'])
   {
   case 'zatwierdz':
     if (!mysql_query('update `stuff` set `typ` = `typ` - 4 where `id`=' . ((int) $_GET['id'])  ))
     {
       echo '<p>Nie udało się zaaktualizować rekordu.</p>';
     }
     else
     {
       echo '<p>Filmik został zatwierdzony poprawnie.</p>';
     }
     break;
 
   case 'odrzuc':
     if (mysql_query('delete from `komentarze` where `stuff`=' . ((int) $_GET['id'])) &&
     mysql_query('delete from `oceny` where `stuff`=' . ((int) $_GET['id'])) &&
     mysql_query('delete from `stuff` where `id`=' . ((int) $_GET['id'])))
     {
       echo '<p>Filmik został poprawnie usuniety.</p>';
     }
     else
     {
       echo '<p>Nie udało się usunąć rekordów z tabeli.</p>';
     }
     break;
   }
 }
?>
[PHP] pobierz, plaintext 

a oto efekt w bazie mysql:
(IMG:http://img530.imageshack.us/img530/6329/59122780hp6.jpg)

mam nadzieje że jesno przedstawiłem problem... bardzo proszę o pomoc.

Pozdrawiam

Ten post edytował erix 21.01.2009, 16:25:44

Powód edycji: bbcode + ortografia

[Zyx]

Jesteś pewien, że ten kod albo to zapytanie się dwukrotnie nie wykonuje? Ono samo działa poprawnie, zaś podane kawałki kodu nie mówią nic, dlaczego tak by się mogło dziać.

[Wykrywacz]

Używaj odpowiedniego BB-codu, bo mi się aż nie chce tego czytać aby ci pomóc. Pokaż więcej bo tak jak Zyx napisał tu wygląda wszystko wporzo

[EarthCitizen]

Wklej strukturę tabeli, pewnie robisz działania na stringu...

[Kamil90]

oto struktura tabeli:

(IMG:http://img120.imageshack.us/img120/8223/79618083sw3.jpg)

tutaj mysle ze chyba jest wszystko ok...

edit:

[PHP] pobierz, plaintext 
<?php
if ($admin && isset($_GET['action']))
 {
   switch ($_GET['action'])
   {
   case 'zatwierdz':
     if (!mysql_query('update `stuff` set `typ` = `typ` - 4 where `id`=' . ((int) $_GET['id'])  ))
     {
       echo '<p>Nie udało się zaaktualizować rekordu.</p>';
     }
     else
     {
 
[b]// I TUTAJ MOZE BY DODAC TE 4? (IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif)  Jaki w takim razie kod musiałbym wkleić?[/b]
       echo '<p>Filmik został zatwierdzony poprawnie.</p>';
     }
     break;
?>
[PHP] pobierz, plaintext 

Ten post edytował Kamil90 21.01.2009, 16:33:52

[EarthCitizen]

No masz inta, chociaż nie wiem po co Ci tam int, daj tinyint(2) albo enum.

A co do problemu, to może daj po prostu SET typ = 3 ?
Chociaż chyba lepiej abyś rzeczywiście sprawdził, czy ten kod nie wykonuje się 2 razy...

Ten post edytował EarthCitizen 21.01.2009, 16:24:10

[Kamil90]

no właśnie nie bardzo pasuje dac mi 'SET typ=3', bo typ będzie różny... Jestem pewien że ten odcinek wykonywany jest 2 razy bo jak wpisze '- 3' to odejmuje mi 6. Pomyślełem że po tych linijkach kodu

[PHP] pobierz, plaintext 
<?php
{
case 'zatwierdz':
if (!mysql_query('update `stuff` set `typ` = `typ` - 4 where `id`=' . ((int) $_GET['id']) ))
{
echo 'Nie udało się zaaktualizować rekordu.
 
';
}
else
{
?>
[PHP] pobierz, plaintext 

tutaj ponownie dodać te 4 do 'typ' ale jaki będzie kod?

Ten post edytował Kamil90 21.01.2009, 16:34:17

[Wykrywacz]

[SQL] pobierz, plaintext 
SELECT panstwa FROM panstwa LEFT JOIN miasta ON panstwa.id = miasta.panstwa_id
[SQL] pobierz, plaintext 

chyba "Ja chciałbym pobrać listę tylko tych państw, dla których w tabeli 'miasta' są dodane jakieś wpisy "

if (!mysql_query('update `stuff` set `typ` = `typ` - 4 where `id`=' . ((int) $_GET['id']) ))

a spróbuj zrobić tak

[PHP] pobierz, plaintext 
<?php
if (!mysql_query('update `stuff` set `typ` = (`typ` - 4) where `id`=' . ((int) $_GET['id']) ))
?>
[PHP] pobierz, plaintext 

[EarthCitizen]

tutaj ponownie dodać te 4 do 'typ' ale jaki będzie kod?

To nie ma sensu... znajdź błąd dlaczego kod wykonuje się 2 razy, a nie kombinuj z obejściem tego....

[Kamil90]

Wykrywacz włąsnie tak spróbowałem i niestety dalej to samo...(IMG:http://forum.php.pl/style_emoticons/default/sad.gif) a może ten kod który odejmuje 4 wrzucić by na koniec? albo jeszcze inaczej wpisać w pętle while i żeby 1 raz tylko wykonywało...

Ten post edytował Kamil90 21.01.2009, 17:10:53

[Zyx]

I właśnie tu by się teraz przydała przyzwoita biblioteka do komunikacji z bazami danych, a nie ciągle tylko te debilne mysql_cośtam, które już dawno powinny znaleźć się na śmietniku historii. Choćby nawet i PDO, bo wtedy od biedy można stworzyć prostą nakładkę, która wyświetla wszystkie zapytania i inaczej zainicjować obiekt - wtedy byś od razu widział, czy to wina skryptu, bo dwukrotnie wykonuje jakieś zapytanie, czy też może tego, że przez przypadek jest wywoływana strona dwa razy jakimś magicznym sposobem.

[Kamil90]

też o tym właśnie myślełem że może strona jest wywoływana dwa razy ale nie mam jak sprawdzić, ogólnie z problemem poradziłem sobie tak:

[PHP] pobierz, plaintext 
<?php
if ($admin && isset($_GET['action']))
{
  switch ($_GET['action'])
  {
  case 'zatwierdz':
    if (!mysql_query("update `stuff` set `typ` = `typ` - 4 where `id`=" . ((int) $_GET['id']  )))
    {
      echo '<p>Nie udało się zaaktualizować rekordu.</p>';
    }
    else
    {
(!mysql_query("update `stuff` set `typ` = `typ` + 2 where `id`=" . ((int) $_GET['id']  )));
      echo '<p>Filmik został zatwierdzony poprawnie.</p>';
 
    }
    break;
?>
[PHP] pobierz, plaintext