Cześć, jakiś czas temu zacząłem się bawić z routingiem, ale nie chcę zrobić sobie złych nawyków ani tym bardziej złego kodu. Będę bardzo wdzięczny jeśli rzucicie na to okiem i powiecie czy można tak robić i czy jest to dobre rozwiązanie. Jeśli nie to proszę o sugestię co zmienić/co i jak zrobić.
Otóż korzystam z AltoRouter. Mój kod wygląda tak:
// Router
$router = new altoRouter();
$view = new view($account, $profile);
$router->map('GET', '/', function() use($view){
$view->render('homepage', 'Panel');
});
Stworzyłem obiekt z metodą render, aby nie includować po stokroć plików header oraz footer. Pierwszy string w powyższym kodzie to nazwa pliku jaki ma zostać wczytany, a drugi to tytuł strony. Natomiast klasa wygląda tak:
class view{
public function __construct($account, $profile){
$this->account = $account;
$this->profile = $profile;
}
public function render($page_name, $page_title){
$this->account->isLogged();
global $db;
if(is_readable('views/panel/templates/header.php')){
require 'views/panel/templates/header.php';
}
if(is_readable('views/panel/'.$page_name.'.php')){
require 'views/panel/'.$page_name.'.php';
}
if(is_readable('views/panel/templates/footer.php')){
require 'views/panel/templates/footer.php';
}
}
}
W docelowej stronie (homepage.php), chcąc uzyskać dostęp do jakiejś zmiennej, muszę wcześniej przekazać ją do metody. I na końcowej stronie wyświetlam ją poprzez $this-> moja zmienna/metoda. Czy to jest dobry sposób czy macie jakieś lepsze rozwiązania? Bo przyznam, że wydaje mi się to dziwne, że muszę przekazywać dużo zmiennych do konstruktora klasy view (w przykładzie ich nie ma, ale w skrypcie to m.in zmienna z bazą danych, kontem, profilem i jeszcze parę innych, które akurat na tej stronie są mi potrzebne)