Комментарии 11
Это что лаба в универе вместо Pastebin сюда попала?
+6
Вы молодец, что пытаетесь. Но вряд ли на Хабре место для статей такого уровня. Учитесь, изучайте PHP, изучайте, как устроены другие фреймворки — и не прекращайте писать код, даже если его критикуют. Особенно, если его критикуют!
+5
Поздравляю, вы изобрели ADR.
0
$users->findBySql("DROP DATABAE users");
+4
Не хотелось бы быть учителем… Попробуйте ниже перечисленное.
1) Не нужно делать инклюды классов — попробуйте изучить composer.
2) Освойте Git, это поможет контроллировать изменения в коде.
3) Посмотрите на PSR. Скачайте PhpStorm и установите на него Cs Fixer
4) Попробуйте отделить модель от операций с БД (можно загуглить паттерн «Репозиторий»)
5) Попробуйте «отвязать» модель от итератора. Создайте класс ModelCollection
6) В вашей модели $data это public-свойство. Но также есть публичные методы __get и __set работающие с этим свойством. Нужно оставить что-то одно
7) В модели метод findOne может выбросить ошибку: обработайте строчку $id = $this->data[0][«id»];
8) Удалите в модели метод exec. Он избыточен
…
1) Не нужно делать инклюды классов — попробуйте изучить composer.
2) Освойте Git, это поможет контроллировать изменения в коде.
3) Посмотрите на PSR. Скачайте PhpStorm и установите на него Cs Fixer
4) Попробуйте отделить модель от операций с БД (можно загуглить паттерн «Репозиторий»)
5) Попробуйте «отвязать» модель от итератора. Создайте класс ModelCollection
6) В вашей модели $data это public-свойство. Но также есть публичные методы __get и __set работающие с этим свойством. Нужно оставить что-то одно
7) В модели метод findOne может выбросить ошибку: обработайте строчку $id = $this->data[0][«id»];
8) Удалите в модели метод exec. Он избыточен
…
+1
Почитайте если не про весь SOLID, то хотя бы про SRP и DI
0
О, привет 2005-ой год! Тогда были актуальны такие фреймворки.
0
Такие статьи появляются несколько раз в год и являются результатом нежеления понимать, что происходит в мире PHP. Возможно автор раньше программировал на чем-то старом, теперь узнает о языке PHP, плохо владеет английским и вообще интернетом, и сидит по старинке, что-то выдумтывает сам. Последнее, в принципе, не так уж плохо, но такие поделки никому на хабре в статьях не могут быть интересны, ибо через это все уже прошли лет 10-15, а то и больше, назад. Достаточно позорно в 2020-ом году не знать про Composer (или хотябы просто про автозагрузку) и PSR, я уж не говорю про SOLID.
0
Зарегистрируйтесь на Хабре, чтобы оставить комментарий
Task framework