[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]

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.