Tutoriál – triviální blog

zadáš do prohlížeče URL na kterou chceš něco napsat a začneš psát. :)

https://wiki.nette.org/…ablabladdfgd

Honza Kuchař napsal(a):

Link prozatím hoď na fórum, link na rozcestník přidáme, až to bude hotové. ;)

Jasne, očekávám, že to předtím zkoukne pár lidí a vyladíme to.

Cool jen bych měl pár připomínek…

• Nebylo by od věci dodržovat Nette Coding Standard a místo mezer pro odsazování používat TABulátory.
• obecně je preferováno pro formuláře používat druhou variantu továrničky
• $f jako název proměné pro formulář mě nepřipadá jako šťastná volba
• není použit cizí klíč
• chválím použití {plink} místo {link}
• napopak nechválím {widget} místo {cotrol} (Můžete se také místo makra widget setkat s jeho starším aliasem control. – je lež je to přesně naopak)
• pokud odeslu komentar nedostanu informaci ze je to OK… (chtelo by to flashMessage)

Preferovanější továrnička pro formuláře.

protected function createComponentMyForm($name)
{
	$form = new AppForm($this, $name);
	//...
}

vrtak-cz napsal(a):

Cool jen bych měl pár připomínek…

• Nebylo by od věci dodržovat Nette Coding Standard a místo mezer pro odsazování používat TABulátory.

Pan Grudl je mi vzorem ve všem kromě formátování kódu :). Ale máš pravdu, přdělám. taby se zprasily kopírováním…

• obecně je preferováno pro formuláře používat druhou variantu továrničky

předělám

• $f jako název proměné pro formulář mě nepřipadá jako šťastná volba

předělám

• není použit cizí klíč

omlouvám se, učil mě bastlič :)

• chválím použití {plink} místo {link}

děkuju.

• napopak nechválím {widget} místo {cotrol} (Můžete se také místo makra widget setkat s jeho starším aliasem control. – je lež je to přesně naopak)

opravím, jak stárnu tak blbnu

• pokud odeslu komentar nedostanu informaci ze je to OK… (chtelo by to flashMessage)

to je otázka… na jednu stranu možná stojí za to flash zprávy předvést, na straně druhé je to u takové ukázky skoro zbytečné a dělá to příklad už tak extrémně složitý ještě složitějším
>

Preferovanější továrnička pro formuláře.

protected function createComponentMyForm(AppData $form)
{
	$form = new AppForm($this, $name);
	//...
}

protected function createComponentMyForm(AppData $form)
{
      $form = new AppForm($this, $name);
      //...
}

nestačí to zapsat?

protected function createComponentMyForm()
{
      $form = new AppForm();
      //...
}

• V šablonách presenterů mi přijde vhodnější link namísto plink.

To si s vrtakem prosím vyjasněte mezi sebou :D. Ale vzhledem k tomu, že odkazuji na samostatnou akci presenteru, je plink to pravé, ne?

Už jsme se kvuli tomu pohádali… A zjistili že se neshodneme

Můžeš mi dát link? Zajímá mě to (vzhledem k tomu, že si celkem stojím za plink) a nemůžu to najít…

Jinak syntaxe opravena, tak to prosím zkoukni.
Ještě přidám:

• demo
• chybějící screen
• zdroják
• testovací data
• opravím strukturu db (asi je čas naučit se sql :( )

• je potřeba rozhodnout o té flash zprávě. mě to napadlo už když jsem to psal, jenomže pak jsem si vzpomněl, jak jsem byl v začátcích sám úplně vykulený z toho, jak to všechno samo od sebe funguje. Protože tohle je snad ještě jednodušší něž quickstart, nejsem si tím pořád jistý…

Vrták: Tu tvojí továrničku jsem nějak nepochopil:

protected function createComponentMyForm(AppData $form)
{
      $form = new AppForm($this, $name);
      //...
}

Asi se ti tam něco smíchalo ;) Navíc fakt stačí ten jednodušší tvar, který víc oku a je míň WTF:

protected function createComponentMyForm()
{
      $form = new AppForm;
      //...
}

setDefaults v továrničce jde už dělat i bez připojení k rodiči, jediné, na co je brzké připojení potřeba, je tuším setAction/getAction, kterou stejně v MVC v 99% nepoužíváš.

Také jsem něco poopravoval. Mrni na to a případně vrať, co se ti nelíbí.

Továrnička

protected function createComponentMyForm(AppData $form)
{
      $form = new AppForm($this, $name);
      //...
}

je nesmysl. Třídu AppData neznám a ta továrnička předává parametr, který hned na prvním řádku přepíšu? He?

Co v Nette funguje, je:

protected function createComponentMyForm($name)
{
      $form = new AppForm($this, $name);
      //...
}

přičemž v $name se předá lcfirst(název metody za createComponent) (tedy myForm).

Ale nevidím v tom smysl, od úpravy formulářů, které to anchorování tak brzo už nepotřebují, není toto brzké připojování potřeba.

Majkl578: Co mi brání mít nějaký foreach v továrničce nebo v action* metodě, stále bez brzkého připojení k rodiči?

Ondřej Mirtes napsal(a):

Továrnička

je nesmysl. Třídu AppData neznám a ta továrnička předává parametr, který hned na prvním řádku přepíšu? He?

Sorry upsal jsem se…

Ale nevidím v tom smysl, od úpravy formulářů, které to anchorování tak brzo už nepotřebují, není toto brzké připojování potřeba.

Pokud je získání výchozích dat náročnější, například dotazem do databáze, můžeme jejich nastavení uzavřít do podmínky if ($form->isSubmitted()) { … }. Tato metoda bude ale fungovat pouze v případě, že formulář již je připojen k rodiči jako komponenta. ($form = new Nette\Application\AppForm($this, $name);)

Jasný no… Já to dělám taky… Já to uvedl jenom jako příklad toho proč připojit formulář hned (je to méně náchylné na potencionální chyby).

Edit: tak jsem to prolít a už snad nemám co vytknout. Jenom snad aby jsi pro příště používal takové to zaškrtávátko „minor edit“ a nespamoval nám changes