[PHP][MySQL] Aktywacja konta przy rejestracji Opcje

[hunter1988]

Wszystko powinno działać a gdzieś jest mały błąd

[PHP] pobierz, plaintext 
	public function activation($kod)
	{
		$sql_check = "SELECT * FROM `users` WHERE `kod`='$kod' and `aktywacja`='1'";
		$sql_update = "UPDATE `users` SET `aktywacja`='1' WHERE `kod`='$kod' and `aktywacja`='0'";
 
 
		$stmt = $this->pdo -> exec($sql_check);
 
		if($stmt > 0)
		{
			echo "Aktywowałeś swoje konto już wcześniej";
		}
		else
		{
			$this->pdo -> exec($sql_update);
		}
 
 
	}
 
[PHP] pobierz, plaintext 

Połączenie z bazą jest, zapytanie $sql_update działa na 100% bo wklepywałem je bez warunków i aktywuje konto, coś jest nie tak z $sql_check.
Połączenie UPDATE działa, a już np w warunkach if nie, w tym przykładzie nie wykona się NIC, nie wyświetli się żaden błąd sql ani php, składnia więc jest "teoretycznie ok", zapytanie $sql_check na bank nie działa jak powinno (szukałem literówek i nic).

Powinno być tak skrypt sprawdza czy konto z danym kodem ma już aktywację (w bazie danych 1) i to jest warunek, funkcja exec sprawdza ilość wyników zapytania, w tym przypadku nawet gdy podaje kod i w bazie jest 0 to zawsze zwraca wynik 0 dla zapytania $sql_check, no ok ale skoro dało 0 to powinno wykonać się else czyli aktywacja konta, podczas gdy ten zabieg nie wykonuje się, mimo że $this->pdo -> exec($sql_update); działa.

Na pewno jest to jakaś głupota, ale siedzę przy tym i siedzę i nie mogę znaleźć błędu...

[phpion]

W warunkach przy select masz aktywacja=1, a przy update aktywacja=0.

[hunter1988]

W warunkach przy select masz aktywacja=1, a przy update aktywacja=0.

Kod jest przechowywany w bazie np
kod to 5005 i aktywował konto czyli 1

Pierwszy warunek sprawdza czy ktoś nie próbuje aktywować konta jeszcze raz bo np po 2 dniach przeglądał pocztę jeszcze raz i postanowił kliknąć w link wtedy exec zwróci wartość większą od 0 i update nie odbędzie się i tu jest coś nie tak z zapytaniem prawdopodobnie.
Drugie zapytanie po prostu aktywuje konto

Ten post edytował hunter1988 4.06.2013, 12:59:16

[gitbejbe]

weź napisz to jeszcze raz ;p

ja po 1 zrobiłbym tak, ze zrobiłbym JEDNO zapytanie, gdzie pobieram sobie to co tam chce + pole czy konto jest aktywne.
czyli

[PHP] pobierz, plaintext 
if(znalazł konto)
{
    // teraz sprawdzasz z pobranego rekordu czy aktywacja konta = 1
     if(jeśli aktywne) 
    {
         zaloguj
     }
    else
    {
         wywal komunikat ze nie zostało jeszcze aktywowane
    }
}
else(jesli nie znajdzie)
{
     brak takiego konta
}
[PHP] pobierz, plaintext 

[hunter1988]

Grzebałem grzebałem i zrobiłem w ten sposób, problem był taki, że exec nie zwraca bezpośrednio liczby rekordów (widocznie, źle zrozumiałem czytając bibliotekę PDO)

[PHP] pobierz, plaintext 
 
	public function activation($kod)
	{
		// tu można jeszcze dopisać zapytanie sprawdzające czy kod w ogóle istnieje taki w bazie, w razie gdyby użytkownik spreparował kod ręcznie, a nie klikał w link w mailu 	
		$stmt = $this->pdo -> prepare("SELECT * FROM `users` WHERE `kod`=:kod and `aktywacja`=:aktywacja");
		$stmt->bindValue(':kod',$kod,PDO::PARAM_STR);
		$stmt->bindValue(':aktywacja','1',PDO::PARAM_INT);
		$stmt->execute();
		$how_many = $stmt->rowCount();
		if($how_many > 0)
		{
			echo "Aktywowałeś swoje konto już wcześniej";
		}
		else
		{
 
			$this->pdo -> exec("UPDATE `users` SET `aktywacja`='1' WHERE `kod`='$kod' and `aktywacja`='0'");
                        echo "Konto zostało pomyślnie aktywowane";
		}
	}
[PHP] pobierz, plaintext 

gitbejbe trochę nie widzę sensu w pisanie tego w taki sposób, po co mi logowanie w aktywowaniu konta, tylko powiedz mi jeszcze jak można zrobić JEDNO zapytanie do update i select jednocześnie, bo ja jestem początkującym mocno ale wydaje mi się, że nie ma takiego zapytania (IMG:style_emoticons/default/wink.gif)

Ten post edytował hunter1988 5.06.2013, 09:06:14

[gitbejbe]

zwracam honor. Byłem świecie przekonany, że chodzi o sprawdzenie aktywacji konta przy logowaniu (IMG:style_emoticons/default/facepalmxd.gif) tak to jest gry siedzi się w pracy i stara się jednocześnie pracować i opierdzielać ;p