Пишем простейший роутинг
116  aRiGaTo [Off]
 don't tread on me
(13.11.2015 / 19:48)
Artorius,
* Комментарии вида «инициализируем модель вида» и «загружаем вид» ни разу не полезные. Они очевидны и лишние. А для того, чтобы сказать, что view.load() - это загрузка, есть документация. Например, PHPDoc.
* Стиль. Так-то согласен. Но только по причине отсутствия официальной нотации в PHP (или я что-то пропустил?)
* «Метод, принимающий в аргумент значение метода, принимающего в аргумент значение другого метода, принимающего в аргумент элемент глобального массива». Во многих IDE есть такая штука как ограничение строки по длине. Такая раздражающая полосочка посреди окна, за которую нежелательно выходить - программисты с маленьким могут обидеться.
Хотя бы так:
  1. // if(array_key_exists(htmlspecialchars(stripslashes(trim($_GET['r']))),$data->data["pages"]))
  2. function purify_string($str) {
  3.  
  4.   $trimmed = trim($str);
  5.   $stripped = stripslashes($trimmed);
  6.   return htmlspecialchars($stripped);
  7. }
  8. ...
  9. $request = purify_string($_GET['r']);
  10. $data_pages = $data->data["pages"];
  11. if (array_key_exists($request, $data_pages)) {
  12. ...
  13. }

UPD: Ой, Ксакеп уже опередил :)
Изм. aRiGaTo (13.11 / 19:51) (1)
3254  Ксакеп (FMod) [Off]
(13.11.2015 / 19:38)
Я так понимаю, речь идёт про этот кусочек кода:
  1. if(array_key_exists(htmlspecialchars(stripslashes(trim($_GET['r']))),$data->data["pages"])){
  2.  
  3.    //Если есть, определяем $page именем метода
  4.    $this->page = $data->data["pages"][htmlspecialchars(stripslashes(trim($_GET['r'])))];
  5.    ...

Начнём с того, что ты два раза делаешь одно и то же: экранируешь слеши, вставляешь html-сущности. В переменную, сразу же!
  1. $pages =& $data->data['pages'];
  2. $route = htmlspecialchars(stripslashes( $_GET['r'] ));
  3.  
  4. if (array_key_exists($route, $pages)) {
  5.     $this->page = $pages[$route];
  6.     ...

По-хорошему, класс Data должен реализовывать интерфейс ArrayAccess, чтобы не приходилось писать так:
  1. isset($data->data['pages'][$route]); // плохо
  2. isset($data['pages'][$route]);       // лучше
Тогда и не придётся заводить переменную $pages.
Изм. Ксакеп (13.11 / 19:39) (1)
1  aNNiMON (SV!) [Off]
 let live
(13.11.2015 / 19:14)
Цитата Artorius:
Метод, принимающий в аргумент значение метода, принимающего в аргумент значение другого метода, принимающего в аргумент элемент глобального массива - покажи как надо красиво)Ксакеп, я вызываю тебя!
3446  Artorius [Off]
(13.11.2015 / 18:52)
aRiGaTo, Стиль оформления - мое личное дело, на всех не угодишь) Туда же отнесем шуточки, хоть про руки, хоть про что) Познавательные комментарии необходимы, профи такие статьи не читают, удивлен? Метод, принимающий в аргумент значение метода, принимающего в аргумент значение другого метода, принимающего в аргумент элемент глобального массива - покажи как надо красиво)
Изм. Artorius (13.11 / 18:52) (1)
3446  Artorius [Off]
(13.11.2015 / 18:49)
aNNiMON, Это да, позже переписать надо будет
116  aRiGaTo [Off]
 don't tread on me
(13.11.2015 / 17:52)
Соглашусь с n-2 оратором. Добавлю ещё:
* Жуткое оформление. У тебя «Enter» заклинило что ли?
* Стиль. Метод, принимающий в аргумент значение метода, принимающего в аргумент значение другого метода, принимающего в аргумент элемент глобального массива... Ты серьёзно?
* Копипаста с ошибкой. [email protected]
* Познавательные комментарии вида «складывает 1 и 1, получаем 2. Присваиваем переменной foo».
* Шуточки-шутеечки. Ребят, ну про руки это уже не смешно. Для полного счастья ещё не хватает упоминания vi или emacs.
3254  Ксакеп (FMod) [Off]
(13.11.2015 / 17:40)
Соглашусь с предыдущим оратором, недочётов много.
1  aNNiMON (SV!) [Off]
 let live
(13.11.2015 / 17:27)
Artorius, я минус не ставил, но скажу, что в статье не хватает объяснения: почему именно такая структура файлов, зачем нужен bootstrap и т.д. Просто куча исходников, хоть и с комментариями. Диаграммку бы сюда, какие классы в какой последовательности загружаются, какие функции в них вызываются при доступе к той или иной странице.
3446  Artorius [Off]
(13.11.2015 / 13:22)
Может, вы будете комментировать минусы? А то профит как-то теряется)
3446  Artorius [Off]
(13.11.2015 / 13:22)
dimon_net, спасибо)
Всего: 22
<< 1 2 3 >>
К записи
PHP/MySQL
Категории

Мы в соцсетях

tw tg yt gt