Нужна критика класса

[scb]

http://pastebin.com/ZnBhw2hM

Буду рад любым советам. Заранее спасибо.

 

[AmdY]

Ты вроже уже используешь DI, так избався ещё от жёсткой зависимости $user = new User(); и т.д., используй phpdoc
Ну и тесты, а то так можно только кодестайл проверить.

 

[scb]

Как избавиться: отнаследоваться? Или public function __construct(User $user)?

С тестами беда... буду пробовать...

Спасибо.

Есть еще одна беда в контроллере:

        $user = new User();
        $auth = new Auth($user);
        if($auth->getAuthResult()){
            $_layout = Zend_Layout::getMvcInstance();
            $_layout->info = $auth->getSession();
            $userModel = new BackofficeUser();
            $userModel->login($_layout->info["username"],$_layout->info["password"]);
        }else{
            //$this->_redirect('/index/');
        }

Ухожу в бесконечный редирект. Ошибка где-то в алгоритме отловить пока не смог...

 

[AmdY]

Как избавиться: отнаследоваться? Или public function __construct(User $user)?

Можно инъекцией в конструктор, yj hfp e;t bcgjkmpetim DI
>>$configuration = App_DI_Container::get('ConfigObject');
то логично будет
$user = App_DI_Container::get('User');

По второму вопросу, нужно проверять твою локацию перед редиректом. Типа этого
elseif ($ершы->getRequest()->getController() != 'index' ....) { $this->_redirect('/index/'); }

 

[scb]

А да точно, спасибо. Пойду покапаю.