Объясните что плохого в этом коде. Этот код служит всего лишь для того, чтобы сценарий нельзя было запустить напрямую. Без него можно было обойтись, но это плохое решение.
Ну die(«Hacking attempt!») это стандартная проверка, так во всех файлах движка и со своей задачей справляется, не вижу ничего плохого.
Ну а в целом, мне кажется, качество кода довольно хорошее. Единственная моя оплошность — плохие имена переменных. Но я думаю что любой новичок, прочитав статью и комментарии к коду, разберется что к чему.
Насчет разделения данных и представления: это пример простого модуля, тут идея в том, что все в одном файле, все доступно и понятно. Можно было сделать на шаблонах, но здесь это не нужно, поверьте мне.
Ну а в целом, мне кажется, качество кода довольно хорошее. Единственная моя оплошность — плохие имена переменных. Но я думаю что любой новичок, прочитав статью и комментарии к коду, разберется что к чему.
Насчет разделения данных и представления: это пример простого модуля, тут идея в том, что все в одном файле, все доступно и понятно. Можно было сделать на шаблонах, но здесь это не нужно, поверьте мне.