[PHP] Poprawne wczytanie $_POST do klasy Opcje

[k8dz]

Mam formularz i po kliknięciu wyślij potrzebuję przekazać jego wartości do klasy, aby móc na nich wykonać operacje. Zastanawiam się jak to poprawnie powinno być wykonane. Czy takie rozwiązanie jest poprawne, czy powinienem to jakoś inaczej zrobić?

[PHP] pobierz, plaintext 
$cena_plyty = 25; //domyślna wartość zmiennej
 
if ( isset($_POST['cena_plyty']) and is_numeric($_POST['cena_plyty']))
{
	$cena_plyty = $_POST['cena_plyty'];
}
 
$wycena = new Wycena;
$wycena->ustaw_cene($cena_plyty);
echo $wycena->wyswietl_wycene();
[PHP] pobierz, plaintext 

[vokiel]

Tak, generalnie jest OK.

Chociaż często stosuje się też inny sposób działania - wyświetlanie błędów w przypadku nieudanej walidacji danych wejściowych (zamiast przepuszczania na domyślnych wartościach). Tym samym jeśli tylko gdzieś trafisz na nieprawidłowe dane to nie próbujesz ich naprawiać, nie podstawiasz danych domyślnych (bo mogą spowodować anomalie), tylko zwracasz użytkownikowi błąd i pokazujesz gdzie zostały wpisane niepoprawne dane.

[k8dz]

Rozumiem o co chodzi, chociaż w tym przypadku tworzę prostą aplikację na własne potrzeby, więc nie muszę jej tak zabezpieczać.

Mam jeszcze jedno pytanie. Czy poprawne jest ustawienie za pomocą jednej metody wszystkich zmiennych pobranych z $_POST i później bezpośrednie odwoływanie się do tych zmiennych, czy trzeba do każdej zmiennej pobranej z $_POST tworzyć osobną metodę i potem odwoływać się do metody? Przykład poniżej:

[PHP] pobierz, plaintext 
 
$cena_plyty = $_POST['cena_plyty'];
$cena_ciecia = $_POST['cena_ciecia']; //itd, itd...
 
$wycena = new Wycena;
$wycena->ustaw_cene($cena_plyty, $cena_ciecia, $cena_oklejania, $cena_transportu);
echo $wycena->cena_plyty;
echo $wycena->cena_ciecia; //itd...
[PHP] pobierz, plaintext 

czy jednak tak:

[PHP] pobierz, plaintext 
$wycena = new Wycena;
$wycena->ustaw_cene_plyty($cena_plyty);
echo $wycena->cena_plyty();
 
$wycena->ustaw_cene_ciecia($cena_ciecia);
echo $wycena->cena_ciecia(); //itd...
[PHP] pobierz, plaintext 

[viking]

A może tak:
class Cena -> settery , gettery do poszczególnych cen.
class Wycena -> __construct(Cena $cena)

[k8dz]

ok, czyli w klasie Cena robię settery i gettery dla poszczególnych cen:

[PHP] pobierz, plaintext 
public function set_cena_plyty($cena_plyty)
   {
      $this->cena_plyty = $cena_plyty;
   } 
 
   public function get_cena_plyty()
   {
      return $this->cena_plyty;               
   } 
[PHP] pobierz, plaintext 

ale czy mógłbyś mi wytłumaczyć, do czego potrzebna jest nowa klasa Wycena i dlaczego w konstruktorze jest nazwa klasy Cena i dopiero po niej zmienna $cena?
Rozumiem zasadę działania konstruktora, ale nie rozumiem tego zapisu :/

[vokiel]

Możesz to rozdzielić na początek na dwie klasy, produktu i wyceny.
Zakładam, że produkty masz zdefiniowane gdzieś (np zapisane w bazie, i do formularza przekazujesz ich ID.
Tak, że w $_POST dostajesz tablice produktów z ich ilościami i cenami, np:

Kod

Array
(
    [0] => Array
        (
            [id] => 1
            [count] => 5
            [price] => 2.34
        )

    [1] => Array
        (
            [id] => 2
            [count] => 1
            [price] => 12.09
        )

    [2] => Array
        (
            [id] => 5
            [count] => 1
            [price] => 25.93
        )

    [3] => Array
        (
            [id] => 9
            [count] => 1
            [price] => 52.83
        )
)

[PHP] pobierz, plaintext 
$valuation = new Valuation();
 
foreach($_POST['items'] as $item){
    $product = new Product($item['id']);
    $product->setPrice($item['price']);
    $product->setQuantity($item['quantity']);
    $product->setCount($item['count']);
 
    $valuation->addProduct($product);
}
 
echo "Suma: ".$valuation->getSum().PHP_EOL;
foreach($valuation->getProducts() as $product){
    echo 'Produkt '.$product->getId().' |  nazwa:  '.$product->getName().' | cena: '.$product->getPrice().PHP_EOL;
}
[PHP] pobierz, plaintext 

Api klas dopiszesz sobie sam, oczywiście pamiętając o walidacji danych.

[k8dz]

wielkie dzięki! o to mi właśnie chodziło przetestowałem na prostym przykładzie i działa, więc mogę teraz spokojnie działać

ale mam jeszcze pytanie - gdzie i kiedy najlepiej jest robić walidacje danych z $_POST? Czy w pliku, który przetwarza formularz zrobić odpowiednie if...else i jeśli wszystko jest ok, to wtedy utworzyć obiekty i wpuścić $_POST do dalszego przetwarzania; czy tak jak w przykładzie który podałeś, wpuścić $_POST do obiektu i dopiero później za pomocą odpowiedniej metody sprawdzać, czy jest ok; czy może lepiej było by utworzyć osobną klasę zajmującą się tylko walidacją danych i w niej odpowiednie metody sprawdzające czy z $_POST jest wszystko ok??

ps. może zadaję dziwne pytania, ale powiem tak: zajmuję się produkcją mebli na wymiar, chociaż jestem z wykształcenia informatykiem Mam sporo prostych zrobionych własnoręcznie skryptów przydatnych w mojej firmie. Wszystkie są napisane w php i totalnie strukturalnie, z całą masą różnych if..else, warunków i itp zbędnego kodu. Po 10 latach przerwy w programowaniu postanowiłem zrobić kilka kolejnych aplikacji, jednak teraz chcę to zrobić w miarę profesjonalnie i obiektowo, a niestety kursy są zbyt ogólnikowe i nie można z nich wszystkiego się dowiedzieć. Dlatego proszę o porady ;

[vokiel]

Co do walidacji to różne są podejścia. Raz jako osobna klasa validatora, raz spięte z generatorem formularza. W sumie osobna klasa i tak się przyda, kwestia gdzie jej użyjesz. Ja wyznaję zasadę "Avoid Else, Return Early" i uważam, że czym wcześniej zastosuje się sprawdzanie, które może przerwać kod tym lepiej. Po pierwsze będzie szybciej, mniej niepotrzebnych obiektów/zmiennych się stworzy, mniej kodu się wykona, mniejsza szansa, że wykona się kod, który nie powinien. No i użytkownik dostanie odpowiedź szybciej.

Także w prostych aplikacjach walidację moim zdaniem lepiej jest robić na samym początku (o ile masz już na tym etapie informacje o poprawnym formacie). Czyli tutaj spokojnie po odebraniu danych z formularza możesz zrobić walidację w pierwszej kolejności, a jeśli ona się powiedzie, to dopiero wtedy tworzyć obiekty produktów i wycenę.

Mam sporo prostych zrobionych własnoręcznie skryptów przydatnych w mojej firmie. Wszystkie są napisane w php i totalnie strukturalnie, z całą masą różnych if..else, warunków i itp zbędnego kodu.

Kiedyś poczyniłem aplikację, a ostatnio dowiedziałem się, że po 10 latach nadal z niej korzystają. Voyager 2 leci już 40 lat i nadal się komunikuje z NASA. Stary kod, który działa to niekoniecznie zły kod :-D

[k8dz]

tak właśnie obstawiałem, że taka podstawowa walidacja powinna się odbywać na samym początku, a dodatkowa klasa validująca może bardziej szczegółowo sprawdzać, co znajduje się w $_POST

a czy taki zapis ma sens?:

[PHP] pobierz, plaintext 
$sprawdz = new Validacja();
$sprawdz->validacja_post($_POST['cena1'], $_POST['cena2']);
[PHP] pobierz, plaintext 

według mnie jest to bez sensu, jeśli nie sprawdzi się chociażby, czy $_POST w ogóle istnieje, ponieważ jeśli nie ma $_POST to zwróci błąd o braku $_POST

ps. przez 4 lata nauki w technikum uczyliśmy się głównie programowania strukturalnego, a tak na prawdę tylko jakieś podstawy javy były obiektowe, ale to była dla mnie wtedy czarna magia
nie twierdzę, że stary kod jest zły, bo faktycznie takie aplikacje działają, ale tak dla własnej satysfakcji chcę chociaż w miarę opanować tą obiektowość

[vokiel]

Językowo bym się trzymał angielskich nazw, jeśli już chcesz PL to Walidacja nie Validacja. Ale nadal, Validator brzmi lepiej ;-)

Najczęściej walidatory przyjmują reguły, na podstawie których sprawdzają czy pole ma poprawą wartość.

Możesz np zrobić tak, że przekazujesz całą tablicę $_POST i dla nazw pól ustawiasz warunki, np not_empty, int, string, max_lengt: 16, min_length: 8, unique, match_regex itd.
Następnie w walidatorze bierzesz pole, sprawdzasz czy istnieje w $_POST, czy jest wymagane, czy jego format jest zgodny ze wzorcem. Jeśli nie to zbierasz błędy i na koniec zwracasz true/false plus tablica błędów.

Same sprawdzanie formatu możesz zrobić w jakiejś osobnej klasie, w której zdefiniujesz sobie metody na każde porównanie, w stylu:

[PHP] pobierz, plaintext 
class Rule {
    public function notEmpty($item)
    {
        return !empty($item);
    }
 
    public function matchRegex($item, string $regex)
    {
        return preg_match($item, $regex);
    }
}
[PHP] pobierz, plaintext 

A do walidatora dodajesz przez:

[PHP] pobierz, plaintext 
$validator = new Validator();
$validator->addRule('price', new Rule('not_empty'));
$validator->addRule('price', new Rule('decimal'));
 
$validator->addRule('quantity', new Rule('not_empty'));
$validator->addRule('quantity', new Rule('greater_than', 0));
 
if (!$validator->valid()){
    // Wróć do formularza, podświetl błędne pola.
}
[PHP] pobierz, plaintext 

[k8dz]

ten sposób mi się podoba, ale kurde nie do końca wszystko rozumiem.. Klasa Rule jest dla mnie jasna, ale metoda addRule() już nie za bardzo. Rozumiem, że metoda addRule() przyjmuje 2 parametry. Pierwszy parametr, to ten który ma być sprawdzony, ale ten drugi parametr z klasą Rule nie rozumiem.. jakbyś mi to jeszcze wytłumaczył i już nie zawracam więcej gitary

[viking]

Poczytaj po prostu jak robią to inni https://docs.laminas.dev/laminas-validator/ POdstawowa zasada obecnie. Nie wymyślaj swoich rozwiązań na siłę. Po to są gotowe komponenty żeby z nich korzystać.
Dodatkowo stosuj się do PSR-2.

Ten post edytował viking 14.02.2020, 20:51:43

[vokiel]

ten sposób mi się podoba, ale kurde nie do końca wszystko rozumiem.. Klasa Rule jest dla mnie jasna, ale metoda addRule() już nie za bardzo. Rozumiem, że metoda addRule() przyjmuje 2 parametry. Pierwszy parametr, to ten który ma być sprawdzony, ale ten drugi parametr z klasą Rule nie rozumiem.. jakbyś mi to jeszcze wytłumaczył i już nie zawracam więcej gitary

Tak, addRule() dodaje kolejne elementy do sprawdzenia w walidatorze. Bo domyślnie (bez ich podania) on nie wie co ma sprawdzać i na jakich polach. W samym walidatorze możesz po prostu zrobić sobie tablicę pól z przypisanymi do nich regułami do uruchomienia. Na koniec, przy valid() iterujesz po każdym polu, pobierasz jego wartość z $_POST i odpalasz na nim każdą regułę. Szybki przykład:

[PHP] pobierz, plaintext 
class Validator
{
    protected $items = [];
    protected $invalid = [];
 
    public function addRule(string $field, Rule $rule)
    {
        if (!isset($this->items[$field]) {
            $this->items[$field] = [];
        }
        array_push($this->items[$field], $rule);
    }
 
    public function valid(array $values)
    {
        foreach($this->items as $item => $rules) {
            $value = $values[$item] ?? null;
            forach ($rules as $rule) {
                if (!$rule->check($item)) {
                    array_push($this->invalid, [
                        $item => [
                            'value' => $value,
                            'rule' => $rule
                        ]
                    ] );
                }
            }
        }
 
        return empty($this->invalid);
    }
 
    public function getInvalid()
    {
        return $this->invalid;
    }
}
 
$validator = new Validator();
$validator->addRule('price', new Rule('not_empty'));
$validator->addRule('price', new Rule('decimal'));
 
$isValid = $validator->valid($_POST);
if (!$isValid){
    $validationErrors = $validator->getInvalid();
}
[PHP] pobierz, plaintext 

Oczywiście, jeśli nie chcesz sobie ćwiczyć programowania to możesz wziąć dowolnego gotowca i go użyć tak jak pisał @viking.