Klasa generująca formularz - dobre praktyki Opcje

[Turson]

Hej,
mam małą zagwozdkę logiczną w tworzeniu klasy generującej formularz. Teoretycznie takową klasę napisałem, ale mam kilka wątpliwości.
Od razu dodam, że aplikacja oparta jest o MVC.
Skrócony kod klasy:

[PHP] pobierz, plaintext 
<?php
class MyForm {
 
    protected $_action;
    protected $_method = 'GET';
    protected $_html;
 
    public function setAction($value) {
        $this->_action = $value;
    }
 
    public function setMethod($value) {
        $this->_method = $value;
    }
 
    public function startForm(array $options = array()) {
        $optionsHtml = '';
        if (count($options)) {
            foreach ($options as $key => $value) {
                $optionsHtml .= ' ' . $key . '="' . $value . '"';
            }
        }
        $this->_html = '<form action="' . $this->_action . '" method="' . $this->_method . '"' . $optionsHtml . '>';
    }
 
    public function endForm() {
        $this->_html .= '</form>';
    }
 
    public function createElement($type, $name, $label = null, $value = null, array $values = array(), array $options = array()) {
       // wygenerowanie kodu HTML input/textarea/select itd.
        $this->_html .= $html;
    }
 
    public function render() {
        return $this->_html;
    }
 
}
[PHP] pobierz, plaintext 

Formularz generuję w widoku, np.

[PHP] pobierz, plaintext 
<?php
$form = new MyForm();
$form->setMethod('POST');
$form->startForm();
$form->createElement('text', 'login', 'Nickname');
$form->createElement('password', 'password', 'Password');
$form->createElement('submit', 'submit', null, 'Register new account');
$form->endForm();
echo $form->render();
?>
Jakiś dalszy tekst
[PHP] pobierz, plaintext 

Tutaj mam taką rozkminę, czy formularz lepiej generować w widoku jak powyżej, czy w osobnej klasie, np. application/forms/RegisterForm.php
Pierwsze rozwiązanie widziałem w Yii, a drugie w Zendzie.

A teraz sprawa najważniejsza. Posiadam osobną klasę walidującą pola formularza wg. zasad ustalonych w modelu. Tego nie będę pokazywał, bo nie ma związku. Chcę mieć dostęp do informacji o wszystkich walidujących polach w klasie walidacyjnej.
Kontroler:
Jeżeli naciśnięto przycisk, zbieram dane z formularza, pobieram reguły walidacji i przekazuję to do klasy walidującej np. $this->validate($data, $rules)
$data zawiera tablicę $_POST. W klasie walidacyjnej następuje sprawdzenie poprawności przesłanych danych wg. $rules. W tejże klasie muszę mieć dostęp do tego jaki dane pola ma $label - i to jest cały problem. Nie mam za bardzo pomysłu jak się do tego dostać. Podejrzewam, że w klasie MyForm i metodzie createElement() muszę zapisywać dane każdego tworzonego pola i zapisywać np. w statycznej własności klasy $formData. To nie jest problem. Tylko odwołanie się do np. MyForm::$formData z klasy walidacyjnej przecież nie pobierze tych danych, bo już ich tam nie "będzie".
A po co mi wiedzieć jaki label ma dane pole? Ano po to, że jak nie przejdzie walidacji to do tablicy KlasaWalidacyjna->errors mogę dodać $label.' can not be empty'

Ten post edytował Turson 24.08.2014, 11:23:12

[RiE]

Może zamiast metody z 6 argumentami lepiej będzie przekazać do niej 1 argument w postaci tablicy w której będą się znajdowały wszystkie niezbędne informacje. Bo co w przypadku jeśli chce dodać klasę albo ID do pola tekstowego? Który parametr za to odpowiada? Ostatni? A jak będę chciał dodać klasę do label to też ostatni?

[PHP] pobierz, plaintext 
    public function createElement($type, $name, $label = null, $value = null, array $values = array(), array $options = array()) {
       // wygenerowanie kodu HTML input/textarea/select itd.
        $this->_html .= $html;
    }
[PHP] pobierz, plaintext 

Lepszym wyjściem moim zdaniem byłoby:

[PHP] pobierz, plaintext 
    protected $inputs = array();
 
    public function createElement($input) 
    {
         $this->inputs[$input['input']['name']] = $input;
    }
 
    public function getElement($name)
    {
         return $this->inputs[$name];
    }
 
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
$form = new MyForm();
$form->createElement(array(
    'input' => array(
         'name' => 'pole_1',
         'type' => 'text'
         'options' => array(
              'value' => 'Domyślna wartość',
              ...
         ),
         'attributes' => array(
              'id' => 'pole_1'
              'class' => 'form-input',
              'placeholder' => 'Tekst...',
              ...
          )
     ),
    'label' => array(),
    //'errors' => array()
));
[PHP] pobierz, plaintext 

Następnie przekazujesz ten formularz do widoku i w widoku Dekorator odpowiada za poprawne wyświetlenie.

[PHP] pobierz, plaintext 
<div class="my-form">
    <?php echo $this->renderLabel($form->getElement('pole_1)); ?>
    <?php echo $this->renderInput($form->getElement('pole_1)); ?>
    <?php echo $this->renderErrors($form->getElement('pole_1)); ?>
</div>
[PHP] pobierz, plaintext 

W renderLabel() dajesz input i tworzysz label na podstawie klucza label. W renderInput() tworzysz input na podstawie klucza input. A podczas walidacji dodajesz klucz error i jeżeli nie jest pusty to wyświetlasz błedy pod polem w którym się pojawiły.

W $rules przydałoby się jeszcze przekazać informację z jakiego validatora chcesz skorzystać bo może się zdarzyć tak że dwa validatory będą miały dwa takie same klucze np. min_length albo pattern w którym będzie wyrażenie Regexp i co wtedy? Skąd wiadomo z jakiego validatora skorzystać?