Как избежать дублирования экшенов и редьюсеров?

Question

[samegalt]

Есть проект с кучей компонентов, в котором в разных компонентах могут грузится одинаковые данные (например список юзеров). Для загрузки использую один и тот же экшны и редьюсеры. Делаю проверки на загрузку, но все равно приложение иногда падает c ошибкой undefined. Подсказали что нужно на каждый загружаемый обьект писать свой экшн, свою константу и свой редьюсер. Но это же какое-то безумие. Или может я что-то неправильно понимаю:( Подскажите что можно сделать. Заранее спасибо!

Comments

[Роман Александрович]

перестал делать 1 экшен на две различные махинации с данными как раз из-за такой проблемы. В один момент начинаешь забывать какие данные ты должен передавать в редюсер и как диспатчить и при этом думать а не понадобятся ли тебе старые данные. По этому стал создавать экшены и редюсеры на отделаьный экран. 1 экран - 1 редюсер и 1 экшен. Не всегда конечно так и происходит - иногда лень иногда запара но это работает.
Держу все константы в 1 файле и из него импортирую только те что мне надо. Плюс этого в том что я могу быстро проверить а не писал ли я такую константу для другого экшена.
Так что 1 скрин - 1 редюсер + 1 экшен это хорошая практика.
И человек который придет в команду быстрее освоится и не будет спрашивать а где что у вас лежит.

[samegalt]

Роман Александрович, спасибо за совет) Уточнил вопрос в комменте ниже. Выходит я не могу писать в один редьюсер разные данные, чтобы не писать кучу ключей в обьекте стейта? Тогда получается какая-то каша из названий констант и ключей стейта, если похожих загружаемых обьектов слишком много :(

[Роман Александрович]

samegalt, можете но рано или поздно это вылезет вам боком

[samegalt]

Роман Александрович, Выходит уже вылезло( Разве нет никакого удобного выхода?

[Роман Александрович]

samegalt, я ж говорю для каждого скрина писать свой редюсер и экшены. Так вы 100% будете уверены в данных.

Answer 1

[Максим]

Одинаковые данные грузятся одним и тем же экшеном, в один и тот же редьюсер.
Ситуация с юзерами вполне ясна, загружаете их и используете на разных страницах. Если нужно обновить список юзеров - то вызывается тот же экшен.
Если нужно как-то с юзерами взаимодействовать, но так, чтобы в другом месте они не изменелись, то поясните, я не могу придумать такой кейс, обычно это "не одинаковые" данные.

Comments

[samegalt]

Сори, затупил с формулировкой из-за плохого понимания сути. Попробую исправиться. Я юзаю разные экшены (фетчат разные объекты с данными) но пишу их в один редьюсер. Например 1) fetch ... /users/?search=${name}... 2) fetch ... /users/?search=${name}&is_superuser=${true}... - это два разных экшена, чтобы загрузить 2 разных обьекта, но использую одну константу и пишу их в один редьюсер, так как думается что в разных компонентах на разных страницах итс окей:
сase actions.LOAD_USERS:
return state.set('users', action.data);

Насколько я теперь понял для каждого экшена нужно писать свою константу и свой ключ-значение в обьекте стейта, типа:
1) сase actions.LOAD_USERS:
return state.set('users', action.data);
2) сase actions.LOAD_USERS_ARE_SUPERUSER:
return state.set('usersSuper', action.data);

Я правильно понимаю? Если да, то вопрос в том, что это мне кажется странным, потому что проект большой и это куча работы, не говоря уже о плутанице с именованиями.

Огромное спасибо, что возитесь с глупым джуном)

[Максим]

samegalt, вас понял. Главная мысль: код - это не про оптимизацию строчек, это про "поддержку". Весь кайф редакса в том, что если экшены были хорошо описаны (типы корректные) - то через 3 месяца вы сможете подкрутить код быстро. Согласитесь, есть разница:
- LOAD_USERS_FOR_TABLE_GAMERS (загрузить юзеров в таблицу для геймеров, например с их пк и ником на твиче)
- LOAD_USERS_FOR_PARTY_TONIGHT (загрузить юзеров в страницу встречи, где люди собираются на вечеринку).
Сразу понятно, будучи на экране "вечеринки" - что экшен приехал нужный и что юзеры скорее всего готовые "потусить".

Юзеры разные, страницы разные, цели разные - следовательно все разное (экшены и редьюсеры). Конечно, нужно использовать одинаковый код для запроса по сети, с парой параметров, можно как-то еще оптимизировать копи-пасту. Но вот в случае с редьюсерами и экшенами, обычно, копипаста не косяк, а удобство.

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

Как видите, тут много текста, много тонкостей, но я думаю общую суть передает.

[samegalt]

Максим, Огромное спасибо! Теперь я понял что к чему) Еще один вопрос, если можно. У меня куча экшенов, и соответственно на каждый экшн я должен сделать еще один, который проверяет загружились ли данные, а это еще одна константа и еще одна строка в редьюсере. Нет никакого способа упростить хотя бы проверки?

[Максим]

samegalt, кому-то нравятся middleware (которые сами добавляют REQUEST / SUCCESS в тип экшена), кого-то (например меня) устраивает писать:
- LOAD_USERS_FOR_PARTY_TONIGHT_REQUEST
- LOAD_USERS_FOR_PARTY_TONIGHT_SUCCESS

и хочу вас расстроить) надо не 2, а 3 действия) ведь если вдруг ошибка?)
- LOAD_USERS_FOR_PARTY_TONIGHT_FAILURE

[samegalt]

Максим, если ошибка, пусть смотрят на спиннер;) Вас понял, спасибо еще раз, очень помогли разобраться)

[Максим]

samegalt, нет-нет) нужно не на спинер смотреть, а показывать какой-то контрол, как тут например: https://youtu.be/fsC6r_prek8?t=1h1m15s