killer8080
Если кто-то хочет проверить код целиком, то достаточно попросить ТС показать код целиком

Нельзя утверждать, что код дырявый лишь потому, что ты сам его не видел целиком. Я вот не видел код гугла, я же не утверждаю, что он дырявый. Выше были предложены 4 варианта как можно за строчки с 1 по 28 сделать данный код безопасным. Мне кажется, что один ещё вариант был упущен, грубо говоря их 5 приходят на ум.

_____________
Обучаю веб-программированию качественно и не дорого: http://school-php.com
Фрилансер, принимаю заказы: PHP, JS, AS (видео-чаты). Писать в ЛС (Личные сообщения на phpforum).

Ron
А что ещё надо от кода?
1) Простота
2) Скорость
3) Без багов
4) Безопасен
5) Понятный код

5 принципа идеального кода. На всё отвечает. Ах да, забыл 6-ый принцип, это модные тенденции в ущерб первым 5-и правилам.

Хотя какое это имеет отношение к двум темам:
1) Ошибка ТС. Ему так до сих пор не помогли
2) Безопасность. Уже доказано, что код безопасен.

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

Дискуссии о лучшем коде не будет и это бессмысленно общаться в данной теме. Кто ты такой? Ты неизвестный мне Ron, парень, который немного времени в программировании и прочитал несколько умных книг. Возможно что-то у тебя мудрое есть, но другая часть головы забита глупостями как и остальных. С другой стороны логика PHP-разработчиков, которые создают переменную не константой, значит они как бы говорят: "вот переменная, её можно модифицировать". И спустя года в высшем свете PHP даже идей не было сделать $_GET и $_POST константами.

Как-то так, без обид. Давай лучше сделаем что-то полезное, поможем ТС всё же решить его проблему

_____________
Обучаю веб-программированию качественно и не дорого: http://school-php.com
Фрилансер, принимаю заказы: PHP, JS, AS (видео-чаты). Писать в ЛС (Личные сообщения на phpforum).

inpost
Будь мужиком (С), признайся, что ты королева, а код твой говно, как уже ВСЕ, чье мнение могло бы быть ценное это тебе сказали.

И пройди уже, в конце концов курс Евгения Попова, чтоли...
А то стыдно прям за "эксперта".

Надеюсь, теперь тебе будет спаться спокойнее.

И да, будь мужиком (С)... признайся.... мужиком (С) ...

Выбери для себя другие критерии мужества, а не ссаться кипятком в интернете (дарю народную мудрость)

_____________
[продано копирайтерам]

AllesKlar
Мне твоё мнение не интересует, меня лишь интересовало доказать всем, что ты говоришь глупости и я это сделал

_____________
Обучаю веб-программированию качественно и не дорого: http://school-php.com
Фрилансер, принимаю заказы: PHP, JS, AS (видео-чаты). Писать в ЛС (Личные сообщения на phpforum).

inpost
Ладно, иди с миром... ты меня достаточно потешил.

_____________
[продано копирайтерам]

AllesKlar
Мы не обсуждали тут говнокод, мы обсуждали является ли данный код уязвимым. Цитирую, если ты забыл:

inpost
Я тебя ненавижу? С чего это вдруг?
Ты шут, потешник, незрелый юнОша (хоть уже и годиков есть)
Твоя персона выглядит в моих глазах примерно вот так:



_____________
[продано копирайтерам]

include './modules/'.$_GET['module'].'/'.$_GET['page'].'php';

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

_____________
Свои мозги еще никто не отменял.
Телепатов нету.

twin
Тогда ты взломай, если ты тоже считаешь код уязвимым. Или $module менее уязвима чем $_GET['module'] в твоём понятии?

Правильнее говорить: существует множество неизвестных при условии которых этот код может быть уязвимым, а может быть и нет. Если тебе кажется, что заменить $_GET['module'] на $module или Core::$module, или ещё чего MODULE приведут к созданию кода безопасным, то ты глубоко ошибаешься, так как можно наговнокодить и любой из вариантов сделать уязвимым.

Типичный пример современного программирования:

$module = $_GET['module'];

Вот к чему приводит подобного плана заблуждения, и такие строчки повсеместно у новичков. А причина очень простая, они вычитали на форумах, что работать с $_GET['module'] не безопасно, а если пишут $module - уже безопасно.

_____________
Обучаю веб-программированию качественно и не дорого: http://school-php.com
Фрилансер, принимаю заказы: PHP, JS, AS (видео-чаты). Писать в ЛС (Личные сообщения на phpforum).

include './modules/'. $module .'/'. $page .'php';