Fat controller skinny model vs reverse

[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.

[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.