[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

[NetBeans]

Albo ja niedowidzę, albo nie napisałeś o co Ci chodzi. Oczekujesz może, że ktoś napisze za Ciebie skrypt, czy zapomniałes przedstawić problem?

[Killerek]

[PHP] pobierz, plaintext 
<?php
include 'inc/db.php'; // połączenie się z bazą danych
$tabela = 'add_obrazki'; // zdefiniowanie tabeli MySQL
$tabela2 = 'codes'; // zdefiniowanie tabeli MySQL
 
if(isset($_GET['id'])) {
 
    if ($_POST['wyslane']) { // jeżeli formularz został wysłany, to wykonuje się poniższy skrypt
 
        // filtrowanie treści wprowadzonych przez użytkownika
		$kod = $_POST["kod"];
 
		// system sprawdza czy prawidłowo zostały wprowadzone dane
 
        if ($kod !== $informacja['kod']) {
				$wynik = mysql_query("SELECT * FROM $tabela2 WHERE kod='$kod' and aktywuj='0'");
				if (mysql_num_rows($wynik) <> 1) {
					$blad++;
					echo '<div class="msg error">Ten kod był już aktywowany wcześniej.</div>';
				}
			}	
 
			// jeżeli błąd nie wystąpił, to dane zostają prawidłowo zapisane z bazie MySQL
         if ($blad == 0) {
 
              $wynik=mysql_query("UPDATE $tabela2 SET `aktywuj`='0' WHERE `kod`='$kod'");
				$wynik=mysql_query("UPDATE $tabela SET `plus`= +100 WHERE `id`='".$_GET['id']."'");
 
            } 
 
             if ($wynik) {
                echo '<div class="msg success">Konto zostało doładowane.</div>';
            } else {
                echo '<div class="msg error">Konto nie zostało doładowane.</div>';
            }
            }
 
    $wynik = mysql_query("SELECT * FROM $tabela WHERE id='".$_GET['id']."'");
    if ($wynik) {
        $informacja = mysql_fetch_array($wynik);
 
        // tworzenie formularza HTML z danymi użytkownika
        echo <<< KONIEC
 
	<form class="form" action="doladuj.php?id={$informacja['id']}" method="post">
				<input type="hidden" name="wyslane" value="TRUE" />
					Wprowadź swój 8 cyfrowy kod:<br>
					<input type="text" name="kod" id="kod" value="12w32a" />
				<input type="submit" value="Doładuj konto" />
	</form>
 
KONIEC;
    }
    mysql_close($polaczenie);
 
}
 
?>
[PHP] pobierz, plaintext 

Mam taki ale za kazdym razem mi wyskakuje "Konto zostało doładowane." nawet gdy kod jest zly ale i tak nie doladowuje ani nie zmienia na aktywuj='1'

Ten post edytował Killerek 28.03.2013, 23:59:42

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

[Killerek]

Hmmm, męcze się z tym od wczoraj i za kazdym razem jak zmienie caly skrypt to zawsze wynik byl taki sam... Czy moge was prosic o takiego 'gotowca' lub naprowadzic mnie na dobry trop? Dodam ze jestem poczatkujacy...

@refresh

[BORG]

Jak ma ci zmieniać aktywuj na 1 skoro masz

[PHP] pobierz, plaintext 
$wynik=mysql_query("UPDATE $tabela2 SET `aktywuj`='0' WHERE `kod`='$kod'");
[PHP] pobierz, plaintext 

I nie masz nigdzie zdefiniowanej zmiennej $blad, moze dla tego zawsze pokazuje Ci, że jest okej.

Pierwszy raz też widzę taki operator

[PHP] pobierz, plaintext 
if (mysql_num_rows($wynik) <> 1)
[PHP] pobierz, plaintext 

, co on znaczy?

Ten post edytował BORG 30.03.2013, 12:19:48