Jak spravne vytvaret formulare?

Z mého pohledu na to jdeš velmi dobře. Já osobně mám vlastní třídy nejen pro formuláře, ale i pro obslužné handlery. To mimo jiné velmi zvyšuje znovupoužitelnost kódu, protože mám například nějaký potvrzovací formulář, kterému jen pokaždé nastavím jinou otázku k potvrzení a jiný handler.

Dávat definice a obsluhy formulářu do Presenteru je z mého pohledu nevhodná praktika, protože to do Presenteru vůbec nepatří a stávají se pak z nich 500+ řádkové třídy, ve kterých se je těžké vůbec vyznat.

Pokud si vyseparuješ samostatné třídy pro definice formulářů (a třeba i pro obsluhu formulářů), je už jenom na Tobě, jak je provážeš dědičností/kompozicí. Uričtě je vhodné neopakovat kód a společné prvky formulářů seskupit do nějaké vlastní třídy. K tomu se dá použít dědičnost (třeba nějaký BaseForm) nebo kompozice. Kompozice je zde flexibilnější, dědičnost asi zase používanější, to už je na Tobě…

Dědičnost

class PersonForm extends Nette\Application\UI\Form
{
        public function __construct()
        {
                parent::__construct();

                $this->addText('name', 'Jméno');
                $this->addSubmit('submit', 'Odeslat');
        }
}

class PersonWithAddressForm extends PersonForm
{
        public function __construct()
        {
                parent::__construct();

                $this->addText('city', 'Město');
                $this->addText('street', 'Ulice');
        }
}

Kompozice

class PersonContainer extends Nette\Forms\Container
{
        public function __construct()
        {
                parent::__construct();

                $this->addText('name', 'Jméno');
                $this->addText('surname', 'Příjmení');
        }
}

class AddressContainer extends Nette\Forms\Container
{
        public function __construct()
        {
                parent::__construct();

                $this->addText('city', 'Město');
                $this->addText('street', 'Ulice');
        }
}


class PersonWithAddressesForm extends Nette\Application\UI\Form
{
        public function __construct()
        {
                parent::__construct();

                $this['user'] = new PersonContainer();
                $this['user']['address'] = new AddressContainer();
                $this['user']['correspondence'] = new AddressContainer();
                $this['user']['billing'] = new AddressContainer();

                $this->addSubmit('submit', 'Odeslat');
        }
}

Co myslíš, že je lepší? :)

Editoval HosipLan (30. 11. 2011 14:45)

@HosipLan: Díky za doplnění :).

@tomasnikl: Kompozice je skoro všude lepší, jen ji řada vývojářů nemá prostě „moc zažitou“ (a i v důsledku toho se dědičnost dost nadužívá). Kompozice zpravidla vede ke znovupoužitelnějšímu kódu, protože nějakou funkcionalitu můžeš zakomponovat kamkoliv. U dědičnosti musíš pak sáhnout k dědění od nějaké třídy, která třeba ale k té rodičovské vůbec nemá vztah „is a“.

Nejlepší je používat dědičnost skutečně jenom pro případy typu Škoda Octavia extends Auto a pochopitelně také tam, kde Tě k tomu třeba nějaký framework vyloženě nutí.

Ještě bych k tomu dodal, že i některé návrhové vzory, které tak trochu specifickým způsobem využívají (rozuměj zneužívají) dědičnost, jsou dnes některými programátory brány s rezervou a často je doporučují nahradit řešením založeným na kompozici. Já se řadím mezi ně. :)

Editoval Tharos (30. 11. 2011 15:15)

@pawouk: Tvé addTime má ovšem zásadní nevýhodu. Nezavoláš to totiž ve form containeru :)

Správně tedy

Nette\Forms\Container::extensionMethod('addTime', function ($_this, $name, $arg1, ...) {
        return $_this[$name] = new TimeControl($arg1, ...);
});

@HosipLan Souhlas, to by asi bylo lepší.

jeste jeden takovy dotaz, zdrojaky muzu pouzit z meho prvniho prispevku, takze:

<?php

use Nette\Application\UI,
    Nette\ComponentModel\IContainer;

class TestForm extends UI\Form
{
    public function __construct(IContainer $parent = NULL, $name = NULL)
    {
        parent::__construct($parent, $name);
        $this->addText('name', 'Jméno:');
        $this->addSubmit('submit', 'Odeslat');
    }

        public function testSubmitted(UI\Form $form)
        {
                if ($form->isSuccess()) {
                  echo 'jo jo, formular byl odeslan';
                }
        }
}

me vyhodi chybu:

Call to undefined method FrontModule\TipAddForm::isSuccess().

Jak tento problem prosim opravit?

HosipLan napsal(a):

@pawouk: Tvé addTime má ovšem zásadní nevýhodu. Nezavoláš to totiž ve form containeru :)

Správně tedy

Nette\Forms\Container::extensionMethod('addTime', function ($_this, $name, $arg1, ...) {
      return $_this[$name] = new TimeControl($arg1, ...);
});

Zajímalo by mě, kde přesně by jsi toto rozšířšní definoval?

HosipLan napsal(a):

Já bych ho definoval přesně v souboru, ve kterém mám vlastního potomka Form, takže v momentě, kdy s ním budu pracovat, načtou se mi vždy i rozšíření. No a nebo v app/boostrap.php

Mě právě přijde ten bootstrap špatně, když máš pak 20 různých rozšíření v různých třídách, tak budeš v kařdém voání includovat 20 souboů ikdyž je nevyužiješ… Lepší mi tedy přijde první možnost, ale kde? To jako v __constructoru?

Dobrý den, vzhledem k tomu že hodně moc oprašuji znalost php a rozhodl jsem si to „ulehčit“ s Nette :)
tak jsou mi některé věci trošku nejasné. Formuláře už zvladam, vkladat přez ně a tak, ale když už je mám dělat, chci se to učit rovnou co nejvhodněji.
Kompozice se mi líbí ale krapet tápu v tom jak to v této podobě oživit. Formulář mám v samostatném souboru a potřeboval bych poradit jak jej zavolat z presenteru.

HosipLan napsal(a):

Kompozice

class PersonContainer extends Nette\Forms\Container
{
      public function __construct()

Editoval ruppy (20. 2. 17:06)

HosipLan napsal(a):

Dědičnost

class PersonForm extends Nette\Application\UI\Form
{
      public function __construct()
      {
              parent::__construct();

              $this->addText('name', 'Jméno');
              $this->addSubmit('submit', 'Odeslat');
      }
}

class PersonWithAddressForm extends PersonForm
{
      public function __construct()
      {
              parent::__construct();

              $this->addText('city', 'Město');
              $this->addText('street', 'Ulice');
      }
}

Kompozice

class PersonContainer extends Nette\Forms\Container
{
      public function __construct()
      {
              parent::__construct();

              $this->addText('name', 'Jméno');
              $this->addText('surname', 'Příjmení');
      }
}

class AddressContainer extends Nette\Forms\Container
{
      public function __construct()
      {
              parent::__construct();

              $this->addText('city', 'Město');
              $this->addText('street', 'Ulice');
      }
}


class PersonWithAddressesForm extends Nette\Application\UI\Form
{
      public function __construct()
      {
              parent::__construct();

              $this['user'] = new PersonContainer();
              $this['user']['address'] = new AddressContainer();
              $this['user']['correspondence'] = new AddressContainer();
              $this['user']['billing'] = new AddressContainer();

              $this->addSubmit('submit', 'Odeslat');
      }
}

Co myslíš, že je lepší? :)

Moc pěkný

HosipLanova ukázka by měla jít do FAQ ne?

Rád bych upozornil na to, že pokud chci přiřadit kompozicí container do nějaké Group, musím nejdříve ve formu vytvořit Group a poté ji předat containeru:

$phone = $this->addGroup('phone');
$this['phone'] = new PhoneContainer(…, $phone, …);

který si musí tuto Group nastavit jako současnou

public function __construct(…, \Nette\Forms\ControlGroup $group = null, …) {
        if (!is_null($group)) {
            $this->setCurrentGroup($group);
        }
    …
}

Je nějaká možnost, jak to vyřešit?

Nejspíše to bude způsobeno tím, že v containeru nelze zavolat parent::__construct() s parametry a tím pádem se okamžitě nepřipojí k formu. Teda pokud tomu správně rozumím…

Editoval PavelJurasek (26. 2. 12:34)