[2010–01–10] přepisování createComponent()

Já bych udělal něco s tou velikostí písmen – vím, že se to už jednou měnilo, ale createComponentComponentName + {control componentName} má hodně velký WTF faktor a lidi v tom neustále chybují. Navíc to jde proti tomu, kdy názvy tříd začínají velkým písmenem – název komponenty by podle měl také začínat, přecijen je to něco „víc“ než jen nějaká proměnná nebo členská metoda.

Neni to takové divné? Když už, tak case insensitive celé.

Jsem stejného názoru jako Petr Motejlek.

Jan Tvrdík napsal(a):
Ještě by to chtělo odstranit pokud vím nikým nereportovaný bug, kdy nejde odstranit (removeComponent) komponenta tvořená továrničkou.

Můžeš nahodit nějaký příklad?

David Grudl napsal(a): Můžeš nahodit nějaký příklad?

	public function renderDefault()
	{
		Debug::dump(count($this->getComponents())); // 0

		//$testComponent = new AppForm($this, 'test2'); // OK, není použita továrnička
		$testComponent = $this->getComponent('test');   // Používá továrničku, nelze odebrat.

		Debug::dump(count($this->getComponents())); // 1

		$this->removeComponent($testComponent);

		Debug::dump(count($this->getComponents())); // 1

		$this->terminate();
	}

	protected function createComponentTest()
	{
		echo 'Volána metoda: ' . __METHOD__;
		return new AppForm();
	}

Editoval Jan Tvrdík (20. 1. 2010 18:18)

Téma sensitive a insensitive se už probíralo mnohokrát a ani nevím, k čemu se vždy došlo ;)

case-sensitive kromě prvního písmene, které by bylo case-insensitive

To je ještě míň intuitivnější, než teď.

Kdo by chtěl šmarjá mít dvě komponenty, které by se lišily akorát velikostí písmen?

I když s tím malým písmenem na začátku (jak to funguje teď) to možná není zas až takový průšvih, renderDefault vs. default action je stejná situace a lidi to umí používat…

Editoval Ondřej Mirtes (20. 1. 2010 20:47)

Buď to dát case-insensitive nebo nechat case-sensitive, všechno ostatní je zm[ae]tek.

Implementace je vcelku snadná a ale může to mít vedlejší efekty:

• programátor musí v metodě createComponent($name) také počítat s nerozlišováním velikosti písmen
• metoda getComponents() bude vracet klíče lowercased, takže třeba iterátory foreach(getComponents() as $name => $control) chce nahradit za foreach(getComponents() as $control) { $name = $control->name; ... odvolávám, tohle samozřejmě může getComponents() řešit.

Ale stále zůstává otázka: fakt to chceme? Proměnné a klíče polí jsou stejně case-sensitive.

David Grudl napsal(a):

Buď to dát case-insensitive nebo nechat case-sensitive, všechno ostatní je zm[ae]tek.

Implementace je vcelku snadná a ale může to mít vedlejší efekty:

• programátor musí v metodě createComponent($name) také počítat s nerozlišováním velikosti písmen
• metoda getComponents() bude vracet klíče lowercased, takže třeba iterátory foreach(getComponents() as $name => $control) chce nahradit za foreach(getComponents() as $control) { $name = $control->name; ...

Ale stále zůstává otázka: fakt to chceme? Proměnné a klíče polí jsou stejně case-sensitive.

My chceme, aby jméno komponenty a jméno komponenty ve stromu komponent byla jiná hodnota?

Osobně mi nevadí současný stav. I když kdysi jsem žil v domnění, že hodnoty jsou case-insensitive a taky jsem byl spokojen. Při zjištění pravdy jsem byl velmi překvapen :D

Souhlasím s Ondrou a s Whitkem – řídím se pravidlem, že komponenty pojmenovávám vždy podle camel caps. Takže kde píšu jen název, bude nazevKomponenty, u metody bude logicky createComponentNazevKomponenty. It's so simple.

Mě to nepřijde podobný, mě to přijde úplně stejný ;).