Как написать это условие по нормальному?

Question

[vladislav997]

Есть такой код (пример прикрепляю), он хоть и рабочий, но написан коряво, подскажите, как его переписать по нормальному?
т.е. если незалогиненый юзер попытался зайти по какому-то урлу, по которому у него нет доступа, то его должно средиректить на логин-форму, но при этом если он зашел на signup, то он там должен остаться (логично, что сначала надо зарегаться)

if($_SERVER['REQUEST_URI'] != '/login' || $_SERVER['REQUEST_URI'] != '/signup' && empty($_SESSION['logged_user']))
	    {
	        if($_SERVER['REQUEST_URI'] == '/signup')
	        {
	            
	        }
	        elseif($_SERVER['REQUEST_URI'] == '/login')
	        {
	            
	        }
	        else
	        {
	            header("Location: /login");
	        }
	    }

Comments

[Never Ever]

зачем так усложнять ?
Не зареган ну и пусть бегает по двум страничкам signup и login

[vladislav997]

Never Ever, потому что незареганый без этих условий сможет бегать по всем урлам которые есть, а должен только по логину и сигнапу

Answer 1

[vdem]

if (empty($_SESSION['logged_user']) && !in_array($_SERVER['REQUEST_URI'], ['/login', '/signup'])) {
    header('Location: /login');
}

Answer 2

[Lander]

Если по нормальному, что через Router и Middleware.

Comments

[vladislav997]

без фанатизма я имею ввиду

[kafkiansky]

Вы бы ему еще на симфони предложили переписать этот кусок. Понятное дело, что код говно, но это говно дает понять, что тс далеко от терминов роутер и миддлвары.

[kafkiansky]

vladislav997, это не фанатизм, это нормальный код.

[Lander]

kafkiansky, Автор поставил вопрос "по нормальному". Если бы он написал "по-короче" или "по-проще" это был бы другой вопрос и ответ был бы другим.