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

Question

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

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

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

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

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

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

#angular #frontend #javascript #programming #russian

0

30.06.2021

14 ответов

2 просмотра

Евгений Гений

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

0

30.06.2021

Евгений Гений

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

0

30.06.2021

Smooth Operator

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

0

30.06.2021

Vitalii Yatchenko Автор вопроса

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

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

0

30.06.2021

IL'shat Khamitov

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

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

0

30.06.2021

Smooth Operator

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

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

0

30.06.2021

IL'shat Khamitov

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

Еэс

0

30.06.2021