Jak použít jeden objekt pro všechny akce v presenteru?

Etch napsal(a):

Třeba v BasePresenteru

abstract class BasePresenter extends Presenter{
	protected $osoba;

	public function startup(){
		parent::startup();
		$this->osoba = $this->osobaRepository->find($id);
	}
}

Jasně díky :). Ještě jedna věc – Když provedu toto:

$this->osoba = $this->osobaRepository->searchOsobaById($id);

tak mi to vrátí pole těch osob, ale vždy tam bude jen jedna. Mohu k té jediné osobě pak přistupovat takto:

$this->osoba[0] ;

?

Přesně jak píše CherryBoss, podle názvu metody „searchOsobaById“ se očekává, že bude vrácena pouze jedna osoba nikoliv pole (*Id = identifikátor), tudíž máš problém v tvém repozitáři.

Což mimochodem ti nikdo neporadí, když nevidíme implementaci tvého objektu osobaRepository – vesměs to může být úplně cokoliv :)

davez napsal(a):

Etch napsal(a):

Třeba v BasePresenteru

abstract class BasePresenter extends Presenter{
	protected $osoba;

	public function startup(){
		parent::startup();
		$this->osoba = $this->osobaRepository->find($id);
	}
}

Jasně díky :). Ještě jedna věc – Když provedu toto:

$this->osoba = $this->osobaRepository->searchOsobaById($id);

tak mi to vrátí pole těch osob, ale vždy tam bude jen jedna. Mohu k té jediné osobě pak přistupovat takto:

$this->osoba[0] ;

?

odpověď na tvojí otázku zní: ano, v tomhle presenteru a v jeho potomcích – za předpokladu, že inicializace té proměnné proběhla před jejím použitím.

Nad všemi action visí metoda startup(), která se spustí nejdříve – tam bych to nejspíš směroval

Editoval vitush93 (17. 7. 2013 17:36)

Přepokládám, že jsi dělal podle tohoto tutoriálu https://doc.nette.org/cs/quickstart a spíše by mě zajímala tvá implementace repozitáře, kde máš metodu „find“ správně by měla být snad takto (NDB nepoužívám, ale mělo by to tak jít):

<?php
public function find($id) {
  return $this->getTable()->find($id);
}
?>

Odhadl bych, že ty sis ji dopsal jako:

<?php
public function find($id) {
  return $this->getTable()->where(array("id" => $id));
}
?>

což ti vrátí kolekci, potom tedy bys tam musel zavolat ještě fetch(), abys získal první výsledek, ale zároveň tedy musíš se spoléhat na to, že primární klíč má název „id“, což u abstraktního (nějaký baserepository, z něhož dědí ostatní repozitáře) není ideální. Metoda find, což jsem napsal výše, tak ta si zjistí název PK sama a rovnou vrací jeden záznam.

A co se týče té tvé původní otázka, pak pokud to chceš lazy a chceš tahat osobu jen když bude potřeba, pak je elegantnější mít v tom basepresenteru namísto startup způsobu:

<?php
private $osoba;

protected function getOsoba()
{
   if (!isset($this->osoba)) {
   	$this->osoba = $this->osobaRepository->find($id);
   }
   return $this->osoba;
}
?>

A potom ve zděděných presenterech budeš volat tuto metodu namísto přistupování k property.

Ano to je pravda, leč tedy název metody find může být zavádějící, tak ale get rovněž =) Na druhou stranu find je zažitější termín, má ho Doctrine 1, Doctrine 2, NotORM i NDB :) a vždy vrací již jeden záznam nikoliv kolekci. Ale samozřejmě get bych očekával rovněž tak.

frosty22 napsal(a):

Ano to je pravda, leč tedy název metody find může být zavádějící, tak ale get rovněž =) Na druhou stranu find je zažitější termín, má ho Doctrine 1, Doctrine 2, NotORM i NDB :) a vždy vrací již jeden záznam nikoliv kolekci. Ale samozřejmě get bych očekával rovněž tak.

Doctrine ano, NotORM nevím, ale Ndb používá get a všechno ostatní je jen ukázka v quickstaru, tedy uživatelký kód, nikoliv kód knihovny.

Jinak tu konvenci jsem přebral skutečně z Petrova ORMu a nemám k ní jiný důvod, než že mi vyhovuje. Řešení od @Jan Tvrdík je stejně dobré a jistě by se našly i další. Ale bez nějaké konvence budeš při ladění častěji dumpovat, abys viděl s čím vlastně pracuješ.