PHP Stilovi pisanja aplikacija (Best design practices)

[Ilija Studen]

Ne vidim ništa loše u tome da se u view-u pozivaju metodi modela, dokle god si ti pozivi tiču izvlačenja i prezentovanja podataka. Ako ih koristiš za nešto drugo (tipa osvežavanje sesije, modifikacija ili brisanje vrednosti itd) tome nije mesto u view-u (duh!).

U gornjem primeru getUsername() može da bude prost accessor, a opet iza njega može da se krije nešto mnogo složenije. Koga je briga dokle god dobijamo očekivani rezultat.

Stvari bi bile izuzetno loše kada bi u viewu imao nešto tipa:

PHP kôd:

{$user->setUsername('new one')};
{if $user->save()}
  <p>Saved!</p>
{else}
  <p>Failed to save!</p>
{/if} 


[dee]

ma to je jasno

[iako, tnx na volji da se bude jasan, svakako]

[ppavlovic]

Dizajneri ZendFramework-a su malo zeznuli stvar oko MVC-a, pa su na kraju morali da isprave model. Detalji su na
http://devzone.zend.com/article/2072...e-ViewRenderer

Controller - setuje podatke dobijene iz Model
View - radi samo "glupu" prezentaciju

[robi-bobi]

a evo i necega sto nisam mogao da nadjem kad mi je trebalo
cake lifecycle
http://www.cakecollab.org/lifecycle.png

[kaspar]

Originalno napisao robi-bobi

a evo i necega sto nisam mogao da nadjem kad mi je trebalo
cake lifecycle
http://www.cakecollab.org/lifecycle.png

Uf, a meni bas sad treba


Inace slazem se sa Ilijom u pogledu View dela MVC strukture. Dok god ne menja stanje aplikacije i samo prezentuje podatke nema nikakvih problema sa pozivima bilo direktno modela bilo akcija kontrolera koji na neki nacin komuniciraju sa modelom.

[dee]

Originalno napisao Ilija Studen

Kontroler je spona između viewova i modela. Ako imaš $smarty->assign() u modelu nešto ne radiš kako treba

Evo ga jedan konkretan primer gde mislim da je odvajanje dobro urađeno:

PHP kôd:

class SomeController {

  function index() {
    $this->smarty->assign('users', Users::findAll());
  }

} 


Gde kasnije u templateu imaš nešto tipa:

PHP kôd:

{if $users}
<ul>
{foreach from=$users item=user}
  <li>{$user->getUsername()|clean}</li>
{/foreach}
</ul>
{else}
<p>No users here</p>
{/if} 


Ilija, kako ti ugrubo izgleda ovaj objekt kojeg zoves sa Users::findAll() ? i sta ti on vraca? array objekata, referenci na objekte, nesto trece?

[Ilija Studen]

Nije objekat već klasa sa par statičkih metoda. Sama klasa se nikada ne instancira jer potrebe za instancom nema.

Funkcija findAll() vraća niz objekata. S druge strane findById() vraća samo objekat koji odgovara prosleđenom ID-ju ili NULL ako ga ne nađe.