Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

> [MySQL][PHP]Logowanie w php
pawelit
post
Post #1





Grupa: Zarejestrowani
Postów: 12
Pomógł: 0
Dołączył: 14.09.2011

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

[PHP] pobierz, plaintext 
  1. <?php
  2.  
  3. class Logowanie {
  4.     private $user;
  5.     private $password;
  6.     private $errors;
  7.  
  8.     public function __construct() {
  9.         $this->user =  $this->filter($_POST['user']);
  10.         $this->password =  md5($this->filter($_POST['pass']));
  11.         $this->errors = array();
  12.     }
  13.  
  14.     public function filter($var){
  15.         return preg_replace('/[^a-zA-Z0-9@.]/','',$var);
  16.     }
  17.  
  18.  
  19.     public function validate(){
  20.  
  21.         if(empty($this->user))
  22.             $this->errors[] = 'Podaj poprawna nazwe uzytkownika';
  23.         if(empty($this->password))
  24.             $this->errors[] = 'Podaj poprawne haslo';
  25.  
  26.          return count($this->errors)? 0 : 1;
  27.  
  28.     }
  29.  
  30.     public function process(){
  31.         if($this->validate())
  32.  
  33.        if (login() == $this->password){
  34.        $_SESSION['permissions'] = 1;
  35.        echo "Zalogowano";
  36.    }
  37.  
  38.    else { echo "kura";} 
  39.  
  40.      return count($this->errors)? 0 : 1;
  41.     }
  42.  
  43.  
  44.       public function show_errors()
  45.   {
  46.     echo "<h3>Errors</h3>";
  47.  
  48.     foreach($this->errors as $key=>$value)
  49.       echo $value."<br>";
  50.   }
  51.  
  52.     public function login(){
  53.         try
  54.  
  55.     {
  56.           $pdo = new PDO('mysql:host=localhost;dbname=kom', 'root', '');
  57.           $pdo -> setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
  58.  
  59.            $stmt = $pdo -> query("SELECT password FROM users WHERE user = '{$this->user}'");
  60.            $stmt -> closeCursor();
  61.    }
  62.  
  63.    catch(PDOException $e)
  64.  
  65.    {
  66.       echo 'Połączenie nie mogło zostać utworzone: ' . $e->getMessage();
  67.    }
  68.     return $stmt;
  69.  }
  70. ?>
[PHP] pobierz, plaintext 



[PHP] pobierz, plaintext 
  1. <html>
  2.     <head>
  3.         <title>
  4.             logowanie
  5.         </title>
  6.     </head>
  7.     <body>
  8.  
  9.  
  10.  
  11. <?php  
  12.         if (isset ($_POST['login']))
  13.         {
  14.                 include_once '/logowanie.php';
  15.                 $a = new Logowanie();
  16.             if($a->process())
  17.     echo "Login!";
  18.   else
  19.     $a->show_errors();
  20.  
  21.         }     
  22.  
  23. ?>
  24.  
  25.  
  26.  
  27.         <form method="POST" action="<?php echo $_SERVER['PHP_SELF'];?>">
  28.  <table>
  29.   <tr><td>User:</td><td><input type="text" name="user"/></td></tr>
  30.   <tr><td>Password:</td><td><input type="password" name="pass"/></td></tr>
  31.  </table>
  32.  
  33.  <input type="submit" name="login" value="Log in"/>
  34. </form>
  35.  
  36.  
  37.     </body>
  38. </html>
[PHP] pobierz, plaintext



Niestety nie loguje. Czy taka konstrukcja jest prawidłowa: 
[PHP] pobierz, plaintext 
  1.  if (login() == $this->password){
  2.        $_SESSION['permissions'] = 1;
  3.        echo "Zalogowano";
  4.    }
[PHP] pobierz, plaintext
?

Ten post edytował pawelit 14.09.2011, 12:58:04
Go to the top of the page
+Quote Post
 
 Start new topic
Odpowiedzi
!*!
post
Post #2





Grupa: Zarejestrowani
Postów: 4 298
Pomógł: 447
Dołączył: 16.11.2006

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

mortus - nie, źle się wyżej wyraziłem.

[PHP] pobierz, plaintext 
  1. $stmt = $pdo -> query("SELECT password FROM users WHERE user = '{$this->user}'");
[PHP] pobierz, plaintext


Do czego Ci te nawiasy ? I nie jestem przekonany co do poprawności samego zapytania. Użyj też bindowania do PDO (tak na przyszłość, coby zapobiec SQLi)

Poza tym nie musisz sprawdzać czy $_SESSION['permissions'] jest równe 1 (sesja ta w ogóle istnieje w kodzie?)

Oddziel też walidacje od samego procesu logowania.

Ten post edytował !*! 14.09.2011, 14:10:11
Go to the top of the page
+Quote Post
pawelit
post
Post #3





Grupa: Zarejestrowani
Postów: 12
Pomógł: 0
Dołączył: 14.09.2011

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

Cytat(!*! @ 14.09.2011, 15:07:33 ) *
mortus - nie, źle się wyżej wyraziłem.

[PHP] pobierz, plaintext 
  1. $stmt = $pdo -> query("SELECT password FROM users WHERE user = '{$this->user}'");
[PHP] pobierz, plaintext


Do czego Ci te nawiasy ? I nie jestem przekonany co do poprawności samego zapytania. Użyj też bindowania do PDO (tak na przyszłość, coby zapobiec SQLi)

Poza tym nie musisz sprawdzać czy $_SESSION['permissions'] jest równe 1 (sesja ta w ogóle istnieje w kodzie?)

Oddziel też walidacje od samego procesu logowania.


[PHP] pobierz, plaintext 
  1. public function process(){
  2.         if($this->validate()){
  3.  
  4.        if($this->login() == $this->password) {
  5.        $_SESSION['permissions'] = 1;
  6.  
  7.    }
  8.  
  9.    }
  10.    return count($this->errors)? 0 : 1;
  11.     }
[PHP] pobierz, plaintext
Go to the top of the page
+Quote Post

Posty w temacie
- pawelit   [MySQL][PHP]Logowanie w php   14.09.2011, 12:29:35
- - !*!   W końcu ktoś kto napisał klasę logowania w php5 i ...   14.09.2011, 12:36:01
- - pawelit   Teoretycznie metoda login() wyciąga hasło usera z ...   14.09.2011, 12:56:34
- - mortus   @!*! A gdyby metoda login() była statyczna...   14.09.2011, 13:16:47
- - pawelit   Dzięki za odpowiedzi. Teraz zrobiłem tak, chyba c...   14.09.2011, 13:46:37
- - mortus   Czego nie wyświetla? Pojawiają się jakieś błędy? W...   14.09.2011, 14:00:27
- - pawelit   [PHP] pobierz, plaintext if(isset($_SESSI...   14.09.2011, 14:05:30
- - !*!   mortus - nie, źle się wyżej wyraziłem. [PHP] pobi...   14.09.2011, 14:07:33
|- - pawelit   Cytat(!*! @ 14.09.2011, 15:07...   14.09.2011, 14:11:43
- - mortus   Źle używasz PDO, w związku z czym warunek if(...   14.09.2011, 14:34:46

« Następny starszy · Przedszkole · Następny nowszy »
 

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

 

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

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

RSS Aktualny czas: 10.10.2025 - 21:37
Powered By IP.Board © 2025  IPS, Inc.
All changes by PHP.pl Administrators
Hosting zapewnia NQ.pl hosting, trac, svn