Best Practice: Mají mít komponenty parenta? Být připojeny k presenteru?

Z diskuze zde mi vyplynula jedna otázka: https://forum.nette.org/…at-formulare

Všiml jsem si (tu na foru, v docách), že strašmě mockrát se dělají komponenty bez parenta. Je to správně? Důsledkem totiž je, že komponenta není připojena k presenteru a to přináší různé problémy.

Mám zvyk, že všechny presentery v aplikaci dědí o jednoho BasePresenteru, všechny komponenty od BaseControlu a všechny formuláře dědí nebo jsou přímo instancí nějuakého mého App\Formu. Navíc důsledně všechny komponenty a formuláře připojují k presenteru.

Je to správně? Proč případně nění a jaký je best practise?

Zde je ukázka kódu jak ± vypadá můj BaseControl.

abstract class BaseControl extends Control {

    /** @var Translator  */
    public $translator;

    public function __construct(IContainer $parent = NULL, $name = NULL) {
        if (!$parent) {
            throw new \Exception("Component must have \$parent.", NULL);
        }
        if (!$parent instanceof \App\Controls\BaseControl AND !$parent instanceof BasePresenter) {
            throw new \Exception("\$parent must be instance of \App\Controls\BaseControl or \BasePresenter.");
        }
        $this->translator = $parent->translator;
        parent::__construct($parent, $name);
    }

    public function setLocalTemplateFile($name) {
        $this->template->setFile(dirname($this->reflection->getFileName()) . '/' . $name);
    }

    protected function registerHelper($helper, $template) {
        if (method_exists ($helper , 'injectTranslator')) {
            $helper->injectTranslator($this->translator);
        }
        $template->getLatte()->addFilter(NULL, array($helper, 'loader'));
    }

    public function createTemplate($class = NULL) {
        $template = parent::createTemplate($class);
        $template->setTranslator($this->translator);

        // HELPERS
        $this->registerHelper(new Helpers(), $template);
        $this->registerHelper(new \App\Helpers\Helpers(), $template);
        return $template;
    }

    /**
     * @param bool $need
     * @return \BasePresenter
     */
    public function getPresenter($need = TRUE) {
        return parent::getPresenter($need);
    }

    /**
     * NOTE: render() can't be declared here, breakes child controls with render with arguments
     */

}

Editoval Achse (16. 6. 2014 13:26)