Question

I have a form which I need for some actions but not for others. In the actions that need it, I repeat the same code.

$form = New BlaBla_Form();

I’ve seen an example which declared it in init(). This means the form gets initialized even for actions that don’t need it. It does make the code cleaner, but is it resource intensive enough to make it a bad practice?

Answer 1

Try this:

  class RegistrationController extends Zend_Controller_Action
  {
      protected $_form;

      public function fooAction()
      {
            // ...

            if($this->getForm()->isValid()) { }

           // ...
      }

      protected function getForm()
      {

          if (null === $this->_form) {
              $this->_form = new My_Form_Registration();
          }

          return $this->_form;
      }
  }