[PHP] [MYSQL] System kodów Opcje

[Killerek]

Witam, posiadam swoją stronę PHP i mam tam system punktów utworzyłem już taką tabelę:

[SQL] pobierz, plaintext 
CREATE TABLE IF NOT EXISTS `codes` (
  `kod` varchar(8) NOT NULL,
  `aktywuj` varchar(1) NOT NULL DEFAULT '0'
) ENGINE=InnoDB DEFAULT CHARSET=latin1;
[SQL] pobierz, plaintext 

Oraz zrobiłem takie zapytanie

[SQL] pobierz, plaintext 
INSERT INTO `codes` (`kod`, `aktywuj`) VALUES
('12w31a', '0'),
('12w32a', '0');
[SQL] pobierz, plaintext 

Oraz mam np. link http://www.adres-strony.pl/doladuj.php?id=15&kod=12w31a

I teraz skrypt powinnien wyszukać w tabelce 'uzytkownicy' uzytkownika o id 15 oraz w tabelece 'codes' ma wyszukać kod 12w31a i jeżeli w kolumnie aktywuj będzie 0 to wykonują się takie zapytania

[PHP] pobierz, plaintext 
$zapytanie = 'UPDATE uzytkownicy SET `punkty` = +100 WHERE `id` = '15'';
$zapytanie = 'UPDATE codes SET `status` = '1' WHERE `kod` = '12w31a'';
[PHP] pobierz, plaintext 

Oraz ma się w echo wyświetlić np.

[PHP] pobierz, plaintext 
echo '<div class="msg success">Konto zostało doładowane!</div>'
[PHP] pobierz, plaintext 

Ale jeżeli w kolumnie aktywuj będzie 1 to wyświetla się takie coś

[PHP] pobierz, plaintext 
echo '<div class="msg error">Konto nie zostało doładowane!</div>'
[PHP] pobierz, plaintext 

Ten post edytował Killerek 28.03.2013, 22:52:07

[weedget]

jeśli zapytanie:

[PHP] pobierz, plaintext 
$wynik=mysql_query("UPDATE $tabela SET `plus`= +100 WHERE `id`='".$_GET['id']."'");
[PHP] pobierz, plaintext 

nie zwróci błędu to będziesz miał taki komunikat, nigdzie nie inicjalizujesz zmiennej $wynik (wiem ze w PHP nie musisz, ale wtedy nie masz pewnosci ze jest "negatywna", a to może prowadzić do tego że warunek będzie prawdziwy).

Jednocześnie jeśli zrobiłbyś osobne sprawdzanie (w osobnej zmiennej) dla każdej kwerendy to możesz wyświetlić/zalogować odpowiedni komunikat o błędzie.

Dodatkowo nigdzie nie widzę filtrowania zmiennych przesyłanych od usera, a to dość niebezpieczne...