Enhancement: Co takhle create<componen­tName>($name)?

jasir napsal(a):

Mě to přijde jako velice pěkný nápad, můžeš sem dát pro inspiraci svojí implementaci createComponent z BasePresenteru? Díky

Předpokládám, že to vypadá nějak takto:

protected function createComponent($name)
{
	$callback = array($this, 'create' . ucfirst($name));

	if (is_callable($callback)) {
		call_user_func($callback, $name)
	} else {
		// Něco jiného
	}
}

Jod napsal(a):

Asi tak nejak by to malo fungovať aj dla mňa. Ja som nato myslel akurát včera, kedže tam robím rovnakú štrúdlu :))) . Ale som si povedal, že ma to tak neobťažuje a aj tak používam napr. na editačný formulár a ten na pridávanie rovnakú metódu s parametrom true/false.

Me to prijde nejake divne… Co to teda umoznuje? Proc pouzivat metodu createComponent($name)? a nevolat rovnou createName1() ?

Takhle to bude asi vůbec nejlepší:

<?php
class BasePresenter extends Presenter
{
	/**
	 * Vytvořit komponentu - zavolá metodu $this->create{Name}
	 * @param string $name jméno komponenty
	 */
	protected function createComponent($name)
	{
		if (String::lower($name) === "component") {
			throw new InvalidArgumentException("Argument name '$name' is not allowed.");
		}

		$callback = array($this, "create" . $name);

		if (is_callable($callback)) {
			$component = call_user_func($callback, $name);

			if ($component instanceof IComponent && $component->getParent() === NULL) {
				$this->addComponent($component, $name);
			}

		} else {
			parent::createComponent($name);
		}
	}

}
?>

Komponenta se může připojit buď:

1. v metodě create{ComponentName}. Dostane k dispozici parametrem název komponenty.
2. může být vrácena a pokud nemá rodiče, tak se automaticky připojí k presenteru v metodě createComponent.

Editoval Honza M. (3. 4. 2009 14:38)

Honza M. napsal(a):

Komponenta se může připojit buď:

1. v metodě create{ComponentName}. Dostane k dispozici parametrem název komponenty.
2. může být vrácena a pokud nemá rodiče, tak se automaticky připojí k presenteru v metodě createComponent

Jojo, takhle mi to přijde velice pěkné. Dovolil jsem si tam přidat ucfirst() a zase odstranit :P

Editoval jasir (3. 4. 2009 14:42)

LM napsal(a):
Co když si zavolám $component->getComponent('component')?

No tak to funguje podle očekávání, stejně jako továrnička.

Editoval jasir (3. 4. 2009 14:46)

Honza M. napsal(a):

jasir napsal(a):
Jojo, takhle mi to přijde velice pěkné. Dovolil jsem si tam přidat ucfirst()

To není potřeba, protože php není case sensitive.

Máš pravdu, nějak jsem žil v domění, že názvy metod jsou case sensitive, stejně jako jména proměnných (která jsou case sensitive). To PHP je ale schizofrení :)

Editoval jasir (3. 4. 2009 14:26)

jasir napsal(a):

No tak to funguje podle očekávání, stejně jako továrnička.

Ee tohle skončí nekonečným cyklem (createComponent volá sama sebe).

Pokud to dobře chápu, tak už ani není třeba předávat metodě create{$name} parametr $name.

Pokud zavolám $component->getComponent('cosi'), výše popsaná funkce protected function createComponent($name) v basePresenteru
bude hledat protected function createCosi($name) kde v $name bude zase ‚cosi‘.
Je to tak?
Pokud mám definovaný jeden formulář $form = new AppForm($this, $name); a na základě jeho jména se rozhoduji, co se s daty má dělat, tak toho výše uvedeným způsobem nedosáhnu, protože když zavolám $component->getComponent('Form'), budou se všechny formuláře jmenovat ‚form‘.
Abych formům mohl přiřadit jméno musím přidat další parametr.

Honza M. napsal(a): Takhle to bude asi vůbec nejlepší

Tak to už nějakou dobu používám a dnes jsem narazil na dost hustou chybu. Jedná se o nesmyslné použití funkce is_callable, která vždycky vrátí TRUE, protože Presenter dědí od Object, který definuje metodu __call a v případě, že je definována metoda __call, tak funkce is_callable vrací vždy TRUE (viz komentář na php.net).

Tzn., že funkci is_callable je nutné nahradit funkcí method_exists :)