Как сделать переключатель, с помощью оператора switch в php

@Fell-x27, ничего подобного, я всё правильно сделал, и несколько раз на этом сакцентировал внимание в постах:

basename() всё лишнее отрежет. Или хочешь сказать, что можно его обмануть?

 

А, ок, проморгал. Но вообще, если не выставлена setlocale(), там можно через кодировку, потенциально, намутить всякого. Но это будет больше похоже на нейрохирургию с закрытыми глазами со стороны злоумышленника. Если, конечно, он не знает архитектуру твоего проекта.

 

А вот ты разберись, что в нем происходит, и сделай для себя вывод, хорош или нет. Никогда не используй копипасту из интернета, не разобравшись.

 

Люди, ну роутер на switch - это очень плохо. @_ne_scaju_, вот тебе ещё один способ убрать switch, на этот раз даже у @Fell-x27 не возникнет вопросов по безопасности (хотя и в прошлый раз, если бы @Fell-x27 сразу заметил basename, у него бы вопросов не возникло), и никакая слепая игра с кодировками не поможет взломать. И даже в чём-то более гибкий:
1. Делаешь файлик типо config.routes.php, в нём:

Код (PHP):

1. $routes = [
2.      "login" => "/system/modules/login.php",
3.      "logout" => "/system/modules/logout.php"
4.      // и т.д., сколько тебе нужно
5. ];

И сам роутер:

PHP:

1. require_once(__DIR__ . "/config.routes.php");
2. $go = isset($_GET["go"]) ? $_GET["go"]: false;
3.  
4. if (!empty($go)  && isset($routes[$go])) {
5.      include __DIR__ . $routes[$go];
6. } else {
7.    // дефолтное поведение
8. }

Мне так меньше нравится, поскольку, избавившись от проблем безопасности, мы получили необходимость добавлять руками элементы в массив $routes, но, с другой стороны, гибкость в том, что бы теперь можем целевые файлы держать где угодно, а не только в system/modules, ну и всё равно - при добавлении новой фитчи мы модифицируем конфиг, и не трогаем сам роутер. switch-ей и if-ов в программе должно быть как можно меньше, поскольку они затрудняют чтение кода. Совсем без них не обойтись, но если возможно - лучше писать что-то другое. Опять же, строчек в этом решении меньше, масштабируемость лучше.

 

Ну да, этот роутинг железобетонный.

А еще застраховались от атаки, которая может, таки пробить и твой предыдущий пример. Допустим, через уязвимость в аплоаде кто-то умудрился пропихнуть тебе шелл в каталог, откуда идут инклуды. Basename тут не спасет. А роутер с вайтлистом спасет.
--- Добавлено ---

Вложенные тернрарные операторы!11 Шучу. Хотя, порой, и такое решение в кассу заходит.

 

Без php, да. В go ключик от массива надо передавать, ясно же Ну только перед ?go=login надо указать имя файла с роутером, если это не index.php

Я же написал, список скриптов и их ключей я вынес в отдельный файл, config.routes.php, чтоб менять его, но не менять файл с роутером для добавления новых маршрутов. Можно было и в одном файле, но мне так больше нравится, конфиги - отдельно, работа - отдельно.
--- Добавлено ---

Чтобы что-то туда пропихнуть, поскольку аплоады никто по умолчанию в этот каталог не будет делать, надо не просто шелл закачать мне, а ещё его и исполнить. А если у меня папка аплоадов не защищена от исполнения скриптов, то нафига злоумышленнику что-то куда-то переписывать? Он его выполнит прямо из каталога для аплоадов, и возьмёт всё, что ему нужно. Или запишет вредоносный код мне прямо в index.php

 

Совершенно верно, но тем не менее, папка апплоада может быть и закрыта, а системная нет. Чисто вот в сферическом вакууме - вероятность проблемы есть. Ну и мб просто я параноик, что уж там.

 

Ну с другой стороны, ТС зато столько умных вещей почитал ))

 

А во втором варианте он не нужен, если туда какую-нибудь хрень типа ../../../etc передать, он просто её не найдёт в массиве, и сделает дефолтное поведение. basename нужен только в первом варианте скрипта, как подстраховка от таких случаев.

А вообще, чувак, серьёзно, ни в первом, ни во втором случае не написано ничего сверхсложного, в чём нельзя разобраться самому (при условии, что знаешь основы синтаксиса php).....

 

@_ne_scaju_, ёлки, ну ты такие глупые вопросы задаёшь, будто вообще ни слова ни из кода ни из сообщений не понял. Зачем тебе ссылку ставить на login.php ? Ты можешь роутер разместить прямо в index.php в корне
--- Добавлено ---
Мы зачем всё это дело городили - чтоб тебе не приходилось писать полный путь к login. Он уже есть в массиве $routes. Включай голову Или поспи, и перечитай весь тред на свежую голову.

 

Ну так я не знаю, где у тебя роутер, он не обязательно в файле index.php может быть. И если ты подключаешь именно, как сейчас нарисовал, то конечно не переходит - ты же конфиг подключаешь после роутера. У тебя вообще по ходу каша жуткая. Если в index.php роутер, то всё остальное из него нафиг убираешь, для шаблона главной делаешь какой-нибудь home.php. Ну и думай головой, как можно сначала подключить код, использующий массив, а потом - код, его инициализирующий? Я тебе разве так написал? Программа выполняется сверху вниз, если что Потому что не просто копипастить надо чужой код, а разобрать, что каждая строчка делает, посмотреть в доках незнакомые функции и т.п. А ты пробуешь подставлять туда-сюда, вместо того, чтобы подумать
--- Добавлено ---
Логику своей программы сначала словами надо написать

Куда хочешь

 

Ну я сдаюсь

 

Да куда уже прямее....

 

Я уже указал. конфиг надо включать раньше кода роутера. Чувак, ну разбери ты мой код сначала. Вот возьми ниже, напиши по-русски, как мой код работает, по шагам. Потом как твой код. Причём здесь профессионал-непрофессионал в php. 4 строчки разобрать - не нужен большой уровень.