[ZF] Optymalizacja Opcje

[mate]

Witam, mam pytanie związane ze zwiększeniem optymalizacji mojego kodu. Mam zrobione klasy abstrakcyjne i większość zawsze jest dziedziczona w moich akcjach, jednak od czasu do czasu zdarza się, że mam jakieś akcje w kontrolerach, które muszą się różnić od podstawowej i wtedy przeciążam całą akcję, a w 90% i tak powielam kod by zmienić tylko 2 linie. Dlatego chciałem zapytać co jest dobre na zastosowanie tego aby ciągle nie powielać kodu ?
Chodzi mi np. o takie coś:

[PHP] pobierz, plaintext 
        $this->_helper->layout()->disableLayout();
        $this->_helper->viewRenderer->setNoRender(true);
        if(!$this->getRequest()->isXmlHttpRequest()) return;
 
        if ($this->getRequest()->isPost()) {
            $form = new Default_Form_ContactForm();
            if ($form->isValid($this->getRequest()->getPost())) {
              ------------------------------------------------------------------
               TU JAKIEŚ ZADANIA, KTÓRE SĄ UNIKALNE DLA AKCJI
              ------------------------------------------------------------------
            }
            $this->view->form = $form;
        } else {
            throw new Exception('Błędny adres!', 404);
        }
[PHP] pobierz, plaintext 

Jak widać wszystkie te linie mogą być spokojnie raz zaimplementowane, a nie powtarzane tylko dlatego, że coś się różni w tym bloku.
Pozdrawiam

Ten post edytował mate 1.02.2013, 12:07:12

[CuteOne]

[PHP] pobierz, plaintext 
public function viewAction() {
 
        $params = $this->getAllParams();
 
        $this->_helper->layout()->disableLayout();
        $this->_helper->viewRenderer->setNoRender(true);
        if(!$this->getRequest()->isXmlHttpRequest()) return;
 
        if ($this->getRequest()->isPost()) {
            $form = new Default_Form_ContactForm();
            if ($form->isValid($this->getRequest()->getPost())) {
 
                    $method = $params['unikalny_parametr_dla_akcji'].'Forward';
 
                    if(method_exists($this, $method)) {
                           $this->$method($form);
                    }
                    else { throw new Zend_Exception(); }
            }
            $this->view->form = $form;
        } else {
            throw new Exception('Błędny adres!', 404);
        }
}
 
public function ahaForward($form) { }
 
public function eheForward($form) { }
[PHP] pobierz, plaintext 

lub wykorzystaj wbudowany _forward()

[irmidjusz]

rozwiązanie tego jest banalnie proste, poszukaj info o wzorcu projektowym template method, po polsku methoda szablonowa

[karolrynio]

Osobiście użyłbym: Zend_EventManager (dodany chyba w wersji 1.12)
Może robic dokładnie to o czym piszesz.
W Zendzie 2 przykładowy kod wyglądałby tak:

[PHP] pobierz, plaintext 
      $this->_helper->layout()->disableLayout();
$this->_helper->viewRenderer->setNoRender(true);
if(!$this->getRequest()->isXmlHttpRequest()) return;
 
if ($this->getRequest()->isPost()) {
    $form = new Default_Form_ContactForm();
    if ($form->isValid($this->getRequest()->getPost())) {
        $eventManager->trigger('nazwaZdarzenia', $this, array( /* parametry */ ))  
    }
    $this->view->form = $form;
} else {
    throw new Exception('Błędny adres!', 404);
}
[PHP] pobierz, plaintext 

Teraz jeśli chcesz dla jakiegoś kontrolera zmienić zachowanie możesz zrobic tak:

[PHP] pobierz, plaintext 
$events->attach('TwojController', 'nazwaZdarzenia', function($e) {
    $controller = $e->getTarget();
    // zrób coś z czego potrzebujesz, np:
    $controller->jakasMetoda();
});
[PHP] pobierz, plaintext 

Możesz to zrobic w pluginie, bootstrapie gdzie chcesz. Moim zdaniem to jest najlepszy sposób bo możesz nie zmieniając kodu podpinac się w dowolnym miejscu metody.

[mate]

Ciekawe rozwiązanie, dzięki za odopowiedź, na pewno o tym poczytam. Dzięki.

[Lysiur]

popieram ciekawie wyglądająca i nieznana przeze mnie opcja (IMG:style_emoticons/default/smile.gif)