Comments on: Fat model, skinny controller – przykład http://blog.grzegorzpawlik.com/2010/05/fat-model-skinny-controller-przyklad/ Doświadczenie, to coś, co zdobywamy tuż po chwili w której było nam potrzebne ... Tue, 07 Feb 2012 10:09:30 +0000 hourly 1 http://wordpress.org/?v=3.2.1 By: Greg http://blog.grzegorzpawlik.com/2010/05/fat-model-skinny-controller-przyklad/comment-page-1/#comment-2524 Greg Wed, 19 Oct 2011 17:00:12 +0000 http://blog.grzegorzpawlik.com/?p=975#comment-2524 Myślę, że Twoje rozwiązanie jest dobre. Jednak argument o ilości nie jest przekonujący. Gdybyś napisał, że logika powinna być w kontrolerze, ponieważ na podstawie request'a ($params) decyduje o tym skąd pobrać dane - zgodziłbym się z Tobą w 100%. W takim ujęciu - kontroler faktycznie robi to co do niego należy (czyli steruje przepływem na podstawie otrzymanych żądań). Myślę, że Twoje rozwiązanie jest dobre. Jednak argument o ilości nie jest przekonujący. Gdybyś napisał, że logika powinna być w kontrolerze, ponieważ na podstawie request’a ($params) decyduje o tym skąd pobrać dane – zgodziłbym się z Tobą w 100%. W takim ujęciu – kontroler faktycznie robi to co do niego należy (czyli steruje przepływem na podstawie otrzymanych żądań).

]]> By: s http://blog.grzegorzpawlik.com/2010/05/fat-model-skinny-controller-przyklad/comment-page-1/#comment-2522 s Wed, 19 Oct 2011 15:59:37 +0000 http://blog.grzegorzpawlik.com/?p=975#comment-2522 To nie jest zbyt dobry przykład użycia grubego modelu. Wzorzec ten nie polega na przekazaniu do metody modelu surowej tablicy z parametrami. Jeśli metoda add będzie użyta jedynie w tym kontrolerze to powinna zostać metodą kontrolera. Kod w kontrolerze powinien wyglądać tak: if(isset($params["cost_id")) { $this->Cost->getDataForCost($params["cost_id"]); return $some_array; } if(isset($params["entry_id")) { $this->Cost->getDataForEntry($params["entry_id"]); } return array(); To nie jest zbyt dobry przykład użycia grubego modelu. Wzorzec ten nie polega na przekazaniu do metody modelu surowej tablicy z parametrami. Jeśli metoda add będzie użyta jedynie w tym kontrolerze to powinna zostać metodą kontrolera.

Kod w kontrolerze powinien wyglądać tak:
if(isset($params["cost_id")) {
$this->Cost->getDataForCost($params["cost_id"]);
return $some_array;
}
if(isset($params["entry_id")) {
$this->Cost->getDataForEntry($params["entry_id"]);
}
return array();

]]>