Comments / Profile of ellrion / Habr

Простой, современный MVC фреймворк на PHP для разработки сайтов «с нуля»

ellrion Feb 28 2017 at 14:03

Функцию экранирования это само собой. Т. е. в идеале нужен нормальный шаблонизатор. Но если мы используем просто php шаблон то делаем функцию

/**
 * @param mixed $value
 * @return string
 */
function e($value)
{
    return htmlspecialchars((string) $value, ENT_QUOTES, 'UTF-8', false);
}

и используем её: <?= e($some) ?>

Шорт-теги (<?=) — зло

У вас устаревшая информация. Во первых шорттеги считались злом потому что могут быть не включены. Но с PHP 5.4.0 именно <?= не считается шорттегом и директива конфигурации на него не влияет.

редакторы кода не находят такие логические блоки

Я не знаю чем вы пользуетесь. Мой шторм справляется. Но смысл что эти теги делают более читаемым код и без сворачивания

Простой, современный MVC фреймворк на PHP для разработки сайтов «с нуля»

ellrion Feb 28 2017 at 12:31

Не хочется хейтить или негатив лить. Но простите, я не очень понимаю зачем вы это выложили. Нет оно конечно даже боли ваш код не вызвал. Для себя в академических целях, прекрасное занятие.

Дам пару советов что стоило бы поменять: про неймспейсы вам уже сказали, ту да же и автолоад.

Еще очень просится разделить код самого ядра с его тестами и непосредственно приложения на его основе. Вынесите его в отдельный пакет.

Управление ассетами в контролере и конфигах бекенда как то неуклюже, не их это дело.

Php как шаблонизатор если используете то имхо лучше использовать <?= ... ?> вместо <?php echo ...; ?> и для управляющих конструкций вариант синтаксиса с :. Т. е.

<?php foreach(): ?>
...
<?php endforeach ?>

ellrion Feb 20 2017 at 11:44

ну вот в в 2017(условно) пункт 2 уже выкинули) но добавили webpack)

ellrion Feb 20 2017 at 11:42

Это если в команде договор использовать продукт от jetbrains. Смысла так сильно ограничивать команду я лично не вижу. А если же у вас не проект а библиотека, то .idea уж точно быть не должно не в репозитории не в .gitignore

ellrion Feb 20 2017 at 11:38

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

ellrion Feb 20 2017 at 09:55

Точно, вопрос деплоя и окружения же никак не затронут. "Нужно больше ~~золота~~ инфы"

ellrion Feb 20 2017 at 08:46

Есть мнение что помещать в .gitignore тоже моветон) Нужно в .git/info/exclude. Но это конечно спорно

ellrion Feb 20 2017 at 08:29

Автор, В 2017 держать под гитом .idea моветон)

+15

ellrion Feb 20 2017 at 08:28

Крайне спорное утверждение

ellrion Feb 20 2017 at 08:24

Это смотря какое приложение. Да и фреймворки разные есть. Тут больше важно что нужно знать современные концепции и стандарты современного веб приложения. Вот composer — да; PSR'ы; Неймспейсы; Автолоадинг; Система контроля версий; Единая точка входа; Паблик директория; Роутинг; Шаблонизация; Query builder. Остальное имхо опционально.

И если уж говорить о самой статье, то замените symfony на laravel и react на vue. И статья выйдет короче)

ellrion Feb 16 2017 at 14:05

Это понятно. Просто хотелось указать человеку на какие то конкретные вещи. Хоть на пару моментов.
К слову, автор, вот тут для фана ну и в обучающих целях писалось(так и не дописалось) так вот там буквально в нескольких функциях сделано то, что и у тебя но в разы гибче.

ellrion Feb 16 2017 at 12:23

Огромную роль в современном php комьюнити играет PSR и composer. Я очень советую воспользоваться ими. Хотя бы PSR-2 и PSR-4

ellrion Feb 16 2017 at 12:16

Реврайты в .htaccess?! А если у меня nginx+fpm (вернее не если а только так). А если мне нужна иная структура например resources/{resource}/subresources. А еще разная обработка для get И post. Ну есть же концепция роутинга и без нее никуда.

ellrion Feb 16 2017 at 12:06

И ещё, дабы защитить свой проект от доступа к скриптам, в папки controller, model и. view надо положить .htacess с содержанием: Deny from all

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

Вообще всё очень спорно и не понятно зачем оно на хабре.

PHP-Дайджест № 102 – интересные новости, материалы и инструменты (1 – 12 февраля 2017)

ellrion Feb 14 2017 at 15:15

Т.е. вы предлагаете везде копипастить код ?

Auth::user() && Auth::user()->id == $user->id

Ну это тоже не дело.

Явное лучше, чем неявное

Но повторение кода тоже зло. Именно из-за этого есть "магия" которая позволяет убрать копипаст. Так что тут нужен баланс и компромисс.

PHP-Дайджест № 102 – интересные новости, материалы и инструменты (1 – 12 февраля 2017)

ellrion Feb 13 2017 at 13:44

собственно вы и показали что используете префиксы: ace_editor_, send_email_

PHP-Дайджест № 102 – интересные новости, материалы и инструменты (1 – 12 февраля 2017)

ellrion Feb 13 2017 at 08:05

по мне так fn выглядит самым приятным и логичным из 4 вариантов (5 если считать еще и |). Как раз сокращенное function.

PHP-Дайджест № 102 – интересные новости, материалы и инструменты (1 – 12 февраля 2017)

ellrion Feb 13 2017 at 07:25

удивляет нежелание его принимать. Последнее время ананимок в коде очень много. И код бы стал гораздо приятнее.

JSON API Мой Склад, самообучение

ellrion Jan 23 2017 at 15:36

Код очень стилистически соответствует самому изложению. Эдакое размашистое разгильдяйство. Исключительно по моим ощущениям конечно.