search

yandex-ui / noscript

Noscript: JavaScript MVC Framework for building SPA
http://yandex-ui.github.io/noscript/
MIT License
34 stars 22 forks source link

[ns.Update] Оптимизировал цепочку промисов #485

Closed doochik closed 9 years ago

doochik commented 9 years ago

Станет немного быстрее. Убрал возврат промисов из #_insertNodes и #_updateDOM, он все равно статичный Из #_render убрал цепочки для запоминания async-промисов, сделав их синхронными.

chestozo commented 9 years ago

Я тут давно думаю про то, чтобы скрестить noscript и react. Для этого нужно, чтобы шаг "рендеринг" был асинхронный (потому что в react он асинхронный). Это я к тому, что как вы смотрите на то, чтобы сделать этот шаг асинхронным?

doochik commented 9 years ago

Для начала тогда надо сделать шаг под названием "обновляем то, что отрендерили".

У нас раньше была проблема, что updateTree + yate работали асинхронно с updateHTML, в итоге получалось, что отрендерить могли одно, в момент передачи вызова что-то поменялось и updateHTML делает уже что-то другое. Из-за этой проблемы собственно updateTree + yate + updateHTML стали синхронными

chestozo commented 9 years ago

Была какая-то issue про это? Я что-то смутно припоминаю, но точно не вспомнил. Это про то, что модель уже могла поменяться на момент updateHTML? Синхронность конечно много что может решить, но, кажется, мы теряем гибкость.

edoroshenko commented 9 years ago

:+1:

doochik commented 9 years ago

@chestozo была, я это фиксил

chestozo commented 9 years ago

Т.е. вы не считаете, что мы гибкость теряем?

edoroshenko commented 9 years ago

Роман, ты знаешь моё мнение по этому вопросу

chestozo commented 9 years ago

Роман, ты знаешь моё мнение по этому вопросу

на тему реакта знаю на тему гибкости нет

edoroshenko commented 9 years ago

Я не вижу смысла бороться за гибкость, направленную на решение задач, которые кажутся мне странными. Моё мнение.

chestozo commented 9 years ago

вот что я думаю про это )

edoroshenko commented 9 years ago

Гениальная цитата, только кажется она противоречит твоей позиции в этой дискуссии, нет?

chestozo commented 9 years ago

Так это сарказм. Нельзя решать проблемы производительности за счёт того, чтобы урезать фичи. Зачем эти фичи тогда было делать?

alexeyten commented 9 years ago

Так может быть и незачем, просто because we can

edoroshenko commented 9 years ago

+1 к @alexeyten

chestozo commented 9 years ago

Рендерить react виды мы научились без того, чтобы залезать в механизм update. Так что я не против предлагаемой оптимизации.