[PHP]OOP - kilka pytań o sens i poprawność "mojego" kodu Opcje

[sadistic_son]

Cześć,
Przerobiłem na własne potrzeby kod z ksiązki ale nie do końca rozumiem, czy robię to logicznie. Generalnie to kod działa, ale sądzę, że mam tam zbędne kawałki. dodatkowo pytanie - czy to w ogóle tak powinno wyglądać?

klasa PageDisplay:

[PHP] pobierz, plaintext 
namespace nameOne;
 
class PageDisplay
{
    private $request;
    private $post;
    private $get;
 
    public function __construct($request)
    {
        $requestString = \explode("?", $request);
        $this->request = empty($requestString) ? $request : $requestString[0];
    }
    public function processRequest()
    {
 
        if (!$this->request) {
            return;
        }
 
        $this->post = filter_input_array(INPUT_POST, FILTER_SANITIZE_STRING);
        $this->get = filter_input_array(INPUT_GET, FILTER_SANITIZE_STRING);
 
        switch ($this->request) {
            case "/":
                $title = 'Product List';
                $pageContent = "ProductList";
                include("ProductList.php");
                break;
            case SUBDOMAIN."add-product":
                $title = 'Product Add';
                $pageContent = "ProductAdd";
                include("ProductAdd.php");
                break;
            case "/mass_delete":
                //metoda massDelete
                break;
            case "/save":
                //metoda save new product
                break;
            default:
                $title = 'Product List';
                $pageContent = "ProductList";
                include("ProductList.php");
                break;
 
        }
        $this->displayHeader($title);
        require('views/' . $pageContent . '.view.php');
        $this->displayFooter();
    }
 
 
    private function displayHeader($title)
    {
        require_once("views/html/header.html");
    }
 
 
    private function displayFooter()
    {
        require_once("views/html/footer.html");
 
    }
 
}
[PHP] pobierz, plaintext 

Teraz w index.php mam wywołanie tej klasy i metod:

[PHP] pobierz, plaintext 
include("PageDisplay.php");
$page = new PageDisplay($_SERVER['REQUEST_URI']);
$page->processRequest();
[PHP] pobierz, plaintext 

Moje pytania.

1. Uważam, że obiekt klasy PageDisplay jest zbędny. Skoro odwołuję się tylko do jej metod to przerobiłbym processRequest() , displayHeader($title) oraz displayFooter() na metody statyczne i w index.php odwoływałbym się do PageDisplay::processRequest Ma to sens?

2. Czy w $page = new PageDisplay($_SERVER['REQUEST_URI']); powinienem jako paramert podać właśnie REQUEST_URI? Właściwie tylko to działa....

3. Jak mam się odwoływać do $_POST czy $_GET w reszcie kodu ? No bo już nie tradycyjnie if(isset($POST['add'])){ //add(); } skoro mamy taki zapis $this->post = filter_input_array(INPUT_POST, FILTER_SANITIZE_STRING);

Trochę się w tym pogubiłem i potrzebuję naprowadzenia. Dzięki z góry.

Ten post edytował sadistic_son 5.01.2023, 10:21:38

[sadistic_son]

Tak, chcę się uczyć nowych dobrych praktyk. Ale jak coś zapamiętałem inaczej, a nie dowiedziałem się jeszcze że obecnie wygląda inaczej to dopytuję, nie kłócę się (IMG:style_emoticons/default/tongue.gif) Ok, czyli typujemy zmienne. Kurde, sporo zmieniania kodu przede mną (IMG:style_emoticons/default/sad.gif) Chyba zostawię to na koniec, na dopieszczanie kodu jak już wszystko będzie działać.

przyklady: $request i $this->request
przeciez te zmiennej nazywaja sie tak samo a zawieraja zupelnie co innego

Ten fragment zrozumiałem, że $request oraz $this->request zawierają co innego, czyli to inne zmienne. Nie to miałes na myśli? Więc co?

To powinno byc przekazane w konstruktorze PAgeDisplay. Generalne zadne klasy nie powinny wiedziec co sie dzieje poza nimi, czyli nie powinny miedzy innymi znac jakie sa stale w twojej aplikacji. Klasa to zamkniety twor, jesli cos potrzebuje to dostaje to w konstruktorze

A ok, czaję. Czyli tak?

[PHP] pobierz, plaintext 
// (...)
private $subdomain;
// (...)
    public function __construct($request)
    {
// (...)
        $this->subdomain = SUBDOMAIN;
    }
 
// (...)
 
            case $this->subdomain."add-product":
[PHP] pobierz, plaintext 

Co do pytania 2 to...
Konstrukcja switch w klasie PageDisplay zakłada, że zostanie zaincludowane coś w zależności od tego co user kliknie, wpisze itd. Czyli założyłem, że jak wpisze strona.pl/stronka/add-product no to go przeniesie na stronę dodawania produkctu. W tym wypadku zczytuję z $_SERVER['REQUEST_URI'] to co user wpisał w pasku adresu, lub czy kliknął w link do tego adresu. Ale kiedy naciśnie on submit w formularzu przenoszącym do stronka.pl/stronka i prześle dane $_POSTem czy $_GETem (GET nie planuję używać) to jakoś muszę wychwycić, że np został naciśnięty submit i że wypełnił input. Stąd moje pytanie jak to w OOP realizować. Czy w switchu, w konkretnym case powinienem mieć jeszcze zwykłe if(isset($_POST['submit'] && $_POST['inputText'])) ? I z ostatnim zdaniem wiąże się moje 3. pytanie. Skoro mam utworzyć oddzielną klasę do obsługi post i get (nazwijmy ją requestProcessor) to mam się potem tak odwoływać do danych przesłanych z formularza?

[PHP] pobierz, plaintext 
$dataFromForm = new RequestProcessor();
 
// zamiast if(isset($_POST['submit']))
if(isset($dataFromForm->postArray['submit']))
[PHP] pobierz, plaintext 

Dobrze rozumuję?
Moje pytania mogą być pokrętne bo w OOP czuję się trochę jak dziecko we mgle.


Ten post edytował sadistic_son 5.01.2023, 11:19:37