169 похожих чатов

Ребята, привет Вопрос слегка философский. Меня просто смущает то, как сейчас

сделано в проекте. Кажется что это bad practices и есть желание переделать) Хотел бы узнать, может я не прав и лучше не трогать

На проекте ngrx. Переходя на страницу статей, через эффект подтягивается список статей, выводятся превью. По нажатию на превью, выкидывается экшн, его перехватывает эффект, подтягивает полную статью, кладет в стору и в конце эффекта есть tap() , который делает navigate на страницу со статьей.

На странице со статьей из сторы селектится currentArticle

Для меня это выглядит как ngrx головного мозга, в данном случае можно было вообще без него обойтись. Или в этом все же какое-то зерно есть?)

И очень смущает такая навигация в эффекте. Это не является плохой практикой? (Если это не редирект какой-то в случае ошибки)
Спасибо!

14 ответов

24 просмотра

да норм вроде. мы также делаем у себя. как вариант лучше вынести навигацию в компонент наверное.

норм сделано все

просто ты ещё не видел плохие примеры. твой пример сразу глаз радует)

для меня выглядит как и для тебя)

Vitalii-Yatchenko Автор вопроса
Евгений Гений
просто ты ещё не видел плохие примеры. твой пример...

Просто я бы переделал следующим образом: При нажатии на превью делал бы сразу навигейт на страницу статьи, через резолвер подтягивал бы полную статью. И уж если все таки надо поместить статью в стору, то засовывал уже бы в компоненте статьи (хотя на первый взгляд это не должно быть в глобальном стейте) Мне кажется, что так проще, очевиднее. Не нужны вот эти пляски с экшнами, эффектами)

Vitalii Yatchenko
Просто я бы переделал следующим образом: При нажат...

Ну они же нужны, эти пляски, у тя гуи всего лишь отображает инфу, все должно ходить через некую шину, все события Если везде так, и в твоей фиче будет прямая работа, то ты посеишь говнокод Или все гоним через редакс/нгркс или не юзаем этот потход вообще Чуть того чуть сяго - говно

IL'shat Khamitov
Ну они же нужны, эти пляски, у тя гуи всего лишь о...

В заложниках короче вы, один раз его подключили и все

Vitalii Yatchenko
Просто я бы переделал следующим образом: При нажат...

смотри. и так можно. но с беком должен общаться эффект. + резолвер не показывает компонент. это может по дизайну не подойти

Vitalii Yatchenko
Просто я бы переделал следующим образом: При нажат...

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

Smooth Operator
В заложниках короче вы, один раз его подключили и ...

поидее нгркс можно юзать через фасад. и менять его на что хош. хоть сервисы обычные.

Smooth Operator
А еще можно его не использовать

это только когда сильно шариш. и все шарят если не один

Похожие вопросы

Обсуждают сегодня

Господа, а что сейчас вообще с рынком труда на делфи происходит? Какова ситуация?
Rꙮman Yankꙮvsky
29
А вообще, что может смущать в самой Julia - бы сказал, что нет единого стандартного подхода по многим моментам, поэтому многое выглядит как "хаки" и произвол. Короче говоря, с...
Viktor G.
2
30500 за редактор? )
Владимир
47
а через ESC-код ?
Alexey Kulakov
29
Чёт не понял, я ж правильной функцией воспользовался чтобы вывести отладочную информацию? но что-то она не ловится
notme
18
У меня есть функция где происходит это: write_bit(buffer, 1); write_bit(buffer, 0); write_bit(buffer, 1); write_bit(buffer, 1); write_bit(buffer, 1); w...
~
14
Добрый день! Скажите пожалуйста, а какие программы вы бы рекомендовали написать для того, чтобы научиться управлять памятью? Можно написать динамический массив, можно связный ...
Филипп
7
Недавно Google Project Zero нашёл багу в SQLite с помощью LLM, о чём достаточно было шумно в определённых интернетах, которые сопровождались рассказами, что скоро всех "ибешни...
Alex Sherbakov
5
Ребят в СИ можно реализовать ООП?
Николай
33
https://github.com/erlang/otp/blob/OTP-27.1/lib/kernel/src/logger_h_common.erl#L174 https://github.com/erlang/otp/blob/OTP-27.1/lib/kernel/src/logger_olp.erl#L76 15 лет назад...
Maksim Lapshin
20
Карта сайта