Bessere Möglichkeiten der Verwendung von Zend_Form in Zend MVC

Question

Wenn Zend_Form mit finde ich mich viel Controller-Methoden erstellen, die wie folgt aussehen:

function editPersonAction() 
{ 
    $model = $this->getPersonModel(); 
    $form = $this->getPersonEditForm(); 
    if ($this->getRequest()->isPost() { 
     $data = $this->getRequest()->getPost(); 
     //$form->populate($data); [removed in edit] 

     if ($form->isValid($data)) { 
      $data = $form->getValues(); 
      $model->setFromArray($data); 
      // code to save model then decide which page to redirect to 
     } 
    } else { 
     $form->populate($model->toArray()); 
    } 
    $this->view->form = $form; 
} 

Die meisten dieser Code ist immer das gleiche, und ich bin es sicher sind bessere Möglichkeiten, dies zu tun. Welche anderen Muster verwenden Menschen mit Zend_Form, um die Menge des verwendeten Standardcodes zu reduzieren?

Answer 1

Ist $ form-> populate() wirklich notwendig? IIRC ungültige Formulare werden automatisch gefüllt.

Answer 2

Um ehrlich zu sein, habe ich auch ähnliche Aktionen in meinen Controllern. Eine Sache, die ich tue, um das Gewicht in der Steuerung und durch Konvention zu halten, besteht darin, die Validierungsprüfung im Modell durchzuführen. Ich rufe auch das Formularobjekt aus dem Modell um dies zu erleichtern (man könnte dies bereits in Ihrem Controller über Ihre getPersonEditForm Methode tut Also, wenn ich Ihre Aktion geschrieben hatte, es würde wie folgt aussehen:.

function editPersonAction() 
{ 
    $model = $this->getPersonModel(); 
    $form = $this->getPersonEditForm(); 

    if($this->getRequest()->isPost()) 
    { 
     if($model->setFromArray($this->getRequest()->getPost())) 
     { 
      // code to decide which page to redirect to 
     } 
    } 
    else 
    { 
     $form->populate($model->toArray()); 
    } 

    $this->view->form = $form; 
} 

So dann im Modell Methode setFromArray würde ich:

public function setFromArray(array $data) 
{ 
    $form = $this->getRegistrationForm(); 

    if($form->isValid($data)) 
    { 
     // code to save model state 

     return true; 
    } 
    else 
    { 
     return false; 
    } 
} 

Zugegeben, es ist nicht wesentlich prägnanter als Ihre bestehende Ansatz ist und ich, wie sie selbst, fühlen sich oft das besser abstrahiert werden könnte

Answer 3

ich mag zu halten. so viel wie möglich lich in den Modellen

function editPersonAction() 
{ 
    $model = $this->getPersonModel(); 

    if ($this->getRequest()->isPost() { 
     $data = $this->getRequest()->getPost(); 

     if ($model->validateForm($data)) { 
      // code to save model then decide which page to redirect to 
     } else { 
      // error management 
     } 
    } 

    $this->view->form = $model->getForm(); 
} 

im Modell also dann müsste ich:

public function validateForm(array $data) 
{  
    $form = $this->getForm(); 

    if($form->isValid($data)) 
    { 
     // code to save model state 

     return true; 
    } else { 
     return false; 
    } 
} 

public function getForm($instance = 'default') { 
    if (!isset($this->_forms[$instance])) { 
     $this->_forms[$instance] = new Your_Form_Class();   
     $this->_forms[$instance]->populate($this->toArray()); 
    } 

    return $this->_forms[$instance]; 
} 

Und Sie diese Methoden zu einem abstrakten Modell hinzufügen können, die alle Ihre Anwendungsmodelle erweitern würden, und dann auch nur überschreiben Sie, wenn Sie etwas Besonderes tun müssen.