Fat controller skinny model vs reverse

[jablan]

Originalno napisao tasmaniski

nije ni blizu da je model tezi od kontrolera ...

Mislim da ovde malo mešaš babe i žabe, tj. poistovećuješ "debljinu" sa brojem linija. To što kontroler ima više linija koda od modela ne znači da je "deblji". Pored toga, različiti frejmvorci (i različiti jezici) pružaju mogućnost da se neki kod piše kraće, imaš mogućnost da koristiš razne helper metode itd, tako da isti taj tvoj primer kontrolera:

    public function indexAction() {
           $month = $this->_request->getParam('month');
           $year = $this->_request->getParam('year');
           $day = $this->_request->getParam('day');
           
           // obrade se podaci i kreira ispravan format datuma  $date
           // eventualno se po potrebi pozovu jos neke funkcije i odrade upiti

           $event_model = new Application_Model_Event();
           $this->view->events = $event_model->getEvents($date);
    }

U nekom drugom hipotetičkom (khm) frejmvorku može glasiti:

def index
  date = Time.parse(params[:date])
  @events = Event.get_approved(date)
end

itd.

To takođe zavisi i od toga kako si nazvao i organizovao form elemente, kako si osmislio rute itd, dosta tog "plumbinga" u kontroleru načelno može da se izbegne.

[bluesman]

Može neko objašnjenje zašto nikako POST u model? Gde vi radite preveru i sanitizaciju podataka iz POST-a, u kontroleru?

[jablan]

Ne znam šta ti je to "sanitizacija". Konvertovanje parametara iz stringova u štagod treba modelu (npr u datum) obavljam u kontroleru. Logičku validaciju podataka vrši model.

[tasmaniski]

Ja uglavnom 90% validacije obavljam u kontroleru, jer ako neki podatak nije validan i treba da se izbaci greska sto bi uopste skripta ulazila u metodu iz modela.

[tasmaniski]

Sto se tice slanja $_POST u model, radio sam na kodu gde je slucaj ovakva:

Polja na formi se nazovu isto kao i polja u bazi, i tako da kad se post prosledi modelu
moze odmah da se uradi insert(selekt i dr.) jer je niz koji je key => value vec u postu.

Sve je secure tu nema sta, al jednostavno nisam za to, jer potencjalni napadac moze da izvuce koliko toliko semu baze, jeste da ne moze nista al mi se to ne svidja.
Ako neki junior sutradan treba da nesto dodaje/oduzme radice copy-paste i moguce da ce negde da zajebe nesto.
Citljiviji mi je kod ako u kontroleru pripremim post i samo odredjene parametre prosledim u model.

[djipko]

Originalno napisao tasmaniski

Ja uglavnom 90% validacije obavljam u kontroleru, jer ako neki podatak nije validan i treba da se izbaci greska sto bi uopste skripta ulazila u metodu iz modela.

Ja mislim da ovo nije skoro nikad dobro resenje - validacija je usko povezana sa logikom modela (datum moze biti validan za jedan model a nevalidan za drugi itd.) i ovo jednostavno treba da bude enkapsulirano u modelu. (Rails npr. sve validacije stavlja u model i ne pozivaju se eksplicitno nigde, nego postoji konvencija kako se vraca greska, Django ima Form klasu koja ja usko vezana sa modelom i vrsi validacije, i model ih moze imati vise).

Sve ostalo skoro 99% sam siguran vodi ka ruznom kodu i izuzetno smanjenom re-usabilitiju.

Validacija u kontroleru mi ima smisla samo ako se radi o nekoj app-wide vrednosti koja nije konkretno vezana za neki model koji ce kasnije postati record u bazi.

[tasmaniski]

Originalno napisao djipko

datum moze biti validan za jedan model a nevalidan za drugi itd.

Zar ne bi onda bas zbog toga validaciju prepustio kontroleru da ne bi u modelu morao da pises
if(datum1){validacija 1}else{validacija 2}

Originalno napisao djipko

Django ima Form klasu koja ja usko vezana sa modelom i vrsi validacije, i model ih moze imati vise ...

Upravu si kad kazes da je validacija povezana sa modelom ali hendlovanje greske u PHPu nije (u railsu je kako kazes drugacije).

Zend koji ima svoju klasu za forme i validacija toga se radi u kontroleru (koliko se secam tako je bilo i u drugim PHP FW).

[djipko]

Kad kazes za Zend - radi se validacija da li mislis na nesto ovako u kontroleru (fiktivni kod ali ovako se slicno radi u Django-u) :

f = MyModelForm(request['POST'])
f.validate()
if f.is_valid():
    MyModel(f).save()
else:
    #prijavi gresku

Jer ako na to mislis - to je samo pozivanje validacije iz kontrolera i to je okej. 'validate' metoda forme (u ovom slucaju) radi validaciju i ovaj pattern osim ako ne zelis da ga customizujes moze biti i implicitno pozivan.

Ako mislis da code validate metode (u ovom slucaju) treba da pripada kontroleru - ne slazem se... mada mogu da zamislim frejmvork koji je tako koncipiran da validaciju vrsi u kontroleru ali mi se cini kao losiji pristup dizajnu od ovog... bar konceptualno.

Mesanje ova dva pristupa je svakako lose.

[tasmaniski]

Da, bas tako (prvi primer)

Ok, jeste da validaciju vrsi funkcija forme 'validate' koja se samo poziva u kontroleru,
al moja poenta je bila da se validacija podataka radi pre nego sto se posalju u model,
sto bi onda dovelo na vracanje prvobitne teme "fat model skinny c. vs ..."

U svakom slucaju je bolje koristiti ili jedan ili drugi pristup, mesanje nikako.

[djipko]

Originalno napisao tasmaniski

al moja poenta je bila da se validacija podataka radi pre nego sto se posalju u model

Pa ovaj primer je po meni bas obrnuto.

Samo ako zamislimo da forma ne postoji kao koncept i da se validate poziva iz konstruktora modela i ako ne uspe baca exception naprimer (sto je lagani refactoring) - onda cak i eksplicitno pripada modelu a ne kontroleru. Kontroler samo pokusa da napravi model sa POST podacima i javi rezultat.