Как вы считаете так делать тупо?

Question

[hili]

Почему я так сделал? Что бы проинклудить все в индексе и отсюда уже рулить всем процессом. Как вы считаете, такой подход приемлимый, или быдлокодинг в квадрате? :))

<?php
require_once('cfg/database.php');
require_once('views/view.php');
require_once('model/model.php');

if($_SERVER['REQUEST_METHOD'] == 'GET'){
switch ($_GET['c'])
{
	case 'buy' : require('inc/buy.php'); break;
	case 'contacts' : require('inc/contacts.php'); break;
	case 'faq' : require('inc/faq.php'); break;
	case 'processorder' : require('inc/processorder.php'); break;
	default : $title = 'Super mega title';
						$content = view_include('v_index.php');
						$page = view_include('views/v_base.php', array
						(
						'title' => $title,
						'content' => $content,
						));
						echo $page; break; 
}
}else if ($_SERVER['REQUEST_METHOD'] == 'POST'){
//Прячем в скрытых полях формы название контроллера и по нему определяем какой скрипт грузить
switch ($_POST['c']){
	case 'processorder' : require('inc/processorder.php'); break;
	case 'gateway' : require('inc/gateway.php'); break;
		}
	}
		
?>

Answer 1

[Александр Аксентьев]

Да чтож такое-то... 2016-ый год на дворе.

https://toster.ru/answer?answer_id=858394

Comments

[hili]

Это все понятно, я же взял подправить старый проект. Там вообще все плохо было. В корне все файлы валялись.

[Александр Аксентьев]

hili: ну при таком коде особо ничего не поменяется, в корне они или нет.

Такое обычно не трогают, если работает особенно.

[Дмитрий]

hili: если без глобальных переделок но что-то потрогать в этом коде очень хочется, я б заменил тут $_GET/$_POST на $_REQUEST
так оно тупо на 5 строк меньше занимать будет

[index0h]

Дмитрий
> ..я б заменил тут $_GET/$_POST на $_REQUEST...
Не стоит давать хреновых советов.

[Дмитрий]

index0h: а почему, по вашему, этот совет плохой?

[#алгоптимизируй #отботизируй]

Дмитрий: например ,секьюрити.

[#алгоптимизируй #отботизируй]

Дмитрий: в целом, не конкретно для этого кода.

[index0h]

Дмитрий
Потому, что:
* api, которое вы предоставляете в данном случае более вариативно. Это приведет к ошибкам и костылям.
* супер глобальные переменные (просто глобальные - тоже) в принципе не стоит использовать так как они доступны на запись.
* покрывая тестами код вы обнаружите, что ваши супер и просто глобальные переменные - это внешние неявные зависимости. Как следствие код может менять свое поведения основываясь на этих самых неявных зависимостях. Как результат - ваши тесты будут давать ошибочно-положительные результаты.
* использование $_REQUEST примерно значит: да мне в принципе по боку, как придут данные, как следствие там, где вы рассчитываете на POST может придти GET и сломать вам часть обработки, просто потому, что вы понадеялись.

[Дмитрий]

Sharp:
>например ,секьюрити.
и как использование _REQUEST повлияет на безопасность конкретно этого кода?

index0h:
>api, которое вы предоставляете в данном случае более вариативно
конкретно в этом коде вариативности не вижу
посмотрите на строчку case 'processorder' : require('inc/processorder.php'); break;
подключается и для _GET и для _POST
это, на ваш взгляд, "вариативность"?

>супер глобальные переменные (просто глобальные - тоже) в принципе не стоит использовать
вы, возможно, удивитесь, но и _REQUEST и _POST - это все суперглобальные переменные
разницы, что именно использовать (если не обусловлено логикой приложения) нет

>да мне в принципе по боку, как придут данные
конкретно в этом коде так и есть. тут принципиально по боку как придут данные.
Посмотрите, опять же, на строчку case 'processorder' : require('inc/processorder.php'); break;

Из минусов:
если записать что-то в _REQUEST, позже попробовать считать с _GET или _POST. там этих данных не будет. Но, поскольку, делать так считается быдлокодингом, можно этот минус игнорировать.

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

Для данногго куска кода оба эти минуса не критичны вообще никак.

[index0h]

Дмитрий
> это, на ваш взгляд, "вариативность"?
Чуть ниже я написла, что использование супеглобальных переменных - в принципе плохая идея.

> разницы, что именно использовать (если не обусловлено логикой приложения) нет
разница есть, причем существенная. Подвязываясь на суперглобальные переменные - с консольным запуском будут проблемы.

[Дмитрий]

index0h:
>Чуть ниже я написла, что использование супеглобальных переменных - в принципе плохая идея.
ну так и _POST это тоже суперглобальная переменная.

Answer 2

[index0h]

Для 2006-го года вполне норм.
С тех пор на php нашли следующие напасти:
* PSR-2 - это набор соглашений по форматированию кода.
* PSR-4 - это правила по реализации автолодинга.
* Composer - это библиотека для управления зависимостями и генерации автолодинга.
* Появилось много шаблонизаторов типа twig.
* Устоялось понятие router и это понятие реализовано практически в каждом современном фреймворке.

Как вы считаете, такой подход приемлимый, или быдлокодинг в квадрате?

Это не говнокод, который нет смысла поддерживать.

Почитайте на досуге, думаю вам будет полезно.