Drukowana wersja tematu

Napisany przez: Lukasz4 19.05.2019, 15:37:18

Cześć, jestem dość świeży w routingu. Co prawda już się zacząłem z tym bawić, 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:

[PHP] pobierz, plaintext 
// Router
$router = new altoRouter();
$view = new view($account, $profile);
 
$router->map('GET', '/', function() use($view){
	$view->render('homepage', 'Panel');
});
[PHP] pobierz, plaintext 

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:

[PHP] pobierz, plaintext 
class view{
		public function __construct($account, $profile){
			$this->account = $account;
			$this->profile = $profile;
		}
		public function render($page_name, $page_title){
			$this->account->isLogged();
			http://www.php.net/global $db;
			if(http://www.php.net/is_readable('views/panel/templates/header.php')){
				require 'views/panel/templates/header.php';
			}
			if(http://www.php.net/is_readable('views/panel/'.$page_name.'.php')){
				require 'views/panel/'.$page_name.'.php';
			}
			if(http://www.php.net/is_readable('views/panel/templates/footer.php')){
				require 'views/panel/templates/footer.php';
			}
		}
	}
[PHP] pobierz, plaintext 

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)

Napisany przez: adbacz 20.05.2019, 17:55:08

Do metody render przekaż sobie tablicę. W metodzie render ją przyjmij i wykonaj funkcję extract(). Funkcja ta stworzy tyle zmiennych ile będziesz miał indeksów w tablicy. Dzięki czemu każdą wartość z tablicy będziesz miał w osobnej zmiennej.

[PHP] pobierz, plaintext 
$router->map('GET', '/', function() use($view){
    $view->render('homepage', [
        'title' => 'Panel'
    ]);
});
 
// W metodzie renderującej
public function render($view, http://www.php.net/array $data = []) {
    http://www.php.net/extract($data);
 
    if(http://www.php.net/is_readable('views/panel/'.$page_name.'.php')){
        require 'views/panel/'.$page_name.'.php';
    }
}
 
// W widoku
http://www.php.net/echo $title;
[PHP] pobierz, plaintext 

Poszukaj sobie przykładów renderowania widoków z innych systemów szablonów. Poczytaj dokumentację, kod. Jeśli uczyć to od najlepszych

Podeślę Ci dwa linki do krótkiego snippetu kodu dwóch systemów szablonów, na których możesz na szybko zobaczyć jak je uruchomić.
Twig - https://phpcenter.pl/biblioteki/twig-system-szablonow/
Aura View - https://phpcenter.pl/biblioteki/aura-view-system-szablonow/

W obu jest podobnie. Podajemy nazwę widoku i tablicę z danymi które mają być do niego przesłane.

Napisany przez: viking 20.05.2019, 18:48:20

Ja bym wziął cały https://www.slimframework.com/ bo np global $db; w render woła o pomstę do nieba.

Napisany przez: Lukasz4 20.05.2019, 19:10:02

Dziękuję bardzo za wskazówki
a to global $db to było tylko chwilowe i przez przypadek tu wstawiłem. Już tego nie mam w kodzie