Best Practise: jak zpracovávat formuláře

Chápu, prostě uděláš N hook-pointů typu: onEventOccured. A pak je (když například nastane vyjímka) pustíš.

Kód by tedy pakvypadal:

class FormComponentX extend Object {

    // array of callbacks
    public $onSignatureFileNotFound;

    public function processFormX(Form $form) {
        if ($form->isValid()) {
            $values = $form->getValues();
                try {
                    $this->injectedService->doY($values, $this->user);
                    $this->flashMessage("Hell Yeah!", 'success');

                } catch (SignatureFileNotFoundException $e) {
                    $this->fireMyEvents($this->onSignatureFileNotFound);

                } // catch ....
               // ... atd...
        }
    }

Ono, pro dosažení této funkčnosti ani není třeba formulář obalovat imho. Rád bych se vyhnul over-engineering tam kde servis takhle mocného řešení vlastně nepotřebuji.

Každopádně díky za tip, ten se mi líbí → jdu to vyzkoušet napsat.

Editoval Achse (16. 6. 2014 10:11)

Yep, ale když píšu nějaký backend, kde všechno je bootstrap a nemusím nic (zhuleně frontendovýho) řešit, vystačím si s pát úravami v rendereru.

Osobně mám takovou konvenci: kde to jenom trochu jde, manuálnímu renderingu se vyhni. Duplikuje to totiž kód (každá změna se pak musí delat na dvou místech).

Co znamená tohle?

	$this->fireMyEvents($this->onSignatureFileNotFound);

onSignatureFileNotFound je callback ale co vrací do fireMyEvents?

@jenicek:

public function createComponentApplyInvoiceForm($name) {
	$form = $this->applyInvoiceFormFactory->create($this, $name, $this->user, $this->timeEntriesToPay, $this->limitRealHours);
	$form->onSuccess[] = function () { /* flash & redirect*/};
	$form->onSignatureFileNotFound[] = function () { /* flash & redirect*/ };
	// ...

fireMyEvents nevrací nic, jenom provede včechny callbacky pro daný hook.

Editoval Achse (16. 6. 2014 10:49)

Díky moc. Tenhle přístup (autocreating podle interfacu) jsem už viděl – du to implementovat :). Ještě – je to už hodně OT založil jsem tedy další questin: https://forum.nette.org/…k-presenteru

@Achse opět jsem zaktualizoval ukázku, nyní se tam předává hypotetická entita $article přímo z presenteru skrze metodu create().

Já to dělám tak, že si nastavím výsledek do samotné továrničky toho formuláře.

Formulář:

<?php

/**
 * @property $comment
 */
class SetCommentForm extends Nette\Object {

	/** @var CommentManager */
	private $commentManager;

	/** @var Comment */
	private $comment;

	public function __construct(CommentManager $commentManager) {
		$this->commentManager = $commentManager;
	}

	public function setComment(Comment $comment) {
		$this->comment = $comment;
	}

	public function getComment() {
		return $this->comment;
	}

	public function create() {
		$form = new Nette\Application\UI\Form;
		$form->addText("text", "Comment:");
		$form->addSubmit("send", $this->comment ? "Edit comment" : "Add comment");

		if($this->comment) {
			$form->onSuccess[] = $this->processEdit;
		} else {
			$form->onSuccess[] = $this->processAdd;
		}

		return $form;
	}

	public function processAdd(Nette\Application\UI\Form $form) {
		$this->comment = $this->commentManager->addComment($form->values->text);
	}

	public function processEdit(Nette\Application\UI\Form $form) {
		$this->comment->text = $form->values->text;
		$this->commentManager->save($this->comment);
	}

}

Presenter:

<?php

class CommentPresenter extends Presenter {

	use Kdyby\Autowired\AutowireProperties;
	use Kdyby\Autowired\AutowireComponentFactories;

	protected function createComponentAddCommentForm(SetCommentForm $factory) {
		$form = $factory->create();
		$form->onSuccess[] = function() use ($factory) {
			$this->flashMessage("Comment #{$factory->comment->id} was added");
			$this->redirect("this");
		}
		return $form;
	}

}

Při odchytávání vyjímek dáváte chybovou odpověď do $onArticleError?

@Achse Určitě je. Hoď prosím kód třeba na gist a odkaž ho tu.

@Achse Ta tvoje továrnička neodpovídá konstruktoru tvé komponenty.

@Achse Ano, a ty máš v configu a v interfejsu uvedený nějaký parametr, který ale v konstruktoru té komponenty není.

On ten argument nemusí být první, lze to napsat i takto:

implement: XXX
  parameters: [limit]
  arguments: [..., %limit%]

Pak můžeš mít v konstruktoru autowired službu jako první. Ale vem si, když budeš mít 5 služeb, které ta komponenta potřebuje :) – takže takové nepsané pravidlo: argumenty co se musí předat v create metodě dávat jako první.