SarcasmIO
В будующем, будет, когда разберусь
а что это за Error404 в моделях делает?
Максим
Это временно
SarcasmIO
Откажись от названий аля Main и тп по ним не понятно что они делают, если ты накидав mvp делаешь с моделями/контроллерами в них какие-то преобразования
The Ant
Переписал движок, пожалуйста укажите на недочеты и ошибки: https://github.com/umirode/php-mvc
https://github.com/umirode/php-mvc/blob/master/Core/Router.php#L87 http://php.net/manual/ru/function.ucfirst.php
SarcasmIO
называй сущности своими Именами
SarcasmIO
User (model) UsersController
Максим
Main -> HomePage
Максим
Ad.x Спасибо
SarcasmIO
теперь касательно конфигов уже чуть лучше и чище, но все равно не то. мало того fully qualified domain name для роута слишком неудачное
The Ant
https://github.com/umirode/php-mvc/blob/master/Core/Error.php вотэто вот зачем?
Anonymous
Переписал движок, пожалуйста укажите на недочеты и ошибки: https://github.com/umirode/php-mvc
Мне кажется, в index.php аутолоадер такой лишний. Либо вынеси в отдельный файл, либо через composer подключи аутолоадер (самый адекватный вариант)
SarcasmIO
spl_autoload больше оверхеда на самом деле чем composer и psr4
The Ant
изучаем exception'ы
Anonymous
Браузер, по сути - его реализация.
Anonymous
HTML - декларативный язык.
И там пишешь в императивном стиле!
SarcasmIO
ага а потом приходит к нам HATEOAS и говорит что мы до сих пор не умеет интерпритировать гипертекст
SarcasmIO
и что мне мешает на пыхе декларативно прогать на массивах
Anonymous
И там пишешь в императивном стиле!
Императивный* Простите
Максим
Всем спасибо, пойду пробовать исправлять ошибки!
Anonymous
Императивный* Простите
А в чём ошибка? о.о
SarcasmIO
Всем спасибо, пойду пробовать исправлять ошибки!
давай что бы некст бранч был с composer autoload'ом
The Ant
А в чём ошибка? о.о
с большой буквы писать надо, очевидно же
SarcasmIO
если будут какие-то проблема пиши тебе всегда помогуит
Максим
ок, хорошо)
Anonymous
А в чём ошибка? о.о
Не вижу просто причины, говорить, что код на html - это не программв
Anonymous
ок, хорошо)
У тебя проблемы с именованием моделей / контроллеров
Максим
Да, уже понял
Anonymous
Получается, что инициализация формы в каком-нибудь Делфи - это тоже не программа
Anonymous
если следовать такой логике
The Ant
с чего вдруг?
SarcasmIO
ну смотрите html же может обладать каким-то стейтом
The Ant
продолжай )
Anonymous
ну смотрите html же может обладать каким-то стейтом
Это даже не причем, есть декларативный язык, есть реализация
SarcasmIO
мне интересно размышления на эту тему послушать
Anonymous
а
Anonymous
ну ок)
SarcasmIO
это был вопрос
Alex
кто-нибудь может сказать как обосновать использование конкретной библиотеки в научной работе?
Alex
чтобы не прикопались
Anonymous
Забаньте этого бота пожалуйста
Vladislav
кто-нибудь может сказать как обосновать использование конкретной библиотеки в научной работе?
Как пример, что в ней реализован как-то алгоритм, работу которого ты оптимизируешь
Максим
Изменил имена классов, добавил composer: https://github.com/umirode/php-mvc
Максим
Так лучше?
SarcasmIO
Так лучше?
лучше но зачем ты тогда рекваеришь конфиги
SarcasmIO
и роутинг?
SarcasmIO
сложи все допустим в src/
SarcasmIO
и сделай автозагрузку по App неймспейсу оттуда
Максим
ок
SarcasmIO
"autoload": { "psr-4": { "App\\": "src/" } },
SarcasmIO
https://github.com/umirode/php-mvc/commit/d4ad601896cd1b52d1d841e1dd3e370e723ed5d1#diff-f8ba6dd755dd303a580035c9829a574fR19 Сделай use App\Core\View; и делай $this->view = new View;
SarcasmIO
и еще неплохо бы тебе твой $router->add('/404') делать автоматом когда путь который запросил пользоатель отсутствует в карте маршрутов(твоих роутов)
SarcasmIO
аля request site.com/something берешь something смотришь есть ли он у тебя в карте маршрутов если нет return view 404
Максим
То есть убрать $router-add('/404')?
Максим
И автоматом
SarcasmIO
ну смотри у тебя 2 выбора
SarcasmIO
захардкодить твой урл на 404 и когда нету маршрута то делать return redirect('404')
SarcasmIO
либо же убрать что более разумнее
SarcasmIO
и сделать автоматом
Anonymous
Немного не понимаю, как работает аутолоадер композера. Я про: "psr-4": { "Core\\": "Core/", "App\\": "App/" }
Anonymous
Core\\ = директория Core/ (открывается относительно файла composer.json?), почему два \\?
SarcasmIO
PSR4 же
Anonymous
надо конвенцию почитать
Максим
Максим
Максим
@oneerror, пожно подробнее про автозагрузку, мне в корне создать папку src и переместить туда App и конфиги?
SarcasmIO
https://github.com/umirode/php-mvc/commit/121700f461c84b6bc834cfc96b6e48a59f9ddaf9#diff-34507bd82ebabd6205bbb79860ce73e1R20 вот это можно заменить на $this->file = file_exist($file) ?? another_file
SarcasmIO
можешь посмотреть в сторону phpdotenv
SarcasmIO
что бы вынести в енвайрмент названия конфигов и прочий сахар
SarcasmIO
сделай git rm —cached venor/*
SarcasmIO
git add. git commit —amend
SarcasmIO
git push —force
Максим
ок
Максим
Обновил
Максим
папку Core в src надо?