search

fullstack-development / react-redux-starter-kit

Modular starter kit for React+Redux+React Router projects.
https://demo.fullstack-development.com/
MIT License
91 stars 13 forks source link

Выпилить inversify #135

Open sk1e opened 4 years ago

sk1e commented 4 years ago

Не вижу практической пользы от этой монструозной штуки, учитывая архитектуру, и структуру проектов, на которые рассчитан стартер кит. Есть куда более простые альтернативы с прокидыванием зависимостей в аргументы/конструкторы, обновлением стора через экшены в middleware. Только добавляет лишние КБ и ничем неоправданную сложность, которую тяжёло поддерживать тем же джунам. Усложнение на ровном месте.

Znack commented 4 years ago

inversify ещё же не type safe насколько я помню? А какие альтернативы предлагаешь?

sk1e commented 4 years ago

inversify ещё же не type safe насколько я помню?

в какой-то мере

А какие альтернативы предлагаешь?

Там сейчас для двух штук используется: ContainersProvider и FeatureConnector, первый мы собираемся заменить на контексты в https://github.com/fullstack-development/react-redux-starter-kit/issues/134, для второго есть предложение с фабрикой https://github.com/fullstack-development/react-redux-starter-kit/issues/145. Инициализацию и деинициализацию фич я планировал через redux и его middleware сделать

in19farkt commented 4 years ago

как минимум вместо инверсифай, можно рассмотреть First DI