[skrypt] Klasa logowania i rejstracji, Ocena, dobre rady Opcje

[gcdreak]

Witam!<br>Kiedyś stworzyłem klasę logowania. Zaczynam dopiero przygodę z OOP. Proszę o Waszą opinie i jakieś rady. Moja klasa:
http://wklej.org/id/101767/

Powód edycji: [erix] przeniosłem

[NuLL]

Pytanie jaki zwiazek ma uruchamianie sesji oraz laczenie sie z baza danych z samym logowaniem ? Te funkcjonalnosci zupelnie do tej klasy nie pasuja. Pomysl - co tak naprawde powinna zawierac klasa do logowania - tak naprawde TYLKO to co jest zwiazane z logowanie uzytkownika

[gcdreak]

Tak, masz rację. A pozatym to co jeszcze myślicie o tej klasie? Zależy mi, żeby się od was dowiedzieć istotnych rzeczy dotyczących tworzenia systemu kontoroli autoryzacji. Może ktoś mi dokładnie powie, jak to powinno wyglądać bo już któryś dzień przeglądam www i ciężko jest znaleść coś sensownego i OO.

[NuLL]

http://phpgacl.sourceforge.net/ - polecam zapoznanie sie

[gcdreak]

To cały system, który nie jest mi zbyt potrzebny. Ponadto chcę się nauczyć.

[marcio]

http://wklej.org/id/101896/txt ja kiedys dla cwiczen tak to napisalem jednak teraz bym zmienil 2 rzeczy ale ogolnie klasa jest good jak chcesz to zobacz

Oczywiscie dla rejestracji i bazy danych mam osobne klasy.

[gcdreak]

No ciekawie to wygląda.
Ja w między czasie zacząłem tworzyć coś nowego. Proszę o uwagi i pewnie wielu powie, że powninienem odseparować połączenie z bazą. Niestety nie bardzo wiem jak to zrobić, ponieważ przeważnie mam z implementacją jednej klasy w drugiej. Jeśli by komuś się chciało to byłbym bardzo wdzięczny gdyby mi pokazał jak to napisać. Oto kod: http://wklej.org/id/101976/

[marcio]

Po pierwsze funckja zarejestruj powinna byc w innej klasie to raz.

Dwa to co masz w costruct daj do klasy o nazwie Db czyli taki maly zarys jak to powinno wygladac:

[PHP] pobierz, plaintext 
<?php
class DB { public function connect() {
 
        if(!self::$db) {
 
            self::$db = @new mysqli('localhost', 'user', 'pass', 'base');
 
            if(@ self::$db->connect_errno) {
 
                     throw new Exception('Nie można połączyć się z bazą danych.');
           }
 
        }
 
                else {
           return self::$db;
 
        }
 
    }
 
}
 
 
//tutaj twoja klasa Autoryzcja bez funckji zarejestruj z mala modyfikacja:
 
class Autoryzacja {
 
 private $db; 
  public function __construct(DB $db) { przekazujesz obiekt bazy danych
                                                                              
    $this -> db = $db;
    $this -> db -> connect(); // operujesz na przekazanym obiekcie
    
 }
 
//i tutaj ta twoja funkcja zaloguj/wyloguj
 
 
 public function __construct(DB $db) {
 
    unset($this -> db);
    
 }
 
}
?>
[PHP] pobierz, plaintext 

A rejestracja ma byc w osobnym pliku.

Tak mniej wiecej oczywiscie.

Aha o potem oczywiscie robisz tak:

[PHP] pobierz, plaintext 
<?php
$user = new Autoryzacja(new DB); // i potem reszta normalnie $user -> funkcja();
?>
[PHP] pobierz, plaintext 

Ten post edytował marcio 6.06.2009, 21:48:10

[gcdreak]

Kod

public function __construct(DB $db) {
  
    unset($this ->db);
      
    }

1.A dlaczego to jest tak? Dlaczego są dwa konstruktor?
2.Jeśli tworzę klasę rejstracja to poza samą rejstracją chcę w niej umieścić metody odzyskajHaslo(). Jakie jeszcze powinienem w niej umieścić metody?


Ten post edytował gcdreak 7.06.2009, 07:47:44

[marcio]

Kod

public function __construct(DB $db) {
  
     unset($this ->db);
        
     }

1.A dlaczego to jest tak? Dlaczego są dwa konstruktor?
2.Jeśli tworzę klasę rejstracja to poza samą rejstracją chcę w niej umieścić metody odzyskajHaslo(). Jakie jeszcze powinienem w niej umieścić metody?

Sorki literowka mial byc __destruct() poprostu na szybko pisalem.

Nom ogolnie co robia zawsze klasy rejestracji dodaja user'a i ewentualnie odzsykuja jego stare haslo potem jak to sobie juz zaimplementujesz to juz twoja sprawa czy bedziesz uzywal gettery/settery czu pola publiczne lub arg. funckji by ustawiac dane.

Zrob pokaz a my ci powiemy co i jak.

[Moli]

[PHP] pobierz, plaintext 
<?php
#
 
function __get($var){
 
#
 
            return $this->$var;
 
#
 
        }
 
#
 
        
 
#
 
        function __set($var, $val){
 
#
 
            $this->$var = $val;
 
#
 
        }
?>
[PHP] pobierz, plaintext 

Bez sensu

[Kildyt]

Ale bajerujesz z komentarzami. Nie wystarczy jedno liniowy bez gwiazdek?
Używaj angielskim nazw funkcji, zmiennych itp.
Może spodobają ci się skrócone warunki?
Nie lepiej włączyć sesje w konstruktorze?
Polecam error_reporting(E_ALL).

[gcdreak]

@Moli
co Ci w tym nie pasuje?

Używaj angielskim nazw funkcji, zmiennych itp.

Postanowiłem ten projekt zrealizować używając polskich nazw. Wszyscy używają angielskich i nie mówię, że to jest złe bo ang. to język programowania i dla wszystkich czytelne jest. Należy jednak zauważyć, że tego kodu będę używał tylko ja, a jeśli ktoś inny by chciał poużywać to i tak będzie to Polak.

Może spodobają ci się skrócone warunki?

Możesz mi to dokładniej wytłumaczyć? Dać przykład?

Nie lepiej włączyć sesje w konstruktorze?

Do sesji zrobię osobną klasę i pewnie zamieszcze ją, aby sie od Was dowiedzieć co można by zrobić lepiej. Może nawet dzisiaj ją wrzucę.

Polecam error_reporting(E_ALL).

W php.ini mam:

Kod

error_reporting  =  E_ALL|E_STRICT & ~E_NOTICE

[Kildyt]

Możesz mi to dokładniej wytłumaczyć? Dać przykład?

Zamiast:

[PHP] pobierz, plaintext 
<?php
if(mysqli_errno($db)){
  return false;
}else{
  return true;
}
?>
[PHP] pobierz, plaintext 

masz:

[PHP] pobierz, plaintext 
<?php
if(mysqli_errno($db))
  return false;
else
  return true;
?>
[PHP] pobierz, plaintext 

Osobiście dla mnie taki układ jest bardziej czytelny.


Co do angielskich nazw, to należy stosować się do tej zasady zawsze, nawet wtedy gdy programuje się tylko dla siebie ponieważ dobre nawyki trzeba u siebie wyuczyć od samego początku.

[phpion]

@Kildyt:
Jeśli usunięcie 4 znaków sprawia, że kod jest dla Ciebie bardziej czytelny to spoko Jak już bawimy w skracanie warunków to można tak:

[PHP] pobierz, plaintext 
<?php
return mysqli_errno($db) ? false : true;
?>
[PHP] pobierz, plaintext 

lub wręcz:

[PHP] pobierz, plaintext 
<?php
return (bool)mysqli_errno($db);
?>
[PHP] pobierz, plaintext 

[gcdreak]

@Kildyt:
Można i tak, ale to jest objętne, przeważnie piszę nawiasy.

[PHP] pobierz, plaintext 
<?php
return (bool)mysqli_errno($db);
?>
[PHP] pobierz, plaintext 

Ciekawy sposób.

Zrobiłem klasę obsługi sesji:

Kod

session_start();

class ObslugaSesji
{
    
    private $login;


    public function __construct($login){
        $this->login = $login;
    }
    
    
    public function rozpocznijSesje(){
        if(!isset($_SESSION['zalogowany'])){
            $_SESSION['zalogowany'] = true;
            $_SESSION['login'] = $this->login;
        }else{
            session_regenerate_id();
        }
        return true;
    }

    
    public function zakonczSesje(){
        $zalogowany = $_SESSION['zalogowany'];
        session_destroy();
        unset($_SESSION['zalogowany'], $_SESSION['login']);
        if(!empty($zalogowany))
            return true;
        else
            return false;
    }
}

Teraz trochę o zastosowaniu tego.
1. W klasie Autoryzacja mam metodę wyloguj(). Myślałem, żeby do tej metody dopisać odniesienie do metody zakonczSesje() z klasy ObsługaSesji.
Jak to piwinienem zrobić?
2. W klasie Autoryzacja mam metodę zaloguj(). Myślałem, żeby do tej metody dopisać odniesienie do metody rozpocznijSesje() z klasy ObsługaSesji.
Jak to piwinienem zrobić?

Poza tym to co byście ewentualnie zmienili w tej klasie?

[Kildyt]

@Kildyt:
Jeśli usunięcie 4 znaków sprawia, że kod jest dla Ciebie bardziej czytelny to spoko Jak już bawimy w skracanie warunków to można tak:

[PHP] pobierz, plaintext 
<?php
return mysqli_errno($db) ? false : true;
?>
[PHP] pobierz, plaintext 

lub wręcz:

[PHP] pobierz, plaintext 
<?php
return (bool)mysqli_errno($db);
?>
[PHP] pobierz, plaintext 

No mam tak. Ściśle staram się zachować przejrzystość kodu, tabulację i usunięcie dwóch linijek, w którym są same klamry czynią, że kod wydaje się bardziej przejrzysty.

Drugiego sposobu nie znam, ale i tak uważam, że oba nie powinno się stosować ponieważ są one trochę wyjęte z sufitu.

[Moli]

@gcdreak - Jak pisałem jest to bez sensu. Metody magiczne get i set, powinno się używać jeśli zmienne przetrzymujesz np. w tablicach. Czym będzie się różnić ten kod:

[PHP] pobierz, plaintext 
<?php
 
class klasa
{
    public function __set($name,$value)
    {
        $this->{$name} = $value;
    }
 
    public function __get($name)
    {
        return $this->{$name};
    }
}
 
$class = new klasa;
$class->var = 'test';
echo $class->var;
 
?>
[PHP] pobierz, plaintext 

od tego:

[PHP] pobierz, plaintext 
<?php
 
class klasa
{
    
}
 
$class = new klasa;
$class->var = 'test';
echo $class->var;
 
?>
[PHP] pobierz, plaintext 

Odpowiedź: Oprócz większego kodu - niczym.

Ten post edytował Moli 7.06.2009, 18:52:29

[gcdreak]

To ciekawe co piszesz odnośnie metod magicznych, bo nigdy o tym nie słyszałem i dlatego trochę bez sensu (z tego wynika) ich użyłem.
Możesz podać jakiś przykład?

Proszę jeszcze o odpowiedź na moje pytania, które zamieściłem wyżej - odnośnie metod klasy ObslugaSesji

[Moli]

[PHP] pobierz, plaintext 
<?php
 
class klasa
{
    public $Config=array();
 
    public function __set($name,$value)
    {
        $this->Config[$name] = $value;
    }
        
    public function __get($name)
    {
        return $this->Config[$name];
    }
}
 
$class = new klasa;
$class->var = 'test';
echo $class->var;
var_dump($class->Config);
?>
[PHP] pobierz, plaintext 

Oczywiście tablice to tylko prosty przykład. Zastosować metod magicznych jest wiele