Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

> [PHP]błąd w logowaniu
Silny
post 20.07.2010, 14:46:32
Post #1





Grupa: Zarejestrowani
Postów: 117
Pomógł: 2
Dołączył: 29.06.2010

Ostrzeżenie: (0%)
-----

Mam taki kodzik
[PHP] pobierz, plaintext 
  1. <?
  2. session_start();
  3.  
  4. // Sekcja logowania
  5. $login=$_POST[&#8216;login’];
  6. if($login) {
  7.   $login=$_POST[&#8216;login’];
  8.   $md5_haslo=md5($_POST[&#8216;haslo’]); 
  9.  
  10. $database = 'xxx';
  11. include("db.php");
  12.     mysql_select_db($database);
  13.   $rezultat=mysql_query("select * FROM uzytkownicy WHERE login=’$login’ AND haslo=’$md5_haslo’");
  14.     if(@mysql_num_rows($rezultat)){
  15.    session_register("login")
  16.     header("location:main.php");
  17.   exit;
  18.   }else {
  19.   $message="Nieprawidłowa nazwa użytkownika lub hasło";
  20.   }
  21. }
  22. ?>
[PHP] pobierz, plaintext


i taki formularz

[HTML] pobierz, plaintext 
  1. <head>
  2.  
  3. <title>Dokument</title>
  4. </head>
  5.   <body>
  6. <? echo $message; ?>
  7. <form id="form1" name="form1" method="post" action="<? echo $PHP_SELF; ?>">
  8.   <table>
  9.     <tr>
  10.       <td>Użytkownik: </td>
  11.       <td><input name="login" type="text" id="login" /></td>
  12.     </tr>
  13.     <tr>
  14.       <td>Hasło: </td>
  15.       <td><input name="haslo" type="password" id="haslo" /></td>
  16.     </tr>
  17.   </table>
  18. <input name="login" type="submit" id="login" value="login" />
  19. </form>
  20. </body>
  21. </html>
[HTML] pobierz, plaintext


Ale zamiast logować, to odświeża mi stronę... co może być nie tak questionmark.gif
Go to the top of the page
+Quote Post
 
 Start new topic
Odpowiedzi (1 - 4)
tomekpl
post 20.07.2010, 15:20:51
Post #2





Grupa: Zarejestrowani
Postów: 134
Pomógł: 5
Dołączył: 9.08.2008
Skąd: Szczecin

Ostrzeżenie: (0%)
-----

No a co ma robić?
jak masz:
header("location:main.php");

i pewnie logujesz się na pliku main,php

Zrób sobie funkcje czy_zalogowany i jej użyj


ps:

[PHP] pobierz, plaintext 
  1.  
  2.  session_start();
  3.  
  4.       // Sekcja logowania
  5.  
  6.       if(isset($_POST['login?])) {
  7.  
  8.  
[PHP] pobierz, plaintext


dodaj isset($_POST['login?]) zamiast tego $login

Ten post edytował tomekpl 20.07.2010, 15:23:23
Go to the top of the page
+Quote Post
Silny
post 20.07.2010, 18:22:23
Post #3





Grupa: Zarejestrowani
Postów: 117
Pomógł: 2
Dołączył: 29.06.2010

Ostrzeżenie: (0%)
-----

Zrobiłem tak, i teraz cały czas mam, że nieprawidłowa nazwa uzytkownika -.-

[PHP] pobierz, plaintext 
  1. <?
  2. session_start();
  3.  
  4. // Sekcja logowania
  5. if(isset($_POST['login'])) {
  6.   $login=$_POST[&#8216;login’];
  7.   $md5_haslo=md5($_POST[&#8216;haslo’]); 
  8.   // Koduje hasło funkcją md5().
  9.  
  10. $database = 'xxxx';
  11. include("db.php");
  12.     mysql_select_db($database);
  13.   $rezultat=mysql_query("select * FROM uzytkownicy WHERE login=’$login’ AND haslo=’$md5_haslo’");
  14.     if(@mysql_num_rows($rezultat)){
  15.    session_register("login"); 
  16.   exit;
  17.   }else {
  18.   $message="Nieprawidłowa nazwa użytkownika lub hasło";
  19.   }
  20. } 
  21. ?>
[PHP] pobierz, plaintext


Ten post edytował Silny 20.07.2010, 18:23:14
Go to the top of the page
+Quote Post
blahy
post 20.07.2010, 20:09:44
Post #4





Grupa: Zarejestrowani
Postów: 82
Pomógł: 22
Dołączył: 20.07.2010

Ostrzeżenie: (0%)
-----

witam. na wstepie chcialem zaznaczyc ze dopiero co zarejestrowalem sie na forum i nie jestem super wymiataczem winksmiley.jpg

co do Twojego skryptu. Po takim wycinku ciezko powiedzic co dokladnie ma robic ale poprawilem to co wyslales.

ponizej poprawiony kod (komentarze angielskie z tym co poprawilem), dodatkowo pod kodem wyjasnienia

po pierwsze zalozylem ze formularz i kod sa w tym samym pliku (akcja php_self)
[PHP] pobierz, plaintext 
  1. <?php
  2. session_start();
  3.  
  4. // Sekcja logowania
  5. if(isset($_POST['login'])) { //first check if(isset($_POST['submit']))
  6.   $login=$_POST['login']; //different quotes signs
  7.   $md5_haslo=$_POST['haslo']; 
  8.   // Koduje hasło funkcją md5().
  9.  
  10.   include("mysql_connect.php");
  11.  
  12.   $rezultat=mysql_query("select * FROM uzytkownicy WHERE login='$login' AND haslo='$md5_haslo'"); //wrong quotes resulting in wrong sql syntax
  13.  
  14.   if(mysql_num_rows($rezultat)){ //why mute errors when you want to debug?
  15.     $_SESSION["login"] = $login; //use this instead of session_register
  16.     $message="zalogowany jako $login";
  17.   } else {
  18.     $message="Nieprawidłowa nazwa użytkownika lub hasło";
  19.   }
  20. }
  21. ?>
  22.  
  23. <html> <!--html begin tag -->
  24.   <head>
  25.     <title>Dokument</title>
  26.   </head>
  27.   <body>
  28.   <?php echo $message; ?>
  29.     <form id="form1" name="form1" method="post" action="<?php echo $_SERVER['PHP_SELF'] ?>"> <!--php opening tag & $_SERVER['PHP_SELF']-->
  30.       <table>
  31.         <tr>
  32.           <td>Użytkownik: </td>
  33.           <td><input name="login" type="text" id="login" /></td>
  34.         </tr>
  35.         <tr>
  36.           <td>Hasło: </td>
  37.           <td><input name="haslo" type="password" id="haslo" /></td>
  38.         </tr>
  39.       </table>
  40.       <input name="submit" type="submit" id="submit" value="log in" /> <!-- submit and input for login have the same name: login. in result $_POST['login'] => 'login' -->
  41.     </form>
  42.   </body>
  43. </html>
[PHP] pobierz, plaintext


polaczenie z baza zeby byl komplet do testowania:
[PHP] pobierz, plaintext 
  1. <?php
  2. 	$dbUser='xxx';
  3. 	$dbPassword='xxx';
  4. 	$dbHost='localhost';
  5. 	$dbName='silny';
  6.  
  7. 	$dbc=mysql_connect($dbHost,$dbUser,$dbPassword) or die('unable to connect: '.mysql_error());
  8.  
  9. 	mysql_select_db($dbName) or die('unable to select database: '.mysql_error());
  10. ?>
[PHP] pobierz, plaintext


a teraz komentarze co do skryptu:
1. warto zawsze przed analiza pol sprawdzic czy wogole zostal przeslany formularz za pomoca if(isset($_POST['submit']))
2. w pewnym momencie zaczales uzywac ’ zamiast ' - uzywasz jakiegos edytora? w nim od razu mozna wylapac takie bledy
3. otwierajacy tag php powinien wygladac tak <?php. <? moze dzialac, ale nie wszedzie
4. md5 wywalilem zeby sobie ulatwic sprawe
6. cale laczenie z baza i wybor lepiej wrzucic osobno - chyba ze zalezalo Ci na tym zeby bylo tak jak miales
7. jak nie wiesz co jest nie tak w skrypcie, debugujesz go to nie tlum bledow za pomoca @, szczegolnie jesli nie oblugujesz ich w bardziej wyrafinowany sposob
8. olales otwierajacy znacznik html
9. nie trzymasz sie jednego formatowania kodu - ciezej znalezc bledy
10. $_SERVER['PHP_SELF'] zamiast $PHP_SELF
11. tu byl prawdziwy blad: te same nazwy dla pola login i dla przycisku wyslij - login. przez to zmienna $login miala wartosc 'login'. nic dziwngo ze nie znalazl takiego uzytkownika
12. blad w zapytaniu do bazy danych. zawsze warto w myadminie wkleic, zastapic zmienne poprawnymi danymi i sprawdzic czy zapytanie jest poprawne
13. za pomoca print_r($_SESSION), print_r($_POST), echo $login mozna bardzo latwo wylapac takie bledy

to co uwazasz za wazne wez pod uwage, reszte olej. pozdrawiam smile.gif
Go to the top of the page
+Quote Post
Silny
post 21.07.2010, 09:31:20
Post #5





Grupa: Zarejestrowani
Postów: 117
Pomógł: 2
Dołączył: 29.06.2010

Ostrzeżenie: (0%)
-----

dobra dymać ten kod winksmiley.jpg

Ten post edytował Silny 21.07.2010, 10:22:37
Go to the top of the page
+Quote Post
« Następny starszy · Przedszkole · Następny nowszy »
 

Reply to this topicStart new topic
1 Użytkowników czyta ten temat (1 Gości i 0 Anonimowych użytkowników)
0 Zarejestrowanych:

 

Tryb wyświetlania: Przełącz na: Standardowy · Linearny+ · Przełącz na: Drzewo

Śledź ten temat · Wyślij temat na e-mail · Wydrukuj ten temat · Subskrybuj to forum

RSS Wersja Lo-Fi Aktualny czas: 14.08.2025 - 10:49
Powered By IP.Board © 2025  IPS, Inc.
All changes by PHP.pl Administrators
Hosting zapewnia NQ.pl hosting, trac, svn