shri-2015-org / 666

666 team's chat =>
http://anonym.club
4 stars 3 forks source link

Умные action creator-ы #59

Closed tempname11 closed 9 years ago

tempname11 commented 9 years ago

см. https://github.com/shri-2015-org/666/pull/55#discussion_r39920788

mightyaleksey commented 9 years ago

Предлагаю сделать такую структуру:

Бонусы:

tempname11 commented 9 years ago

А-а-а, отжали issue 🙀

tempname11 commented 9 years ago

@sullenor да, мне нравится. если правильно понимаю, заодно избавимся от мутаций DOM-а а-ля findDOMNode(...).value = '' (для поля ввода)

mightyaleksey commented 9 years ago

@tempname11 Сережа начал работу и хотел довести до ума ее :) И да, избавимся от прямой работы с DOM-нодами :)

Предлагаю, потом еще разок на ревью посмотреть вместе )

mightyaleksey commented 9 years ago

@tempname11 Как вариант, ты можешь попробовать доработать функцию reducer и избавить обработку каждого экшена в виде switch case. А входные данные описать в виде объекта:

{
  [ADD]: addFunction,
  // ...
}
pofigizm commented 9 years ago

{

   // ...

}

+1

tempname11 commented 9 years ago

Закрываю, как не решающую никаких непосредственных проблем тему. Как именно можно переструктурировать код, понятно, и в случае возникшей необходимости я смогу это сделать.