Реализовать более умный StreamedState.accept(), который акцептит только уникальные значения

[saturov]

У нас из проекта в проект кочует extension, который фильтрует значения, переданные на accept в StreamedState и акцептит только те, которые отличаются от текущего. Это помогает избежать зацикливания процессов, которое может возникнуть при отсутствии такой проверки.

• Делать это поведением по-умолчанию не нужно, так как в некоторых случаях имеет смысл акцептить любые значения, даже идентичные.
• Также, предлагаю сделать это не в виде extension, а в виде дополнительной функции StreamedState (и всего семейства аналогичные классов).

extension AcceptUnique on StreamedState {
  void acceptUnique([dynamic data]) {
    if (value == data) {
      return;
    }
    accept(data);
  }
}

[artem-zaitsev]

А чем не устраивает distinct на стрим? RX же есть

[saturov]

Насколько я понимаю, в этом случае дублирующее значение физически всё равно будет в стриме, а это иногда не нужно.

[artem-zaitsev]

Это для синхронного получения данных? Но тогда какой смысл?

[artem-zaitsev]

просто кажется distinct вполне решает эту проблему

[pretorean]

а как быть если в StreamedState массив значений?

[saturov]

В этом случае точно такой же массив не будет заакцепчен. Ну а повторяемость элементов массива уже не на уровне работы со стримом проверять нужно, если ты об этом.

[HazarD2311]

Если будем это имплементить, то хотелось бы еще видеть, который все значения экшна будет унифицировать:

Action(distinctUnique: true)

[saturov]

Что это вообще значит, поясни?

[IlyaMax]

Может сделать как предлагает @HazarD2311 и для стейтов? Чтобы не было лишнего метода accept. По-моему не должно быть таких кейсов когда нужно одному state вызвать acceptUnique а потом accept.