nejlepší konstrukce pro formulář „výběr dat->úprava dat->uložení dat a zobrazení úspěchu“

v podstatě mi na tom nevoní fakt, že dvakrát budu tahat jeden a ten samý řádek z databáze a tříkrát budu dělat stejný objekt… poprvé při načtení dat do selectu a podruhé při načtení dat pro edit a třetí vytvoření objektu bude při převedení dat z formuláře pro uložení… a to bych vlastně tahal třikrát z db, kdybych si neudělal v presenteru tu pomocnou proměnnou…

já vím, že časové to je nula k nule, ale není to „čisté“, není to „chytré“… prostě mi to nesedí a nevím jak jinak to řešit…

nechápu proč mi nefungovalo použití statické proměnné presenteru…

Editoval Ofi (8. 9. 2010 23:33)

mozna by to chtelo kousek kodu:-)

keš je v nette souborová nemýlím-li se, takže je to buď db nebo file, sessions mě napadly také, k jejich vyzkoušení jsem se ještě nedostal…

já si ve všech případech tahám všechno, protože to má 4 sloupce a z tech bych vždycky tahal 2–4…
takhle vypadá příslušný presenter..

class Admin_DefaultPresenter extends Admin_SecuredPresenter {

    private $contentManager = NULL;
    private $activeItem = NULL;

    public function actionLogout()  {
        Environment::getUser()->logout();
        $this->flashMessage('Právě jste sa odlásili z administrace.');
        $this->redirect('Auth:login');
    }

    public function actionDescrtiption() {
    }

    public function renderDescription($itemSelected = NULL) {
        $this->activeItem = $this->model->getDescription($itemSelected);
        $this->template->description = $this->activeItem;

        $this->template->itemSelected = $itemSelected;
    }

    public function createComponentEditDescriptionForm() {
        $form = new AppForm();

        $form->addHidden("id");
        $form->addHidden("menuItem");
        $form->addHidden("seoUrl");
        $form->addTextArea("description", "Popis: ");
        $form->addSubmit("send", "Změnit popis");

        if(is_object($this->activeItem)) {
        $form->setValues(array('description'=>$this->activeItem->description,
                                'id'=>$this->activeItem->id,
                                'menuItem'=>$this->activeItem->menuItem,
                                'seoUrl'=>$this->activeItem->seoUrl
                        ));
        }
        $form->onSubmit[] = callback($this, 'processEditDescriptionForm');

        return $form;
    }

    public function processEditDescriptionForm(Appform $form) {
        if($form['send']->isSubmittedBy()) {

            $values = $form->getValues();

            $properties = array();
            foreach($values as $key=>$val) {
                $properties[$key]=$val;
            }

            $updatedItem = new Description($properties);

            $updatedItem->save();

            $this->flashMessage('Popis byl úspěšně změněn.');
            $this->redirect('Default:description');
        }
    }

    public function createComponentSelectDescription() {
        $form = new AppForm();

        $descriptions = $this->model->getDescriptions();
        $items = array();

        foreach ($descriptions as $d) {
            $items[$d->seoUrl] = $d->menuItem;
        }

        $form->addSelect('description', 'Položka: ', $items);
        $form->addSubmit('send', 'Vybrat');

        $form->onSubmit[] = callback($this, 'processSelectDescription');
        return $form;
    }

    public function processSelectDescription($form) {
        if($form['send']->isSubmittedBy()) {

            $values = $form->getValues();

            $this->redirect('description', $values['description']);
        }
    }

    public function getModel() {
        if(!isset($this->contentManager))
            $this->contentManager = new ContentManager;

        return $this->contentManager;
    }
}

a takhle příslušná šablona:

{block content}
<h3>Vyberte, pro kterou položku si přejete změnit popis:</h3>
{widget selectDescription}
{if isset($itemSelected) && $itemSelected != NULL}
<h3>
    Úprava popisu:
</h3>
{widget editDescriptionForm}
{/if}
{/block}

čtení flash message mám v @layout takže o to se v téhle šabloně nestarám..

dnes dopoledne jsem refaktoroval frontend a:

Ani napsal(a):

Přijde mi to celý zbytečně komplikovaně řešený.

• Za cenu úspory dotazu do db, tam děláš různý iterace přes celý pole (ta v processEditDescriptionForm mi přijde obzvlášť nelogická).

už není – nevím proč jsem jí tam dával :D

• V těch process metodách nemusíš dělat $form[‚send‘]->isSubmittedBy(), když to máš jako appform s jedním tlačítkem (raději koukni do dokumentace).

to jsem nevěděl… čili to ještě změním

• Stejně tak předávání těch hidden polí je pěkně kostrbaté a kromě id, nejspíš zbytečné.

ano to už tam také nemám krom toho id…

• V renderDescription předáváš do šablony ten objekt, ale v ní ho pak nepoužíváš.

také už je minulost, všiml jsem si toho…

nakonec jsem to zjednodušil, ale i tak.. teď mě to trápí, protože až přijde nějaký projekt kde si budu muset znovu a znovu sahat pro daleko větí objem dat tak si s tím nebudu vědět rady…