Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: [PHP][MYSQL] Nick/Hasło < 6 znaków - funkcja strlen
Forum PHP.pl > Forum > Przedszkole
Kerth
16.03.2013, 15:20:42
Cześć,
mam taki problem z funkcją strlen w php.
Otóż. Próbuję wykonać własny system rejestracji kicha(jak na razie)
Do rzeczy. Nie wiem czy dobrze skonstruowałem instrukcję if...elseif...else, bo nawet jeśli gracz wpisze nick mniejszy niż 6 znaków lub poprawnie to i tak rejestruje go wysyłając komunikat o błędzie("Nick nie może być, krótszy niż 6 znaków!")

[PHP] pobierz, plaintext 
  1. if ( !empty($nick) || !empty($email) || !empty($haslo) ) { 
  2.  
  3.     // dodajemy rekord do bazy 
  4.  
  5.     $sql = "INSERT INTO `GRACZE` (`nick`, `email`, `haslo`) VALUES('".$nick."', '".$email."', '".$haslo."')";
  6.     if ( !($result = mysql_query($sql)) )
  7.     {
  8.         die('error in sql<br>'.mysql_error());
  9.     }
  10.     else if(!strlen($_POST["nick"])<6)
  11.     {
  12.         echo "<br><b><font color = red>Nick nie może być, krótszy niż 6 znaków!</font></b></br>";
  13.     }
  14.     else if(!strlen($_POST["haslo"])<6)
  15.     {
  16.         echo "<br><b><font color = red>Hasło nie może być krótsze niż 6 znaków!</font></b></br>";
  17.     }
  18.     else 
  19.     {
  20.         echo "<b><font color = green>Zostałeś pomyślnie zarejestrowany.</color></b>"; 
  21.     }
  22.     mysql_close($connection); 
  23. }
[PHP] pobierz, plaintext



Wie ktoś co jest nie tak? Proszę o jakieś rady.
Michael2318
16.03.2013, 15:30:37
ehh.
Spójrz na to co podałeś... Najpierw wrzucasz zapytanie do bazy, a później zaczynasz dopiero sprawdzać czy nick/hasło itd. nie są za krótkie. Logiczne to jest według Ciebie? Według mnie najpierw należałoby sprwadzac dlugość, potem dodawać wpis do bazy.

[PHP] pobierz, plaintext 
  1. if ( !empty($nick) || !empty($email) || !empty($haslo) ) { 
  2.     if(!strlen($_POST["nick"])<6)
  3.     {
  4.         echo "<br><b><font color = red>Nick nie może być, krótszy niż 6 znaków!</font></b></br>";
  5.         exit;
  6.     }
  7.     else if(!strlen($_POST["haslo"])<6)
  8.     {
  9.         echo "<br><b><font color = red>Hasło nie może być krótsze niż 6 znaków!</font></b></br>";
  10.         exit;
  11.     }
  12.     else 
  13.     {
  14.         echo "<b><font color = green>Zostałeś pomyślnie zarejestrowany.</color></b>";
  15.         exit;
  16.     }
  17.  
  18.     // dodajemy rekord do bazy 
  19.  
  20.     $sql = "INSERT INTO `GRACZE` (`nick`, `email`, `haslo`) VALUES('".$nick."', '".$email."', '".$haslo."')";
  21.     if ( !($result = mysql_query($sql)) )
  22.     {
  23.         die('error in sql<br>'.mysql_error());
  24.     }
  25.     else
  26.     {
  27.          echo 'Zarejestrowano poprawnie!';
  28.     }
  29.     mysql_close($connection); 
  30. }
[PHP] pobierz, plaintext
Kerth
16.03.2013, 15:40:45
Ale teraz cały czas pojawia się komunikat: Nick nie może być, krótszy niż 6 znaków! nawet jeśli wszystko poprawnie wpiszę. I dodatkowo powinien pojawić się drugi komunikat Hasło nie może być, krótsze niż 6 znaków! a nic się nie pojawia. Co może być problemem?
Michael2318
16.03.2013, 15:44:34 
[PHP] pobierz, plaintext 
  1. if ( !empty($nick) || !empty($email) || !empty($haslo) ) { 
  2.    $message = '';
  3.     if(!strlen($_POST["nick"])<6)
  4.     {
  5.         $message = "<br><b><font color = red>Nick nie może być, krótszy niż 6 znaków!</font></b></br>";
  6.     }
  7.     else if(!strlen($_POST["haslo"])<6)
  8.     {
  9.         $message .= "<br><b><font color = red>Hasło nie może być krótsze niż 6 znaków!</font></b></br>";
  10.     }
  11.     if ( $message != '' )
  12.     {
  13.        die($message);
  14.     }
  15.  
  16.     // dodajemy rekord do bazy 
  17.  
  18.     $sql = "INSERT INTO `GRACZE` (`nick`, `email`, `haslo`) VALUES('".$nick."', '".$email."', '".$haslo."')";
  19.     if ( !($result = mysql_query($sql)) )
  20.     {
  21.         die('error in sql<br>'.mysql_error());
  22.     }
  23.     else
  24.     {
  25.          echo 'Zarejestrowano poprawnie!';
  26.     }
  27.     mysql_close($connection); 
  28. }
[PHP] pobierz, plaintext
Kerth
16.03.2013, 15:55:03
Dalej to samo się dzieje. Może da się jakoś inaczej?
!*!
16.03.2013, 16:05:47
Zdecyduj się do czego się odwołujesz. Do zmiennych czy do POST ?
edycja:

Poza tym instrukcja if jest bez sensu u Ciebie, dlaczego ona jest niby połączona i zależna od zmiennej nick? Jedno nie powinno wykluczać drugiego.

http://pl.wikibooks.org/wiki/PHP/Instrukcja_if

I zainteresuj się też isset, empty to za mało.
Kerth
16.03.2013, 22:36:33
Ok.

Tylko teraz mam taki kod:


[PHP] pobierz, plaintext 
  1. <html> 
  2. <head> 
  3. <title>Proces rejestracji konta</title> 
  4. </head> 
  5. <body> 
  6.       <form action="register.php" method="post"> 
  7.            Nick:<br /> 
  8.       <input type="text" name="nick" /><br /> 
  9.            Hasło:<br /> 
  10.       <input type="password" name="haslo" /><br /> 
  11.            E-mail:<br /> 
  12.       <input type="text" name="email" /><br /> 
  13.  
  14.       <input type="submit" value="Zarejestruj" /> 
  15.      </form>
  16.  
  17. </body> 
  18. </html>
  19.  
  20. <?php 
  21. error_reporting(E_ALL);
  22. ini_set('display_errors', 1);
  23. // łączenie z bazą danych
  24. include 'inc/sql.php';
  25.  
  26. // odbieramy dane z formularza 
  27. $nick = mysql_real_escape_string(stripslashes($_POST["nick"]));
  28. $email = mysql_real_escape_string(stripslashes($_POST["haslo"]));
  29. $haslo = mysql_real_escape_string(stripslashes($_POST["email"]));
  30.  
  31. if (!empty($nick) && !empty($email) && !empty($haslo) ) 
  32. { 
  33.      if(strlen($nick)>6){
  34.      if(strlen($haslo)>6){
  35.      if(preg_match("[a-zA-Z0-9_\\.]+@([a-zA-Z0-9\\-]+\\.)+[a-zA-Z]{2,4}" , $email))
  36.      {
  37.      mysql_query("INSERT INTO `GRACZE` (`nick`, `email`, `haslo`) VALUES('$nick', '$email', '$haslo')") or die(mysql_error());
  38.  
  39.      echo "<b><font color= green>Zostałeś pomyślnie zarejestrowany.</color></b>"; 
  40.      mail("$email", "Witamy na pokładzie!", "Witaj $nick! Poprawnie założyłeś swoje konto. Możesz teraz się zalogować i zacząć swoją przygodę. Pozdrawiamy.); 
  41.      }
  42.      else 
  43.      {
  44.          echo "<br><b><font color= red >Niepoprawny email!</font></b></br>";
  45.      }
  46.      else 
  47.      {
  48.          echo "<br><b><font color= red>Hasło nie może być krótsze niż 6 znaków!</font></b></br>";
  49.      }
  50.      else
  51.      {
  52.          echo "<br><b><font color= red\">Nick nie może być, krótszy niż 6 znaków!</font></b><br>";
  53.      }
  54. }
  55.  
  56. ?>
[PHP] pobierz, plaintext


I nie wiem dlaczego dostaję taki błąd:

[PHP] pobierz, plaintext 
  1. Parse error: syntax error, unexpected '>' in /virtual/dawid4157.cba.pl/gra.dawid4157.cba.pl/register.php on line 43
[PHP] pobierz, plaintext


Jest to odniesienie do linijki 43 czyli:

[PHP] pobierz, plaintext 
  1. echo "<br><b><font color= red >Niepoprawny email!</font></b></br>";
[PHP] pobierz, plaintext


Jak to naprawić?
Michael2318
16.03.2013, 22:38:34
Linia 40, funkcja mail(). Nawet kolor skladni Ci to podpowiada...
Kerth
16.03.2013, 22:46:09
Ale chyba nie ma to znaczenia, bo funkcja jest (chyba) dobra, ale nic. Po wyrzuceniu jej błąd pojawia się w linijce 46 czyli pierwszym "else"
Michael2318
16.03.2013, 22:54:35 
[PHP] pobierz, plaintext 
  1. mail("$email", "Witamy na pokładzie!", "Witaj $nick! Poprawnie założyłeś swoje konto. Możesz teraz się zalogować i zacząć swoją przygodę. Pozdrawiamy.);
[PHP] pobierz, plaintext


Pomyśl logicznie - czego tu brakuje? Podpowiem Ci, że chodzi o koniec ciągu.
Kerth
16.03.2013, 23:00:08
No wiem brakuje " ale nie o to chodzi. Tą funkcję wyrzuciłem i mam błąd jak w poście wyżej. Odnosi się do pierwszego else:



[PHP] pobierz, plaintext 
  1. <html> 
  2. <head> 
  3. <title>Proces rejestracji konta</title> 
  4. </head> 
  5. <body> 
  6.       <form action="register.php" method="post"> 
  7.            Nick:<br /> 
  8.       <input type="text" name="nick" /><br /> 
  9.            Hasło:<br /> 
  10.       <input type="password" name="haslo" /><br /> 
  11.            E-mail:<br /> 
  12.       <input type="text" name="email" /><br /> 
  13.  
  14.       <input type="submit" value="Zarejestruj" /> 
  15.      </form>
  16.  
  17. </body> 
  18. </html>
  19.  
  20. <?php 
  21. error_reporting(E_ALL);
  22. ini_set('display_errors', 1);
  23. // łączenie z bazą danych
  24. include 'inc/sql.php';
  25.  
  26. // odbieramy dane z formularza 
  27. $nick = mysql_real_escape_string(stripslashes($_POST["nick"]));
  28. $email = mysql_real_escape_string(stripslashes($_POST["haslo"]));
  29. $haslo = mysql_real_escape_string(stripslashes($_POST["email"]));
  30.  
  31. if (!empty($nick) && !empty($email) && !empty($haslo) ) 
  32. { 
  33.      if(strlen($nick)>6){
  34.      if(strlen($haslo)>6){
  35.      if(preg_match("[a-zA-Z0-9_\\.]+@([a-zA-Z0-9\\-]+\\.)+[a-zA-Z]{2,4}" , $email))
  36.      {
  37.      mysql_query("INSERT INTO `GRACZE` (`nick`, `email`, `haslo`) VALUES('$nick', '$email', '$haslo')") or die(mysql_error());
  38.  
  39.      echo "<b><font color= green>Zostałeś pomyślnie zarejestrowany.</color></b>"; 
  40.      //mail("$email", "Witamy na pokładzie!", "Witaj $nick! Poprawnie założyłeś swoje konto. Możesz teraz się zalogować i zacząć swoją przygodę. Pozdrawiamy."); 
  41.      }
  42.      else 
  43.      {
  44.          echo "<br><b><font color= red >Niepoprawny email!</font></b></br>";
  45.      }
  46.      else  
  47.      {
  48.          echo "<br><b><font color= red>Hasło nie może być krótsze niż 6 znaków!</font></b></br>";
  49.      }
  50.      else 
  51.      {
  52.          echo "<br><b><font color= red>Nick nie może być, krótszy niż 6 znaków!</font></b><br>";
  53.      }
  54.  
  55. }
  56.  
  57. ?>	
[PHP] pobierz, plaintext
_Borys_
16.03.2013, 23:41:10
} po 45 i po 49 brakuje
Kerth
17.03.2013, 13:49:02
Ok. Pokombinowałem trochę i wyskrobałem coś takiego:

[PHP] pobierz, plaintext 
  1. <html> 
  2. <head> 
  3. <title>Proces rejestracji konta</title> 
  4. </head> 
  5. <body> 
  6.       <form action="register.php" method="post"> 
  7.            Wpisz Nick:<br /> 
  8.       <input type="text" name="nick" /><br /> 
  9.            Wpisz hasło:<br /> 
  10.       <input type="password" name="haslo" /><br /> 
  11.            Wpisz ponownie hasło:<br /> 
  12.       <input type="password" name="haslo2" /><br /> 
  13.            Wpisz E-mail:<br /> 
  14.       <input type="text" name="email" /><br /> 
  15.           
  16.       <input type="submit" value="Zarejestruj" /> 
  17.      </form>
  18.  
  19. </body> 
  20. </html>
  21.  
  22. <?php 
  23. error_reporting(E_ALL);
  24. ini_set('display_errors', 1);
  25. // łączenie z bazą danych
  26. include 'inc/sql.php';
  27.  
  28. // odbieramy dane z formularza 
  29. $nick = @mysql_real_escape_string(stripslashes($_POST["nick"]));
  30. $email = @mysql_real_escape_string(stripslashes($_POST["email"]));
  31. $haslo = @mysql_real_escape_string(stripslashes($_POST["haslo"]));
  32. $haslo2 = @mysql_real_escape_string(stripslashes($_POST["haslo2"]));
  33.  
  34.  
  35. if (!empty($nick) && !empty($email) && !empty($haslo) && !empty($haslo2) ) 
  36. { 
  37.      if(strlen($nick)<6){
  38.      if(strlen($haslo)<6){
  39.      if($haslo != $haslo2){
  40.  
  41.      {
  42.      mysql_query("INSERT INTO `GRACZE` (`nick`, `email`, `haslo`, `haslo2`) VALUES('$nick', '$email', '$haslo', '$haslo2')") or die(mysql_error());
  43.  
  44.      echo "<b><font color= green>Zostałeś pomyślnie zarejestrowany.</color></b>"; 
  45.      mail("$email", "Witamy na pokładzie!", "Witaj $nick! Poprawnie założyłeś swoje konto. Możesz teraz się zalogować i 
    zacząć swoją przygodę. Pozdrawiamy."    ); 
  46.      }
  47.      }
  48.      else  
  49.      {
  50.          echo "<br><b><font color= red>Hasło nie może być krótsze niż 6 znaków!</font></b></br>";
  51.      }
  52.      }
  53.      else 
  54.      {
  55.          echo "<br><b><font color= red>Nick nie może być, krótszy niż 6 znaków!</font></b><br>";
  56.      }
  57.      }
  58.      else 
  59.      {
  60.          echo "<br><b><font color= red>Hasła nie są takie same!</font></b><br>";
  61.      }
  62.      
  63. }
  64. else 
  65. {
  66.     echo "<br><b><font color= red>Musisz wypełnić wszystkie pola!</font></b><br>";
  67. }
  68. ?>
[PHP] pobierz, plaintext


W tym skrypcie jest pewien błąd, którego nie potrafię naprawić. Objawia się to w ten sposób, że po wpisaniu poprawnych(przykładowych danych):

LOGIN: dawid4157
HASŁO: dawid1996
POWTÓRZONE: dawid1996
E-MAIL: abcd@vp.pl

I pojawia się komunikat: "Hasła nie są takie same!"

Czasem też pojawia się komunikat o "Hasło nie może być krótsze niż 6 znaków!" lub inne. Nie wiem czemu to tak dziwnie jest. Może mi ktoś jakoś rzetelnie wytłumaczyć co ja źle zrobiłem? Czasem jest tak a czasem tak.
Michael2318
17.03.2013, 13:53:35 
[PHP] pobierz, plaintext 
  1. $nick = @mysql_real_escape_string(stripslashes($_POST["nick"]));
  2. $email = @mysql_real_escape_string(stripslashes($_POST["email"]));
  3. $haslo = @mysql_real_escape_string(stripslashes($_POST["haslo"]));
  4. $haslo2 = @mysql_real_escape_string(stripslashes($_POST["haslo2"]));
[PHP] pobierz, plaintext


Po co tyle tego tam? Jedna funkcja wystarczy...
Po drugie:
- do filtracji emaila używamy takiego warunku:
[PHP] pobierz, plaintext 
  1. if ( !filter_var($email, FILTER_VALIDATE_EMAIL) )
  2. {
  3.     die('email jest niepoprawny!');
  4. }
[PHP] pobierz, plaintext

- do nicku nie używaj żadnych funkcji typu addslashes itd. tylko zwykłe wyrazenia regularne, czyli preg_match i sprawdzasz sobie czy np. nick sklada się tylko z liter i cyfr, jeśli tak to nick jest ok.
- hasła koduj jakimiś funkcjami i w ogole ich wtedy nie filtrujesz bo nie ma takiej potrzeby, dobrze jest jeśli w haśle sa jakieś fikuśne znaki, typu apostrofy i inne znaki specjalne. Hash i tak zamieni to na jakiś ciąg znaków złożonych tylko z liter i cyfr.
!*!
17.03.2013, 13:57:32
1. po co łączysz się z bazą gdy nie przechodzisz walidacji?
2. taki kod
[PHP] pobierz, plaintext 
  1. $nick = @mysql_real_escape_string(stripslashes($_POST["nick"]));
[PHP] pobierz, plaintext

Jest zły. Nie używaj @ i mysql_real_escape_string powinno być po walidacji
3. zastanów się co robisz. Teraz, gdy hasło jest inne niż hasło dodajesz do bazy dane.
4. formatuj kod poprawnie, za chwilę będziesz pisał ciurkiem.

Cytat(Michael2318)
- do filtracji emaila używamy takiego warunku:

To ciut za mało wink.gif ale też nie zaszkodzi.
Kerth
17.03.2013, 13:58:31
A czy jak zaszyfruję hasła to jest możliwość zrobienia "Przypomnienia hasła"? Bo nie chcę, żeby zwracało hasło typu: vgvvCXVBxcwer233FSDvxcxcvxRfsd.

Z e-mailem jet na razie dobrze, tylko nie wiem czemu gdy wpisuję wszystkie dane poprawnie pojawia się komunikat o tym, że hasła nie pasują.
!*!
17.03.2013, 14:00:12
Przypomnienie hasła, to wygenerowanie nowego.
Cytat
tylko nie wiem czemu gdy wpisuję wszystkie dane poprawnie pojawia się komunikat o tym, że hasła nie pasują.

Czytaj ze zrozumieniem.
Kerth
17.03.2013, 14:10:34
Cytat(!*! @ 17.03.2013, 13:57:32 ) *
1. po co łączysz się z bazą gdy nie przechodzisz walidacji?
2. taki kod
[PHP] pobierz, plaintext 
  1. $nick = @mysql_real_escape_string(stripslashes($_POST["nick"]));
[PHP] pobierz, plaintext

Jest zły. Nie używaj @ i mysql_real_escape_string powinno być po walidacji
3. zastanów się co robisz. Teraz, gdy hasło jest inne niż hasło dodajesz do bazy dane.
4. formatuj kod poprawnie, za chwilę będziesz pisał ciurkiem.


To ciut za mało ;) ale też nie zaszkodzi.


1. A kiedy mam się łączyć? Wydawało mi się, że najlepiej podłączyć include z osobnym plikiem(i tak zrobiłem)
2. Czemu zły? Jeśli usunę @ to pojawią się błędy:

[PHP] pobierz, plaintext 
  1. Notice: Undefined index: nick in /virtual/dawid4157.cba.pl/gra.dawid4157.cba.pl/register.php on line 29
  2.  
  3. Notice: Undefined index: email in /virtual/dawid4157.cba.pl/gra.dawid4157.cba.pl/register.php on line 30
  4.  
  5. Notice: Undefined index: haslo in /virtual/dawid4157.cba.pl/gra.dawid4157.cba.pl/register.php on line 31
  6.  
  7. Notice: Undefined index: haslo2 in /virtual/dawid4157.cba.pl/gra.dawid4157.cba.pl/register.php on line 32
[PHP] pobierz, plaintext


3. Warunek jest taki tylko, że dodałem to w else.

[PHP] pobierz, plaintext 
  1. if($haslo != $haslo2)
  2. 	echo "<br><b><font color= red>Hasła nie są takie same!</font></b><br>";
[PHP] pobierz, plaintext


To nie wiem co jest nie tak.

PS. Szczerze mówiąc nie bardzo rozumiem o czym mówisz. Uczę się dopiero PHP więc, wiesz...
!*!
17.03.2013, 14:15:29
Cytat
1. A kiedy mam się łączyć? Wydawało mi się, że najlepiej podłączyć include z osobnym plikiem(i tak zrobiłem)

Łącz się z bazą wtedy kiedy tego potrzebujesz.

Cytat
2. Czemu zły? Jeśli usunę @ to pojawią się błędy:

Błędy się naprawia a nie wycisza.

Cytat
3. Warunek jest taki tylko, że dodałem to w else.
PS. Szczerze mówiąc nie bardzo rozumiem o czym mówisz. Uczę się dopiero PHP więc, wiesz...

Zacznij od kursu w takim razie http://phpkurs.pl/ tam przejdź do działu "Operatory porównania". I przeczytaj jeszcze raz punkt 3 jaki podałem.
Kerth
17.03.2013, 17:38:31
A co jest źle w tej linijce?

[PHP] pobierz, plaintext 
  1. $nick = mysql_real_escape_string(stripslashes($_POST["nick"]));
[PHP] pobierz, plaintext
Michael2318
17.03.2013, 17:47:41
Jest totalnie bez sensu i tyle. Zostaw mysql_real_escape_string, strislashes wywal.

EDIT:

To jest nick, więc zrób tak jak pisałem - wyrażenia regularne, chyba że chcesz mieć ludzi z nickami, typu: moj\'nick

Kerth
17.03.2013, 18:27:21
No wyrzuciłem jak mówiłeś:

[PHP] pobierz, plaintext 
  1. $nick = mysql_real_escape_string($_POST["nick"]);
  2. $email = mysql_real_escape_string($_POST["email"]);
  3. $haslo = mysql_real_escape_string($_POST["haslo"]);
  4. $haslo2 = mysql_real_escape_string($_POST["haslo2"]);
[PHP] pobierz, plaintext


I mam taki same błędy.


[PHP] pobierz, plaintext 
  1. Notice: Undefined index: nick in /virtual/dawid4157.cba.pl/gra.dawid4157.cba.pl/register.php on line 29
  2.  
  3. Notice: Undefined index: email in /virtual/dawid4157.cba.pl/gra.dawid4157.cba.pl/register.php on line 30
  4.  
  5. Notice: Undefined index: haslo in /virtual/dawid4157.cba.pl/gra.dawid4157.cba.pl/register.php on line 31
  6.  
  7. Notice: Undefined index: haslo2 in /virtual/dawid4157.cba.pl/gra.dawid4157.cba.pl/register.php on line 32
[PHP] pobierz, plaintext
kubek3898
17.03.2013, 18:55:22
Przed wykonaniem zapytania itd. sprawdzaj czy zmienne są ustawione

isset()
Kerth
17.03.2013, 19:12:58
Nie wiem czy dobrze sformułowałem:

[PHP] pobierz, plaintext 
  1. if(isset($_POST['nick'])&& isset($_POST['email'])&& isset($_POST['haslo'])&& isset($_POST['haslo2']))
[PHP] pobierz, plaintext


Teoretycznie brak błędu ale wolę się upewnić czy nie ma jakieś niepotrzebnych rzeczy.
kubek3898
17.03.2013, 19:26:13
Wystarczy sprawdzić czy pole submit zostało ustawione smile.gif. Czyli:

[PHP] pobierz, plaintext 
  1. if(isset($_POST['submit']))
[PHP] pobierz, plaintext


Teraz po kolei:

1. Możesz usunąć potencjalne znaki spacji - trim()
2. Sprawdzasz czy dane są puste - empty() - jeśli tak, wypisujesz informację o tym
3. Sprawdzasz czy dane są odpowiedniej długości - jeśli nie, wypisujesz informację o tym
4. Filtrujesz dane
5. Wykonujesz zapytanie


Wybacz za moje skromne wypowiedzi ale piszę z telefonu.
Kerth
19.03.2013, 16:41:34
Chyba udało mi się stworzyć rejestrację, bo rejestruje się poprawnie.

Teraz tworzę plik login.php ale znowu coś nie tak. Liczę, że mi coś doradzicie.

Otóż tak. Mam taki kod:

[PHP] pobierz, plaintext 
  1. <html> 
  2. <head> 
  3. <title>Logowanie</title> 
  4. </head> 
  5. <body> 
  6.       <b>Logowanie:</b>
  7.       <form action="login.php" method="post"> 
  8.            Wpisz Nick:<br/> 
  9.       <input type="text" name="nick" /><br/> 
  10.            Wpisz hasło:<br /> 
  11.       <input type="password" name="haslo" /><br/> 
  12.       <input type="submit" value="Zaloguj"/> <br/> 
  13.       <b><font color = blue>Jeśli nie masz jeszcze konta, <a href='register.php'>Kliknij Tu!</a></font></b>
  14.      </form>
  15.  
  16. </body> 
  17. </html>
  18.  
  19. <?php
  20. include('inc/sql.php');
  21. include('zmienne.php');
  22.  
  23.  
  24. session_start();
  25. session_register("zalogowany");
  26.  
  27. if(empty($_SESSION["zalogowany"]))$_SESSION["zalogowany"]=0;
  28.  
  29. if($_SESSION["zalogowany"]!=1)
  30. {
  31. 	if(!empty($_POST["nick"]) && !empty($_POST["haslo"]))
  32.         {
  33. 		if(mysql_num_rows(mysql_query("SELECT * FROM `GRACZE` WHERE `nick` = '".htmlspecialchars($_POST["nick"])."' AND `haslo` = '".htmlspecialchars($_POST["haslo"])."'")))
  34.                 {
  35. 		     echo "<b><font color = green>Zalogowano poprawnie. <a href='index.php'>Przejdź na stronę główną</a></font></b>";
  36. 		     $_SESSION["zalogowany"]=1;
  37.                      exit;
  38. 		}
  39.                 if($nick !== $nick)
  40.                 {
  41.                      echo "<br><b><font color = red>Wpisany nick jest błędny!</font></b><br>";
  42.                      exit;
  43.                 }
  44.                 if($haslo !== $haslo2)
  45.                 {
  46.                      echo "<br><b><font color = red>Wpisane hasło jest błędne!</font></b><br>";
  47.                      exit;
  48.                 }
  49.                 $braknicku = mysql_query("SELECT * FROM `GRACZE` WHERE `nick` = '$nick'");
  50.                 if (mysql_num_rows($braknicku) == 0) 
  51.                 {
  52.                 echo "<br><b><font color = red>Podany nick nie istnieje!</font></b></br>";
  53.                 }
  54. 	}
  55. }
  56.  
  57.  
  58. ?>
  59. <?php mysql_close(); ?>
[PHP] pobierz, plaintext


Kod include zmienne.php

[PHP] pobierz, plaintext 
  1. <?php
  2. $nick = isset($_POST['nick']) ? htmlentities($_POST['nick']) : "";
  3. $email = isset($_POST['email']) ? htmlentities($_POST['email']) : "";
  4. $haslo = isset($_POST['haslo']) ? htmlentities($_POST['haslo']) : "";
  5. $haslo2 = isset($_POST['haslo2']) ? htmlentities($_POST['haslo2']) : "";
  6. ?>
[PHP] pobierz, plaintext



Nie wiem co jest źle, czy kod jest dobry? Oraz jak zrobić logowanie i wylogowywanie?
Kod tworzyłem na podstawie tego: http://webmade.org/porady/sesje-php-system-logowania.php
!*!
19.03.2013, 17:17:25
Nie ogarniam Twojego sposobu myślenia...

[PHP] pobierz, plaintext 
  1. if(empty($_SESSION["zalogowany"]))$_SESSION["zalogowany"]=0;
  2. if($_SESSION["zalogowany"]!=1)
[PHP] pobierz, plaintext


Czyli pusta sesja "zalogowany" jest równa 0, ale gdy sesja "zalogowany" jest inna niż 1 loguje się?
Napisz jeszcze gdzie w tym kodzie wyżej, zastosowałeś się do wskazówek, które padły wcześniej?

I skoro masz zmienne zdeklarowane na poczatku (htmlentities?) to czemu dalej odwołujesz się do POST?
I na co Ci to drugie zapytanie "brak nicku" ?
Kerth
19.03.2013, 18:08:51
Racja. Myślę, że powinno być tak:
[PHP] pobierz, plaintext 
  1. if($_SESSION["zalogowany"]=1)
[PHP] pobierz, plaintext


I z takim kodem loguje.

Odnośnie POST to jak mam się odwoływać? GET? Tylko jak?

PS. To drugie z nickiem to moja pomyłka. Chciałem robić coś jeszcze, skopiowałem i zapomniałem usunąć. Oraz jak to wylogowywanie zrobić?
!*!
19.03.2013, 18:10:41
Nie przeczytałeś linku o operatorach. Logujesz się, bo przypisałeś zmienną do sesji, zamiast ją sprawdzić.
Nie chodziło o sposób, POST jest dobry, ale po co przypisujesz go do zmiennych, skoro z tego nie korzystasz.
Kerth
19.03.2013, 18:22:38
To co to jest nie potrzebne?

[PHP] pobierz, plaintext 
  1. <?php
  2. $nick = isset($_POST['nick']) ? htmlentities($_POST['nick']) : "";
  3. $email = isset($_POST['email']) ? htmlentities($_POST['email']) : "";
  4. $haslo = isset($_POST['haslo']) ? htmlentities($_POST['haslo']) : "";
  5. $haslo2 = isset($_POST['haslo2']) ? htmlentities($_POST['haslo2']) : "";
  6. ?>
[PHP] pobierz, plaintext


Cytat
Nie przeczytałeś linku o operatorach. Logujesz się, bo przypisałeś zmienną do sesji, zamiast ją sprawdzić.


To jaki operator mam zastosować? Bo ja już nic nie rozumiem. Oczywiście czytałem link o operatorach ale nie wiem jak to zastosować.

PS. Przepraszam za te Ciągłe pytania ale po prostu staram się PHP zrozumieć.
To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.
Invision Power Board © 2001-2024 Invision Power Services, Inc.