[PHP][MySQL] Aktywacja konta nowo zarejestrowanego użytkownika Opcje

[jg44]

Witam,
napisałem skrypt rejestrowania się nowych użytkowników na stronie i chciałbym, aby po wypełnieniu formularza rejestracyjnego użytkownik miał możliwość aktywowania konta poprzez kliknięcie linku. Napisałem takie coś i nie działa:

[PHP] pobierz, plaintext 
// rejestracja.php
 
class Register
{
   (...) kod klasy
 
  function addNewUser()
  {
     $uniq1 = uniqid();
     $uniq2 = uniqid();
     $uniq = $uniq1.$uniq2;
 
     (...) kod funkcji
 
     mysql_query("INSERT INTO uzytkownicy ... ");    // dodanie do bazy danych użytkownika
     echo "<h5 class=\"correct\">Rejestracja zakończona powodzeniem!</h5>";
 
     echo "Kliknij w link aktywacyjny: <a href='http://adres/aktywuj.php?iden=$uniq'>Link</a>";
   }
}
[PHP] pobierz, plaintext 

Plik, do którego zostaje przekierowany użytkownik, który się zarejestrował i chce aktywować konto:

[PHP] pobierz, plaintext 
  $iden = trim($_GET['iden']);
  mysql_query("UPDATE uzytkownicy SET status=1 WHERE identyfikator='$iden'"); 
  echo "<h5 class=\"correct\">Twoje konto zostało aktywowane.</h5>";
  echo "<a href=http://adres/portal>Strona główna</a>";
[PHP] pobierz, plaintext 

Po otwarciu drukowany jest tekst, że "Twoje konto zostało aktywowane.", ale tak niestety nie jest.
Mógłby ktoś pomóc i powiedzieć, co robię źle?
Z góry dziękuję.

[mortus]

Wszystko wygląda dobrze, jednak dla klucza aktywacyjnego utworzyłbym osobną tabelę klucze, w której przechowywałbym id użytkownika i klucz. Jest to tylko kwestia tego, że z tabeli użytkowników z pewnością będziesz korzystał często, ale z tej unikalnej informacji jaką jest klucz aktywacyjny skorzystasz być może tylko raz. Zatem kolumna z kluczem w tabeli użytkowników jest rzeczą zbędną.

Pozostaje pytanie, dlaczego to nie działa. Może posługujesz się złymi kolumnami? A może w kolumnie identyfikator w tabeli w bazie danych nie możesz zapisać tylu znaków ile wygeneruje konkatenacja dwóch wyników uniqid() i w efekcie zapisujesz tylko część klucza.

[jg44]

Faktycznie, pomysł z drugą tabelą jest dobry, zrobię tak.
Na razie jednak sprawdzam, czy rzeczywiście nie ma jakiegoś głupiego błędu i widzę, że wszystko jest ok, posługuję się dobrymi kolumnami, w zapytaniu nie ma literówek, dodatkowo zapytanie UPDATE'ujące wykonałem w phpMyAdmin i wszystko pięknie zadziałało. Problem jest tylko z poziomu skryptu.
Co do nie mieszczenia się klucza w tabeli to raczej nie, gdyż uniqid() generuje ciąg 13-znakowy, czyli konkatenacja dwóch takich ciągów będzie miała 26 znaków. W tabeli ustawiłem pole dla klucza na VARCHAR(100), więc musi się mieścić.

[mortus]

Porównaj zawartość $_GET['iden'] (wyświetl ją sobie tymczasowo na stronie) z zawartością tabeli w phpMyAdminie. Dodaj po mysql_query() kod sprawdzający, czy zapytanie zostało wykonane:

[PHP] pobierz, plaintext 
mysql_query("UPDATE uzytkownicy SET status=1 WHERE identyfikator='$iden'") or die(mysql_error());
[PHP] pobierz, plaintext 

Ten post edytował mortus 16.11.2011, 16:48:52

[jg44]

Już działa. Jak dodałem mysql_error() to wyrzucił błąd, który wszystko wyjaśnił. Skoro pracuję na bazie danych, to warto by było nawiązać z nią połączenie (po prostu w pliku aktywacji konta zapomniałem zainclude'ować dane bazy). Dziękuję, mortus, sam bym pewnie na to w końcu wpadł, ale minęłoby trochę czasu.

Ten post edytował jg44 16.11.2011, 17:12:57