react-redux-starter-kit icon indicating copy to clipboard operation
react-redux-starter-kit copied to clipboard

Выпилить inversify

Open sk1e opened this issue 6 years ago • 3 comments

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

sk1e avatar Oct 16 '19 02:10 sk1e

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

Znack avatar Oct 18 '19 08:10 Znack

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 сделать

sk1e avatar Oct 18 '19 08:10 sk1e

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

  • по типам всё норм
  • минималистичное и интуитивное API
  • решает всё что нужно решать
  • в 11 раз меньше весит (837 B против 9.3 kB)
  • но не уверен можно ли на лету подменять реализации, у нас это используется только для хотрелоада

in19farkt avatar Apr 22 '20 14:04 in19farkt