Citat:
Ursprungligen postat av danjel
Ja det var ett dumt exempel, vad jag var ute efter var väl principen att en "model" kan göra flera saker tex maila och spara i db m.m.
När jag menar model menar jag mer ett "lager" , låt säga ett "Service lager" och inte en egentlig model klass.
Ok , om jag spinner vidare på ditt exempel,
Anta att man vill skapa en användare i db och samtidigt skicka ett mail.
(Nu blir ju model ett litet missvisande namn kanske..)
Kod:
//antar detta är en metod som inte ligger i controllern men på ett lämpligt ställe, en separat ControllerFactory klass(?)
public function createModel($name)
{
if($name=='UserService')
{
return new UserService( new UserRepository( new DB() ) , Mailer::getInstance() );
//nu är inte pimple använt
}
}
//och sedan index metoden
public function index(Request $request)
{
$model = $this->createModel('UserService');
$user = new User($request->get('username'), $request->get('userpassword'));
$model->registerUser( $user ); //denna använder repository och mail klassen
)
Är detta korrekt rent principellt? hmm nu är väl kanske inte Service locator använt .. |
Jag har ett par principiella invändningar:
Att ha en if-clause på klassnamnet i controllern blir lätt väldigt bloated, if-listor/switch statements växer ALLTID. En idé är att du dels använder en container för dina dependencies som du skickar in till controllern. Sedan har du i userService t ex en publik lista på vad du vill skicka in t ex array('db', 'mailer') som är keys i din DI container. Det finns givetvis snyggare sätt, men det är ett enkelt sätt att komma runt väldigt mycket if/switch-satser. Se t ex Symfony 2 för ett otroligt smidigt och flexibelt system för det. Jag är dock tveksam till hur decoupled just den komponenten är.
Det andra problemet är ditt sätt att använda Mailer (singleton) och db (intern konfiguration). Felet med både förklaras bättre i ett tal av ZFs lead developer;
https://speakerdeck.com/weierophinne...tiful-software Tror du har video på hans sajt också, men den verkar nere när jag skriver det här.