Толстый контроллер. Куда вынести логику?

Question

[alestro]

В данный момент имею следующее:

class Login{
	public function index(){
		App::redirect_if(Register::auth()->is_authorized(),'/');
		if( Register::request()->has('registration','post')  ){
			$validator=new Validator();
			$validator->make( Register::request()->only('post','email'),[ 'email'=> ['email','required','unique:users:email'] ] );
			if( $validator->messages()->isEmpty() ){
				$LModel=new LModel( new UMapper( 'users', 'user' ) );
				$LModel->registration( array_merge(Register::request()->only('post','email'),['registration_date'=>time()]) );
			}
			else{
				foreach($validator->messages()->all() as $message){
					echo '<li>'.$message.'</li>';
				}
			}
		}
		elseif( Register::request()->has('auth','post') ){
			$validator=new Validator();
			$validator->make( Register::request()->only('post','email','password'),[ 'email'=> ['email'], 'password'=>['regex:'.Validator::H_PASSWORD_PATTERN] ] );
			
			if($validator->messages()->isEmpty()){
				$LModel=new LModel( new UMapper( 'users', 'user') );
				!$LModel->auth()?:App::redirect('/');
				echo 'Неверный логин или пароль';
			}
			else{
				echo 'Неверный email или пароль';
			}
		}
		View::render('login');
}

Как видно контроллер очень толстый и выполняет не те функции которые должен. Как можно решить это?
Куда следует вынести логику, в соответствующую модель или добавить события и переложить часть реализации на них?
В данном случае это один из самых маленьких контроллеров.

Answer 1

[trevoga_su]

а почему регистрация и авторизация не то, что в одном контроллере, а в одном action?

Comments

[DireX]

на сколько я понял по коду это наверное не регистрация... хотя чем черт не шутит :-)

[alestro]

Тут дело в том, что на странице /login выводиться форма регистрации или форма авторизации ну соответственно отлавливаю какая из форм отправлена и обрабатываю в одном действии

[D']

alestro: а почему формы логина\регистрации шлют на один и тот же роут?

[alestro]

D' Normalization: Подбирал верстку для екомерса, во всех вариантах авторизация и регистрация были совмещены из этого и имею то, что имею.

[D']

alestro: я так и не понял, почему формы шлют на один роут? Как наличие обоих форм на странице мешает указать им разный action?

[alestro]

Об этом я и не думал, благодарю.

[trevoga_su]

alestro: разные экшен и шли аяксом данные
экшены будут максимально тонкие, а index по факту только шаблон будет выдавать с полями ввода

[AlikDex]

alestro: т.е. вы пишете логику приложения подстраиваясь под версту? ;D

Answer 2

[DireX]

У Вас метод разбит на два логических блока
Register::request()->has('auth','post')
и
Register::auth()->is_authorized()
Вот Вам уже два метода. Плюс у Вас присутствует html код, что не есть хорошо, от него лучше избавится в вид. Я смотрю Вы в View::render не чего кроме файла вида не передаете, так передайте туда массив $message и там уже его разбирайте вашим шаблонизатором или родным php.
Что касается выборки, перенести или в модель или в отдельный метод

Comments

[alestro]

Register::auth()->is_authorized() в моем случае я достаю из реестра приложения объект отвечающий за аунтефикацию пользователя и в случае, если пользователь авторизован перекидываю его в корень

Answer 3

[Vasiliy Os]

На разные routes раскидать

Comments

[Vasiliy Os]

И `Login` -> `AuthController` переименовать, а методы раскидать в `auth` и `register`