Forum PHP.pl

Hej piszę sobie klasę users, która jest odpowiedzialna za logowanie i dodanie użytkownika, później dojdzie kasowanie i edycja konta. Moje pytanie brzmi czy sposób w który piszę jest dobry:

[PHP] pobierz, plaintext 
<?
class user extends sqli
  {
 
    public function userLogin()
      {    
 
       $this->log=trim($_POST['log']);
       $this->pass=$pass=trim($_POST['pass']);
       $result=$this->mysqli->query("select log,pass from users where log='$this->log' and pass='$this->pass'");
        if(mysqli_num_rows($result)==0)
            {
                echo "brak takiego usera";
             }
             else
             {
                $_SESSION['zalogowany']="$this->log";
                header("Location: test.php");
              }
 
 
            }
            
            
   public function addUser()
            {
 
             $this->log=trim($_POST['log']);
             $this->pass=$pass=trim($_POST['pass']);
             $result=$this->mysqli->query("insert into users values('$this->log','$this->pass')");
             if($result===true)
              {
                  echo "Użytkownik został dodany poprawnie.";
              }
               else
               {
                  echo "Wystąpił błąd podczas dodawania nowego użytkownika!";
              }
 
            }
 
 
 
        }
$user=new user('localhost','root','','katalog');
?>
[PHP] pobierz, plaintext 

I dodawanie użytkownika i samo logowanie działa poprawnie, a mi chodzi o Waszą ocenę czy to jest dobrze napisane. Oczywiście zdaję sobie sprawę, że można to napisać zawsze lepiej.

Ten post edytował ExPlOiT 2.05.2009, 22:57:27

Witam!

Nie jest różowo Na początek taki detal, nazwy klas dobrze zaczynać wielką literą. Twoja klasa dziedziczy po sqli (?). Podejrzewam, że to coś co odpowiada za połączenie z bazą itd. Czyli każda instancja tej klasy tworzy nowe połączenie? Klasę do obsługi bazy danych zrób sobie jako Singleton (najprościej). Nie dziedzicz po niej. Metoda loginUser() powinna przyjmować parametry, a nie czytać z posta. Może od początku. Obiekt user reprezentuje użytkownika, mianowicie jego id, imię, nazwisko, login, email i hasło (dla przykładu). Fajnie zrobić statyczną metodę login( $login, $password ), która po prostu zwróci obiekt User wypełniny kompletem danych z bazy lub false w przypadku błędnych danych logowania.

Dodawanie Usera... Moim zdaniem (podkreślam, że to moje zdanie) powinno to wyglądać tak:

[PHP] pobierz, plaintext 
<?php
$user = new User;
$user->login= "adam";
$user->password = "dupa8";
$user->save();
?>
[PHP] pobierz, plaintext 

Oczywiście można dorobić jakiś obiekt bazowy, który ma metody typu setData( $post ) - czyli ustawia właściwości obiektu na podstawie danych z tablicy $_POST itd. Jednak nie o to chodzi. Obiekt to nie pojemnik na funkcje. Niech faktycznie będzie to obiekt User.

Statyczna metoda login(); powinna ustawiac też flagę w obiekcie czy dane są z bazy czy jest to nowy obiekt User. Wtedy metoda save() wie czy robić insert czy update. Przdałyby się jakieś walidatory jako prywatne metody (w uproszczeniu). Przykład: tworzę nowego Usera, podaję login (ale nie podaję hasła). Robię $user->save(); metoda powinna wyrzucić stosowny wyjątek.

Tak bym to widział.

Pozdrawiam!

Dodałem wzorzec projektowy singleton i coś mi nie działa, mama taki kod:

[PHP] pobierz, plaintext 
<?
 
 
        class Sqli
            {
                
                private static $instance = false;
                
                //Konstruktor
            public function __construct()
            {
                self::$instance=new mysqli('localhost','root','','katalog');
            }
            
            
            static function instance()
            {
            if( !self::$instance )
            {
            self::$instance = new self;
            }                              
        
            return self::$instance;
            }
            
            
            public function getRow($query)
            {
                return Sqli::instance()->query($query)->fetch_object();
                
            }
                    
            
            
            
            // Pobranie danych z bazy
            public function getRows($query)
            {
                $dane=array();
                $result=Sqli::instance()->query($this->query=$query);
                while($wynik=$result->fetch_object()){
                    $data[]=$wynik;
                    
                }
                return $data;
                
                /* $rows=$sqli->getRows("select * from users");
                foreach($rows as $row)
                {
                echo $row->log.' | '.$row->pass;
                }
                */
            }
            
            
            
            //Dodawania wartości do bazy danych
            public function insert($query)
            {
                $result=Sqli::instance()->query($query);
                if($result===true)
                {
                    echo "Dane zostały dodane do tabeli!";
                }
                else {
                    echo "Wystąpił błąd podczas dodawania danych do tabeli!";
                }
            }
            
            
            
            // Aktualizacja danych w bazie
            public function update($query)
            {
                $result=Sqli::instance()->query($query);
                if($result===true)
                {
                    echo "Dane zostały zaaktualizowane!";
                }
                else {
                    echo "Wystąpił błąd podczas aktualizowania danych!";
                }
            }
            
            
            
            
            // Kasowania z bazy danych
            public function delete($query)
            {
                $result=Sqli::instance()->query($query);
                if($result===true)
                {
                    echo "Dane zostały usunięte!";
                }
                else {
                    echo "Wystąpił błąd podczas usuwania danych!";
                }
            }
            
            
            // Koniec klasy
        }
        
 
$sqli = Sqli::instance();
 
 
 
?>
[PHP] pobierz, plaintext 

Robię np:

[PHP] pobierz, plaintext 
<?php
require 'mysqli.class.php';
$sqli->delete("delete from users where log='admin'");
?>
[PHP] pobierz, plaintext 

I dostaje błąd:

[PHP] pobierz, plaintext 
<?php
[b]Fatal error[/b]:  Call to undefined method Sqli::query() in [b]C:wampwwwmysqlimysqli.class.php[/b] on line [b]91[/b]
?>
[PHP] pobierz, plaintext 

Co najdziwniejsze dla mnie w klasie user, działa wszystko bez problemu jakiegokolwiek, a w tej klasie do obsługi bazy danych nie działa

Ale dlaczego

[PHP] pobierz, plaintext 
<?php
$result=Sqli::instance()->query($query);
?>
[PHP] pobierz, plaintext 

Przecież wewnątrz metody masz już obiekt, więc $this.

Poza tym nie korzystaj z (anty)wzorca Singleton tutaj - istnieje prawdopodobieństwo, że możesz w przyszłości pokusić się o to by skrypt działał na dwóch bazach danych - co wtedy?

Pozatym:
1) Klasa użytkownik jest rozszerzeniem klasy sqli - widzisz logikę w tym? Ja nie. Do klasy User powinienenś w jakiś sposób przekazać już istniejący obiekt klasy Sqli (swoją drogą skąd taka nazwa?)
2) Klasa User jak sama nazwa mówi powinna w pierwszej kolejności reprezentować użytkownika - tutaj internaute. I bez znaczenia jest to czy jest on zalogowany czy nie.
3) Pozatym jak już ayeo wskazał: dane powinny być przekazywane w parametrach funkcji, a nie odczytywane z globalnych danych w jej wnętrzu

Taka była rada ayeo, żeby z niego korzystać.

To już zmieniłem.

Tak jakoś, to tylko w celach treningowych.

W jaki sposób? Gdzieś czytałem coś o parent::

Klasa user ma być odpowiedzialna, za logowanie, dodanie, edytowanie i usunięcie użytkownika. Czyli rozumiem że nie mam korzystać z singletonu w tym wypadku.

Ten post edytował ExPlOiT 3.05.2009, 09:20:46

IMO była ona tutaj błędna.parent:: oznacza odwołanie się do rodzica, ale jak już napisałem klasa User nie powinna dziedziczyć po sqli bo ona nie jest rozwinięciem sqli, ona tylko obiektu takiej klasy używa.

Ok dzięki za rady, coś wykombinuje, bo bez wzorca projektowego singleton wszystko ładnie pięknie mi działa.
To powiedz mi jeszcze jak wykorzystać klasę Sqli, w klasie Users nie dziedzicząc jej.
Wiem, że zadaje głupie pytania ale dopiero się uczę OOP.

Ten post edytował ExPlOiT 3.05.2009, 09:41:24

Na przykład:

[PHP] pobierz, plaintext 
<?php
 
class sqli{
  //....
}
 
class User{
   private $db;
 
   public function __construct(sqli $db){
      $this->db = $db;
   }
 
   //...
 
   public function someFunction(){
      $this->db->update('...');
   }
}
 
$db = new sqli(...);
$user = new User($db);
?>
[PHP] pobierz, plaintext 

Sorki że wracam do tego tematu, ale wyczarowałem pewny kod, a dokładnie wykorzystanie klasy sqli w klasie user i chciałem się dowiedzieć czy dobrze to zrobiłem. Oto kod:

[PHP] pobierz, plaintext 
<?php
 
 
        class sqli
            {
                
                
                
                //Konstruktor
            public function __construct($host,$login,$haslo,$baza)
            {
                $this->mysqli=new mysqli($host,$login,$haslo,$baza);
            }
            
            
            
            
            
            public function getRow($query)
            {
                return $this->mysqli->query($this->query=$query)->fetch_object();
                
            }
                    
            
            
            
            // Pobranie danych z bazy
            public function getRows($query)
            {
                $dane=array();
                $result=$this->mysqli->query($this->query=$query);
                while($wynik=$result->fetch_object()){
                    $data[]=$wynik;
                    
                }
                return $data;
                
                /* $rows=$sqli->getRows("select * from users");
                foreach($rows as $row)
                {
                echo $row->log.' | '.$row->pass;
                }
                */
            }
            
            
            
            //Dodawania wartości do bazy danych
            public function insert($query)
            {
                $result=$this->mysqli->query($this->query=$query);
                if($result===true)
                {
                    echo "Dane zostały dodane do tabeli!";
                }
                else {
                    echo "Wystąpił błąd podczas dodawania danych do tabeli!";
                }
            }
            
            
            
            // Aktualizacja danych w bazie
            public function update($query)
            {
                $result=$this->mysqli->query($this->query=$query);
                if($result===true)
                {
                    echo "Dane zostały zaaktualizowane!";
                }
                else {
                    echo "Wystąpił błąd podczas aktualizowania danych!";
                }
            }
            
            
            
            
            // Kasowania z bazy danych
            public function delete($query)
            {
                $result=$this->mysqli->query($this->query=$query);
                if($result===true)
                {
                    echo "Dane zostały usunięte!";
                }
                else {
                    echo "Wystąpił błąd podczas usuwania danych!";
                }
            }
            
            
            // Koniec klasy
        }
        
        
        class User{
              private $db;
 
              public function __construct(sqli $db){
             $this->db = $db;
              }
 
 
              public function addUser($login,$haslo){
             $this->db->insert("insert into user values('$login','$haslo')");
              }
        }
?>
[PHP] pobierz, plaintext 

A tak go używam:

[PHP] pobierz, plaintext 
<?php
$db = new sqli('localhost','root','','spam');
 $user = new User($db);
 $user->addUser('adminek','adminek2');
?>
[PHP] pobierz, plaintext 

Oczywiście kod działa jak należy, tylko chodzi mi o to czy jest to poprawnie napisane. Krótko mówiąc rozumuję iż, w taki sposób jak zrobiłem powyżej, uzyskuje dostęp do klasy sqli oraz jej metod, dobrze myślę czy może się mylę? Wnioskuje to po tym że uzyskałem dostęp do metody update, która jest w klasie sqli.

Ten post edytował ExPlOiT 6.06.2009, 00:11:09

A klasa User ma reprezentować jakiegoś konkretnego użytkownika z bazy danych, czy ma robić za swego rodzaju Managera pozwalającego dodawać, pobierać użytkowników?

Ma robić za managera, czyli dodawać usera, kasować, pobierać użytkowników, logować itp

Ten post edytował ExPlOiT 6.06.2009, 11:31:53

[PHP] pobierz, plaintext 
<?php
public function __construct($host,$login,$haslo,$baza) {
 
      $this->mysqli=new mysqli($host,$login,$haslo,$baza);
            
   }
?>
[PHP] pobierz, plaintext 

To nie jest najlepsze rozwiazanie a czym nie jest tez bledne lepiej przekazuj obiekt do construct'a jak to robisz w klase User

I do tego dla klasy sqli ja bym zrobil wspolne APi(mowa o interfejsach) zebys potem mogl latwo rozszerzac twoja klase user o nowe database'y nie zmieniajac starego kodu tylko tworzysz nowy sterownik ze takimi samymi nazwami funkcji ktore analogicznie beda mialy robic to co robia tez z innej klasy i wtedy tylko przekazujesz odpowiedni obiekt bazy ktorej chcesz uzywac do klasy User.

P.S ewentualnie mozesz uzyc inne nazwe funckji jesli ma to byc np baze dla plikow *.txt robiac adapter na stare APi i bedzie good to tylko takie moje zdanie.

Ten post edytował marcio 6.06.2009, 15:29:15

Co do bazy danych... chyba lepiej będzie skorzystać z PDO. A obiekt ów klasy, przekazać, a nie tworzyć we wnętrzu.

O tym samym mowie jednak z tym PDo to nie tak do konca na poczatek sam niech sie pobawi w pisanie sterownikow przynajmniej czlowiek sie czegos nauczy.

Niech użyje PDO... nauczy się korzystać z całkiem dobrych, gotowych narzędzi.

Ćwiczyć może przecież robiąc to co teraz robi.

Ta, i nieraz wkurzających (tzn. jak się nie uważa ze składnią i ciężej błędy wyłapać). Ale to swoją drogą. Mi w PDO brakuje czegoś w rodzaju seek po wynikach...

Czyli krótko mówiąc podążam w dobrym kierunku, z tym że mam zamienić MySQLI na PDO

Ten post edytował ExPlOiT 6.06.2009, 21:22:09

To juz zalezy od ciebie ja bym nie zmienial

Ogolnie nie jest zle ale przy tak malych kodach to jest dosyc latwo napisz system news'ow z obsluga xml/rss/html i wspomagajacy bazy mysql,pliki i np oracle oczywiscie zaimplementuj co chcesz nie musisz wszystkiego ale daj taka mozliwosc bez ingerencji w kod ze strony user'a zmieniajac tylko 1-2 linijki to juz bedzie cos sam pisalem takie cos po wczesniejszym napisaniu klasu do autoryzacji i powiem latwo nie bylo ale duzo sie nauczylem

Na wielu forach, doradzają mi abym korzystał z PDO, zamiast MySQLI

Zgadza się, gorzej jak się kod bardziej rozbuduje.

A właśnie co do obsługi użytkowników, to robić osobne klasy na dodawanie usera, kasowanie, pobieranie, logowanie, rejestracja etc... i na zarządzanie profilem, czyli aktualizacja profilu itp
Czy lepiej zrobić jedną klasę, takiego managera który będzie robił wszystko z użytkownikiem np dodawane, aktualizacja profilu itp

Mi się wydaje, że lepszym rozwiązaniem będzie stworzenie jednej klasy do obsługi użytkowników, a w niej wszystko co z użytkownikami związane, ale chciałbym dowiedzieć się, jakie jest wasze zdanie na ten temat.

Ten post edytował ExPlOiT 6.06.2009, 22:12:15

Ja nie powiedzialem ze masz zmienic na rzecz mysqli jednak albo uzywasz FW i ich gotowe sterowniki dla baz albo korzystasz z wlasnych nie warto mieszac wedlug mnie.

Ogolnie Rejestracja to calkiem inna klasa ewentualnie do rejestracji/logowania mozesz dodac edycje/usuwanie konta etc... jednak lepiej stwotrzyc 3 klasy gdzie kazda ma swoje zadanie czyli:
-Register
-Auth
-User_manager

I to wsio

Pozdro