setState компонента у реактивной вьюшки

[vitkarpov]

Вот здесь написана неверно: https://github.com/yandex-ui/noscript-react#setstate-не-вызывает-перерисовку

Если дергать setState компонента, то ns-апдейта не будет и мы никогда не придем в _prepareRender для установки нового состояния _needBeUpdated.

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

[i2r]

Но, поддержав полноценный state, мы рискуем никогда не перейти с реакт-видов на дата-провайдеры.

[vitkarpov]

Ну, это уже больше политический вопрос — перейдем, конечно :)

[Rebulus]

Если я правильно понимаю, новый setState же будет делать

super.prototype.setState.call(this, someNewState);
this.props.view.invalidate();
ns.page.go()

?

[Rebulus]

Иначе концепция немного меняется. Ибо ns.ViewReact одновременно и вьюшка, и компонент.

[Rebulus]

Поидее ns должен знать, что вьюшка стала по сути не валидной.

[vitkarpov]

Если я правильно понимаю, новый setState же будет делать

Не совсем. Здесь не нужен ns-апдейт, именно потому что вьюшка одновременно является и вьюшкой и компонентом.

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

[Rebulus]

Может быть стоит пересмотреть shouldComponentUpdate. Там, где сейчас влияющий флаг проверяется. Например, добавить там проверку, что newState !== previousState?

[Rebulus]

Через какой-нибудь shallowEqual.

[Rebulus]

Что-то раньше эта простая, но хорошая мысль в голову не приходила 😄 Единственное, что в этом случае не очень нравится - это возможность пересечения апдейтов ректа и ns.