Walidacja danych umieszczona w MVC Opcje

[brzoza91]

Witam
Chciałbym stworzyć walidacje danych umiejscowioną w MVC. W poniższym kodzie walidacje umieściłem w kontrolerze(akcja insert). Mój cały problem sprowadza się do tego miejsca

[PHP] pobierz, plaintext 
if(!$validation->isError==0){
 
         //co tutaj zrobic
 
 
 
        }else {
            $model=$this->loadModel('dish');
            $model->insert($_POST);
            $this->redirect('?task=dish&action=index');
 
        }
[PHP] pobierz, plaintext 

Gdy dane są poprawne to jest ładowany model, który umieszcza ładnie i poprawnie dane w bazie danych.
Lecz co zrobić gdy dane są niepoprawne? W moim założeniu jest żeby po wprowadzeniu błędnych danych wyświetliła się dalej strona z formularzem z informacją o błędnych danych. Więc jak zrobić aby przekazać obiekt $validation do widoku z formularzem tak abym mógł potem w widoku wyciągnąć informacje o błędach


dishcontroller.php

[PHP] pobierz, plaintext 
require_once ('controller.php');
require_once ('validator.php');
 
 
class DishController extends Controller{
 
 
    public function index(){
        $view=$this->loadView('dish');
        $view->index();
    }
 
    //laduje widok z formularzem dodania potrawy
    public function add(){
        $view=$this->loadView('dish');
        $view->add();
    }
    //laduje widok z przegladem potrawy
    public function show(){
        $view=$this->loadView('dish');
        $view->show($_GET['id']);
    }
 
    //laduje model ktory umieszcza dane w bazie
    public function insert(){
        $validation= new Validator;
 
        $validation->MessageText=NULL;
        $validation->isError=0;
 
        $dishname=$_POST['dishname'];
 
        echo "PRZED:<br>";
        $validation->viewMessage();
 
        if(!$validation->CheckString($dishname,2)){
            $validation->setMessage('dishname','Nazwa dania jest zbyt krótka');
        };
 
        echo "PO:<br>";
        $validation->viewMessage();
 
        if(!$validation->isError==0){
 
         //co tutaj zrobic
 
 
 
        }else {
            $model=$this->loadModel('dish');
            $model->insert($_POST);
            $this->redirect('?task=dish&action=index');
 
        }
    }
 
    public function delete(){
        $model=$this->loadModel('dish');
        $model->delete($_GET['id']);
        $this->redirect('?task=dish&action=index');
    }
};
[PHP] pobierz, plaintext 

dishview.php

[PHP] pobierz, plaintext 
class DishView extends View{
 
    public function index(){
        $dish=$this->loadModel('dish');
        $this->set('dishList', $dish->getAll()); 
        $this->render('indexDish','dish/');
    }
 
    public function add($obj=null){
        $this->render('addDish', 'dish/'); //laduje plik z formularzem addDish.html.php, formularz ten przesyła dane do akcji insert();
        } 
 
 
    public function show($id){
        $dish=$this->loadModel('dish');
        $this->set('dishShow', $dish->getOne($id)); 
        $this->render('showDish', 'dish/');
    }
}
[PHP] pobierz, plaintext 

[thek]

Jak dla mnie chyba sensowniejsze byłoby wrzucenie walidacji do modelu i tam reagowanie na tego typu sytuację. Kontroler tylko by monitorował "co jest grane". Przykładowo walidacja mogła by rzucać jakiś wyjątek informujący o nieprawidłowym jej przebiegu.

[brzoza91]

umieszczenie walidacji w modelu daje mi w sumie to samo, bo nie wiem jak wtedy wyświetlić stronę z formularzem i informacją o błędach.

Ten post edytował brzoza91 19.07.2012, 18:55:19

[thek]

No pomyśl... Masz model z metodą, która może rzucić wyjątek w przypadku naprawdę problemów, albo strukturę danych wynikową wspartą strukturą zwracającą komunikaty błędów. Całość wywołania metody z modelu obejmujesz try-catch w razie czego, ale monitorujesz też strukturę błędów. Jeśli jest ona pusta to ok, ale jeśli nie - to reagujesz.

[Pilsener]

Najsensowniej jest powiązać ściśle walidację z formularzem i to niezależnie od OOP czy MVC. W strukturalnym radziłem sobie tak, że definiowałem walidację w szablonie formularza a następnie ją obsługiwałem odpowiednimi funkcjami - wystarczyło raz napisać validator a potem tylko go stosować.

Tutaj radzę na obiektach, jakoś tak logicznie:
Fomularz - obiekt
Walidator - też obiekt

Zatem:

[PHP] pobierz, plaintext 
 
$form = new My_Form();
$validator = new My_Validator();
$form->addValidator($validator);
 
if($form->isValid()){
 
}
[PHP] pobierz, plaintext 

Oczywiście walidator można dodać bezpośrednio w klasie My_Form, wtedy nie zaprzątasz sobie nim głowy wcale, w kontrolerze skupiasz się na logice aplikacji.

[thek]

@Pilsener: Bo to bardzo dobre podejście. Klasa walidatora powinna być osobną, bo w końcu to ona się zajmuje tym zadaniem, nie formularz sam w sobie i dzięi temu całość jest elastyczniejsza, gdyż nie ograniczamy walidacji jedynie do formularza. Możemy walidować wtedy wszelkie dane przychodzące do aplikacji, także pliki, strumienie. Walidator powinien mieć więc funkcje, która przyjmuje jakieś dane i konfrontuje je z regułami dla tych danych wybranymi oraz informuje o powodzeniu lub nie. Można jeszcze wcześniej dane przefiltrować (trimowanie, usuwanie znaków niedozwolonych itp). Przykład?

[PHP] pobierz, plaintext 
$form = new Form();
$form->add_data($_POST);
$form->add_data($_FILES);
$filters = new Filterset();
$filters->add_rules(
  array(
     'trim',  //bez klucza, to wytrimuje wszystkie pola formularza
     'zipcode' => array('expr' => '[^0-9-]*')  //oczyści pole z wszystkich znaków innych niż podane w regule 
  )
);
$validator = new Validator();
$validator->add_rules(
  array(
    'id' => array(
        'not_empty',
        'numeric',
        'greater_than' => 0,
    ),
    'zipcode' => array('match' => '[0-9]{2}-[0-9]{3}'),
    'username' => array('custom' => array('user_func' => 'tu_funkcja_usera_do_sprawdzenia_unikatowości_nicka')),
    'password' => array('min_length' => 8),
    'retype_pass' => array('match_field' => 'password'),
    'avatar' => array('not_empty', 'sizelimit' => '128 kB',)
  )
);
$form->apply_filters($filters);
$form->apply_validator($validator);
if($form->is_valid())
{
}
[PHP] pobierz, plaintext 

Oczywiście to przykładowo jest i może być inaczej w modelu implementowane i nie musi nawet być podobne, ale takie coś jest w miarę elastyczne, gdyż co dane wejściowe (niekoniecznie formularz!) to można reguły modyfikować, dodawać nowe operacje dla danych, czy wręcz pozwolić sobie na wywoływanie funkcji własnych. Zauważcie custom przy regułach dla pola username, gdzie można przekazać obiekt modelu User z wywołaniem funkcji sprawdzającej unikatowość. To tylko przykład, który obrazuje jak bardzo elastyczny można walidator zrobić. Przy walidacji formularza jedynie sprawdzi się czy formularz potraktowany dodanym walidatorem zawiera strukturę błędów czy może jest ona pusta. Zwróć też uwagę, że można obsłużyć bardziej złożone rzeczy. Ja w przykładzie połączyłem tablice POST i FILES Równie dobrze mogłem posłać do walidacji uchwyt do pliku, strumień czy wywołanie z CLI. Zależnie jak bardzo klasa walidatora byłaby uniwersalna i jakie reguły bym przewidział. Jednak ich uzupełnianie byłoby bajecznie proste.