Что мне изменить в этом проекте?

Question

[Денис Пабло]

Я Пишу свой framework, в частности для себя, но если кто-то хочет попользоваться, логично что я не против.
Так вот : я пишу его уже так давненько и мне нужны советы по устранению ненужного кода более оптимизированных функции ну и вообще что бы было лучше сделать.

Ссылка на репозиторий : https://github.com/flowerphp/flower

Насчёт оформления readme.md не придираться я пока только пишу framework как только допишу оформлю его как надо

Я бы не хотел видеть ответы или коментарии насчёт того что есть уже хорошие движки и т.д, как я и сказал пишу в частности для себя и для своих будущих проектов

Comments

[Roll Back]

На, лишней не будет: https://www.ozon.ru/context/detail/id/5011068/

[Дмитрий]

Тесты обычно ни чего не возвращают void

Answer 1

[Daria Motorina]

1) в репозитории у Вас получился скорее скелетон, а не пакет фреймворка, у папки шаблонов должно быть осознанное предназначение, нп. приветствующая страница, папка public тоже обычно отсутствует
2) именование интерфейсов как iSomething морально устарело и неправильно по PSR (для классов тоже, там был класс reCaptcha)
3) тесты должны отражать тесты на функционал фреймворка, пока там заглушки
4) непонятен слой Promises, под ним там видно много классов, которые отвечают за разные вещи, стоило бы их структурировать, приставка extreme тоже не отражает предназначение классов
5) странно видеть в core класс рекапчи, это типично клиентская задача и фреймворк не должен ее решать
6) сейчас есть тенденция подключать сторонние orm во фреймворк, потому что проектирование абстракций для базы с нуля это огромная работа

Comments

[Денис Пабло]

Спасибо учту!

[kafkiansky]

Немного дополню:

1. https://github.com/flowerphp/flower/blob/master/ap...
В интерфейсе не должно быть конструктора. Вы очень ограничиваете реализации.

2. https://github.com/flowerphp/flower/blob/master/ap...
Подчеркивания не юзают уже. Ну и слабовато для коллекции.

3. https://github.com/flowerphp/flower/blob/master/ap...

$core
            ->getPromise()
            ->getConfig()
            ->getPromise()
            ->getDatabases()
            ->getMySQL()
            ->getHost();

Очень странная запись, особенно два вызова `getPromise()`. Непонятно, что мешало сразу вызвать getDatabases().

4. https://github.com/flowerphp/flower/blob/master/ap...

Зачем вам снаружи данные подключения? Зачем драйверу возвращать пароль, юзернейм и так далее? Ему их передают, он подключается и возвращает коннект.

5. индексный файл должен быть в паблике и на него гнать все запросы
https://github.com/flowerphp/flower/blob/master/in...

6. https://github.com/flowerphp/flower/blob/master/ap...

Магия от любителей ларавеля (я про то, что объекта Environ нет в файле, за его создание отвечает тот, кто подключит файл, а это срань, как по мне).

7. Ну и нет контейнера. Черт разберет, как все ваши зависимости менеджить, подключать, менять, если все они зашиты в конструкторах через `new`. Шаг влево-вправо и ваш фреймворк сломается.

Answer 2

[index0h]

1. Это не фреймворк, а скелет обросший жиром.
В отличии от фреймворка, как внешней зависимости другого вендора, ваш скелет - полностью под вендорингом конечного инженера. Плохо то, что обновляя версию скелета риски что-то сломать на много выше, чем для внешней зависимости.

2. В public нету точки входа index.php, я очень надеюсь что это просто ошибка, а index.php в корне проекта просто забыли удалить.

3. Что делать, если нужно 2 разных подключения к БД, а не одно?

4. Если я правильно понимаю, то что вы называете promise - это DTO + Builder. Очень похоже на лишние сущности.

5. Сессии в файлах - это очень странное решение. Как только вам потребуется завести еще один инстанс - все пойдет по ***де. Обычно сессии хранятся в memcached / redis как раз для того, что бы обеспечить горизонтальное масштабирование.

Очень рекомендую почитать на досуге https://github.com/index0h/php-conventions