Witam

Dopiero uczę się obiektowości i chcę napisać sobie lekki framework MVC, na potrzeby prostych stron z systemem CMS.

Napisałem system rejestracji użytkownika, aktywacji konto i logowania, chodzi o ocenę tego przez doświadczone osoby, czy dobrze rozumiem MVC.

[PHP] pobierz, plaintext 
<?php
namespace Controllers;
 
use Core\View;
use Core\Controller;
use Helpers\Session;
use Helpers\Url;
use Helpers\Token;
use Helpers\ClientIP;
use Helpers\Password;
 
class Users extends Controller
{
    private $_model_users;
 
    public function __construct()
    {
        parent::__construct();
        $this->_model_users = new \Models\Users();
    }
 
    public function logowanie()
    {
        $data['meta_title']       = 'Zaloguj się';
        $data['meta_description'] = 'Logowanie w ...';
        View::render('users/logowanie', $data);
    }    
 
    public function rejestracja()
    {
        if (Session::get('loggedin')) {
            Url::redirect(URL_ZALOGOWANY);   
        }
 
        $data['meta_title']       = 'Zarejestruj się';
        $data['meta_description'] = 'Rejestracja w ...';
        View::render('users/rejestracja', $data);
    }   
 
    public function zalogowany()
    {
        if (!Session::get('loggedin')) {
            Url::redirect(URL_LOGOWANIE);   
        }
 
        $data['meta_title']       = 'Zalogowany';        
        View::render('users/zalogowany', $data);
    }
 
    public function loguj()
    {
        if (Session::get('loggedin')) {
            Url::redirect(URL_ZALOGOWANY);   
        }
 
        $FiltrujZmienna = new \Helpers\FiltrujZmienna();
        $email  = $FiltrujZmienna->postStr('email');
        $haslo  = $FiltrujZmienna->postStr('haslo');
 
        if (empty($email) || empty($haslo)){
            Session::set('blad_logowanie', 'Podaj e-mail i hasło');
            Url::redirect(URL_LOGOWANIE);            
        }            
        elseif (Password::verify($haslo, $this->_model_users->getHaslo($email)) == false) {
            Session::set('blad_logowanie', 'Błędny email lub hasło');
            Url::redirect(URL_LOGOWANIE);
        }
 
        if (!$error) {
            Session::set('loggedin', true);
            Session::set('userID', $this->_model_users->getID($email));
            Url::redirect(URL_ZALOGOWANY);
        }
    }    
 
    public function wyloguj()
    {
        Session::destroy();
        Url::redirect();
    }
 
    public function rejestruj()
    {  
        $FiltrujZmienna = new \Helpers\FiltrujZmienna();
 
        if ($this->_model_users->getID($FiltrujZmienna->postStr('email')) > 0) {
           Session::set('blad_rejestracja','<br /><div style="color:red;">Istnieje już taki e-mail w systemie, podaj inny adres e-mail.</div><br />');
           Url::redirect(URL_REJESTRACJA);
        }       
        else {
            $dane=array(
                'token'     => Token::get(15),
                'imie'      => $FiltrujZmienna->postStr('imie'),
                'nazwisko'  => $FiltrujZmienna->postStr('nazwisko'),
                'login'     => $FiltrujZmienna->postStr('email'),
                'haslo'     => Password::make($FiltrujZmienna->postStr('haslo')),
                'data'      => date('Y-m-d h:m:s'),
                'ip'        => ClientIP::get(),
                'aktywny'   => '0'
            );
            $this->_model_users->insert($dane);
            Url::redirect(URL_ZAREJESTROWANY);
        }    
    }   
 
    public function zarejestrowany()        
    {
        $data['meta_title']       = 'Zarejestrowany';
        $data['meta_description'] = '';
        View::render('users/zarejestrowany', $data);
    }   
 
    public function aktywacja($token)
    {        
        $FiltrujZmienna = new \Helpers\FiltrujZmienna();
        $id = $this->_model_users->checkToken($FiltrujZmienna->Str($token);
 
        if ($id > 0) {
            $this->_model_users->update(array('aktywny'=>1), array('id'=>$id)));
            $data['aktywacja_yes'] = 1;
            $data['aktywacja_not'] = 0;
        } 
        else {
            $data['aktywacja_yes'] = 0;
            $data['aktywacja_not'] = 1;
        }
 
        View::render('users/aktywacja', $data); 
    }   
 
}
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
 
namespace Models;
 
class Users extends \Core\Model{
 
    public function __construct(){
        parent::__construct();
    }
 
 
    public function getHaslo($login){
        $data = $this->_db->select("SELECT haslo FROM ".DB_PREFIX."users WHERE login = :login",
                                array(':login' => $login));
        return $data[0]->haslo;
    }    
 
    public function getID($login){
        $data = $this->_db->select("SELECT id FROM ".DB_PREFIX."users WHERE login = :login",
                                array(':login' => $login));
        return $data[0]->id;
    }        
 
    public function checkToken($token){
        $data = $this->_db->select("SELECT id FROM ".DB_PREFIX."users WHERE token = :token",
                                array(':token' => $token));
        return $data[0]->id;
    }        
 
    public function insert($array){        
        return $this->_db->insert(DB_PREFIX.'users',$array);
    }    
 
    public function update($array, $where){        
        return $this->_db->update(DB_PREFIX.'users',$array, $where);
    }      
 }
 
[PHP] pobierz, plaintext 

Do oceny powinieneś raczej dac to co robią klasy Core`owe czyli główny kontroler, model i widok itd.

Ale one teoretycznie nic nie robią dlatego nie dawałem bo po co dawać Corową klasę kontrolera jak jest w niej tylko uruchamiana klasy widoku.

Dla mnie Core Class to kontroler i model usera ale to pewno dlatego że dopiero się uczę OOP stare przyzwyczajenia nie pozwalają mi pisać czysto obiektowo tylko funkcje owijam w klasy.

Dlatego liczę na kilka cennych uwag aby ruszyć z miejsca.

[PHP] pobierz, plaintext 
<?php
 
 namespace Core;
 
use Core\View;
 
abstract class Controller
{
    public $view;
 
    public function __construct()
    {
        $this->view = new View();
    }
}
 
[PHP] pobierz, plaintext 

No ok. A gdzie robisz rozruch całego frameworka i parsowanie adresow ? Jaki jest flow ?

polecam poczytać o psr, czyli spisany zbiór zasad standardu na którym warto sie opierać wpisz w google psr php, jest nawet po polsku ale nwm czy wszystko przetłumaczyli, choć ang nie powinie być problemem co do kodu powyżej z ost posta to widok przekazuj jako zależność, nie twórz go w w klasie, tutaj warto poczytać o depedency injection i o IoC

Jak zajrzysz do psr to tam zalecają nie używać uderscore dla prywatnych, no i raczej teraz piszemy stylem cammelCase

No i wykorzystaj autoloading z psr-4 ( jest jeszcze psr-0, ale to już można zapomnieć )

W dodatku staraj się trzymać jednego standardu bo raz klamra w tej samej lini raz w innej, dla metod i klas w nowej

Ale to jest Abstrakcyjna klasa kontrolera stworzona tylko do uruchamiania widoku, ewentualnie wersji językowej ale nie zaimplementowałem jej jeszcze.

Co do uderscore to spodobał mi się ten sposób eksponowania zmienny prywatnych, ale nie wiem czy jest sens wyróżniać zmienne prywatne, publiczne i chronione, czy to w ogóle się przydaje?

owszem, ale na etapie projektowania powinieneś brać pod uwagę to, że ktoś kto powiedzmy będzie chciał skorzystać z Twojego framewoka, zamiast tego widoku który mu serwujesz zechce użyć innego, wtedy by musiał cały kod zmienić, żeby to zrobić , poza tym skoro to jest kontroler to powinien posiadać info o widoku ale widok nie powinien się tworzyć sam. Weźmy przykład bazy danych, chcesz ja wywołać np w 3 klasach, nie zależnych od siebie, to teraz dla każdej z nich musisz stworzyć nowa instancje, na nowo się połączyć etc...

osobiście nigdy nie używałem i nie mam problemu, wystarczy dobre IDE

I tak na przyszłość nie mieszaj polskich nazw z angielskim bo wyjdzie z tego straszny bałagan i szybko sam się zgubisz angielski to jednak standard

Model usera podobnie jak klasa Core jest jedynie abstrakcją inicjującą połączenie z bazą, Model tworzy instancję i sprawdza jej stan nie pozwalając na ponowne ustanowienie połączenia z tą samą bazą danych.

Tworząc abstrakcyjny Model2 mogę uruchomić całkowicie inną bazę danych, może to na wyrost zrobiłem ale tak samo wyszło.

Co do obsługi bazy to miałem wcześniej zaimplementowane RedBeanPHP ale przemyślałem sprawę że armata na muchę nie jest mi potrzebna dlatego zmieniłem szybko model na PDO, oczywiście brancha z RedBeanPHP zachowałem w razie czego



Wykorzystuję w tym projekcie pojedyncze klasy, które kiedyś napisałem, ale powoli zmienię nazewnictwo w jednym stylu czyli tak jak piszesz.

No właśnie tworząc Model2 a ja nie chcę Modelu2 tylko zrobić to w Model, a się w takim przypadku nie da Ograniczasz i zamykasz swój kod wokół jednej słusznej implementacji co nie jest dobre

a wystarczyło by zrobić tak:

[PHP] pobierz, plaintext 
$pdo = new PDO('mysql...');
$model = new User($pdo); //Model
$pdo2 = new PDO('sqlite...');
$model2 = new Other($pdo2); //Model
 
$controller = Controller($pdo);
[PHP] pobierz, plaintext 

Jasne, rozumiem

Zapoznaj się z http://www.php-fig.org/

pisałem przecież o tym , ale nie dałem linka