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.

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.

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.

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.

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

[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.

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

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.

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

Krótko mówiąc miałbym 3 klasy w jednym pliku, ponieważ musiałaby być klasa do:
- obsługi bazy danych która jest wykorzystywana do rejestracji itp
- obsługi rejestracji/logowania
- obsługi użytkownika

Bo jak inaczej, można wykorzystać klasę do obsługi bazy danych jak nie w taki sposób, w jaki zrobiłem to parę postów wyżej.

Masz rację, tylko nie do wszystkich projektów potrzeba FW, dlatego dobrze jest mieć własne klasy.

No to wlasnie albo korzystasz z FW w 100% i z gotowych klas albo wszystko piszesz sam robic z kodu takie spaghetti gdzie mieszasz swoj kod z cudzym to raczej wyjdzie @#*$&* i tobie tez nie bedzie smakowalo taka moja skromna uwaga



Jesli chodzi o post nor #9 to cos w ten desen ale troche zabardzo kombinujesz i kod nie jest elastyczny tylko 1 baza .

Wczesniej to robiles z tego sieczke wszystko w jednej klasie lub klasa autoryzacji dziedziczyla po klasie dla mysql ogolnie robisz te same bledy co robilem i ja

Oczywiscie teraz tez pro nie jestem ale mniej wiecej wiem co i jak poczytaj troche kodow to jest najlepsza nauka ^^

No tak, tylko po co korzystać z FW przy projekcie którego kod zajmuje maks 100 linijek?

Nie chodzi mi o to że mieszam kod FW ze swoim, tylko albo korzystam z FW albo ze swojego kodu

Bo będę tej klasy używał tylko na bazie MySQL

Odnośnie PDO to jeżeli jesteś początkującym programistą (jak wywnioskowałem z twoich postów) to moim zdaniem nie warto utrudniać sobie pracy.

Odnośnie klas to proponuje 2 klasy:
- klasa do obsługi tabeli z użytkownikami w MySqlu
- klasa do obsługi pojedynczego użytkownika podczas pracy z widokiem/kontrolerem (->getNick, ->getMail, ->getProfil (link do profilu), ->checkPass($string) itp)

Mówisz że pisząc 100 liniowy projekt nie opłaca sie używać FW? Idąc tym tropem to czy opłaca się używać klas?

Moim zdaniem pisząc 100 liniowy projekt szkoda czasu na zastanawianie się nad tym problemem

A co jest złego w korzystaniu z cudzych bibliotek? Jeżeli jest to w ramach treningu to może, ale jeżeli nie to po co ma tracić czas?

Hmmm... a co w PDO tak dokładnie miałoby mu utrudnić pracę?


Możesz spróbować zaprojektować sobie swój model nieco inaczej. Zrób wszystko od drugiej strony - nie "jakich klas potrzebuję" tylko "jakiej funkcjonalności potrzebuję".
Przykładowo masz prostą strukturę danych, gdzie każdy użytkownik jest przypisany do jakiejś grupy (relacja 1-n)

[PHP] pobierz, plaintext 
<?php
 
//zwraca nam jeden obiekt, klasy Group
$group = GroupPeer::retrieveByName('user');
 
//tworzymy sobie nowy obiekt, reprezentujący użytkownika
$user = new User();
$user->setUsername('blah blah')
     ->setFirstName('blag blah')
     ->setLastName('blah blag')
     //tak wiem, że można by przekazać poprostu $group :)
     ->setGroupId($group->getId());
 
$user->save();
?>
[PHP] pobierz, plaintext 

To jest akurat pokazane na przykładzie Propela. Do tego dochodzi jeszcze walidacja danych - ale to już "przy formularzach".

Crozin pomysł jak i samo podejście ciekawe zobaczę co mi z tego wyjdzie. A co do klas to stworzę 3, do obsługi bazy danych, do zarządzania użytkownikami czyli dodawanie, kasowanie, rejestracja i logowanie, a 3 i ostatnia to klasa która będzie obsługiwała profil danego usera czyli aktualizacja profilu, wyświetlenie profilu itp. Wydaje mi się że taki podział jest najlepszym według mnie rozwiązaniem.

Może zrób sobie:
- klasę do obsługi bazy
- klasę do operacji na tabeli "users" - insert, update, select itd. np Model_User
- klasę user reprezentującą obiekt aktualnie zalogowanego/odwiedzającego użytkownika - np. My_User - tutaj metody setAuthenticated(), isAuthenticated(), setGroups(), hasGroups() i co tylko chces oraz dostęp do danych z sesji - do sesji oddzielna klasa ;]
Dwa pierwsze punkty zastępuje wspomniany wcześniej Propel

Samo logowanie, edycja czy wyswietlanie to umiejętne używanie tego co napisałem, budujesz kontroler i w nim działasz na obietkach Model_User i My_User