M
store architecture как отдельный вид программирования)
Size: a a a
2020 July 24
(
там 4 разных варианта вообще-то)
A
Да, но у меня на проекте используется именно этот, и я не понимаю зачем вообще иметь возможность провайдить редьюсеры по токену? В каком кейсе это может пригодиться?
AS
Да, но у меня на проекте используется именно этот, и я не понимаю зачем вообще иметь возможность провайдить редьюсеры по токену? В каком кейсе это может пригодиться?
я до недавнего думал, что мне не пригодится необходимость писать свой кастомный forRoot/forFeature. Анннет. Теперь пригодилось 😄
так что лучше запомни, что оно существует и потом внезапмно вспомни в час нужды…)
так что лучше запомни, что оно существует и потом внезапмно вспомни в час нужды…)
A
:)
AS
а
AS
ну я придумал
AS
может быть у тебя есть монорипо, в которой есть 1 библиотечка, но она должна как-то зависеть от разных.. стейтов? хз, сырая идея. И ты инжектишь в эту либу разные стейты в зависимости от приложения или чего-либо есчо
AS
как-то так
тож слабо пока предствляю
тож слабо пока предствляю
(
https://github.com/ngrx/platform/issues/116
вот нашел причины появления InjectionToken-ов в forFeature
вот нашел причины появления InjectionToken-ов в forFeature
(
интересно) это появилось как резльтат workaround-а при ошибке компиляции AOT, когда CLI был еще сырой и бажный, возможно потом пофиксили, а воркараунд остался
A
(λx.Me) Panda
https://github.com/ngrx/platform/issues/116
вот нашел причины появления InjectionToken-ов в forFeature
вот нашел причины появления InjectionToken-ов в forFeature
Насколько я понял читая коменты, то в этом тикете была проблема с тем что редьюсеры по токену не инжектились, а инжектился обьект создаваемый
это ламало логику в этом месте https://github.com/ngrx/platform/pull/36/commits/b7ec98282a304b529c7944707fa08286f7abd02f#diff-9efc753836490d7009e489514515dfd1R42
и воркераундом в их кейсе было привязывать редьюсеры к токену через Object.assign https://github.com/ngrx/platform/issues/116#issuecomment-316843727
new InjectionToken<ActionReducerMap<State>>('Registered Reducers'); https://github.com/ngrx/platform/issues/116#issuecomment-316835579это ламало логику в этом месте https://github.com/ngrx/platform/pull/36/commits/b7ec98282a304b529c7944707fa08286f7abd02f#diff-9efc753836490d7009e489514515dfd1R42
и воркераундом в их кейсе было привязывать редьюсеры к токену через Object.assign https://github.com/ngrx/platform/issues/116#issuecomment-316843727
A
Тоесть InjectionToken'ы добавили до этого
A
(λx.Me) Panda
Не совсем понимаю как именно это полезно для тестов. Насколько я понимаю, в юнит тесте с этого PR'а https://github.com/ngrx/platform/pull/36/commits/b7ec98282a304b529c7944707fa08286f7abd02f#diff-5aa39e264e3e67e9957ca5c3ed95a044R40 можно было передать просто редьюсеры в forRoot
(
да, вы правы) я наискосок прочитал историю
(
В общем, сегодня единственная причина использовать через инжекшн токен : "а почему бы и нет"
A
Надо будет как-то спросить у кого-то из разработчиков. Возможно @AlexOkrushko сможет помочь
AO
Спасибо за таг @wonder_tony 👍
Деталей точно не знаю, они были добавлены до меня, но очень похоже на AOT проблемы + combineReducers.
Деталей точно не знаю, они были добавлены до меня, но очень похоже на AOT проблемы + combineReducers.
AO