A
ну так мы же не про change detection говорили
Size: a a a
2018 March 27
MI
про то, что выносить это красиво и правильно
A
а про гетеры
Вキ
а если эта логика будет в шаблоне, то не будет дергаться постоянно?
Не будет, если расчеты делаются до того, как приходят в component
MI
*ngIf="heavyCheck(value) && detectProperty(fuck)"
// ну вынести в геттер, это даже хорошо, по сравнению с вариантом выше!
*ngIf="isLoading"
// ну вынести в геттер, это даже хорошо, по сравнению с вариантом выше!
*ngIf="isLoading"
MI
но вот уже как часто, этот геттер будет срабатывать, и что он там делать внутри себя и как долго, это уже другой вопрос
MI
Не будет, если расчеты делаются до того, как приходят в component
ну короче, по максимум кешируйте)
MI
в принципе серебрянная пуля:
- OnPush
- Pure pipe или сразу отсортированные массивы (закешированные)
- При желании отключение зоны и точечное обновление нужных компонентов
- OnPush
- Pure pipe или сразу отсортированные массивы (закешированные)
- При желании отключение зоны и точечное обновление нужных компонентов
A
ой, ребят.. что вы там пишете такое тяжелое? =) у меня вроде большое real-time (slack like) приложение и ребята, которые писали, про onPush вообще не слышали, а проблем с перформансом нет..
MI
ну зависит от прямых рук 😃
A
переписываю потихонечку на ngrx с onPush, но потому что стэйт мэнеджмент проще
MI
у меня в команде 4 джуна, там жестят
DT
так а что, пока не лагает, то можно писать шлак?)
АД
А о чем разговор и спор?
DT
начали с декораторов, закончили на ченчж детекшне)
АД
А геттеры как цепанули?)
DT
да я там с декораторами баловался
Вキ
А о чем разговор и спор?
{ get; private set; }
OnPush, ngzone
OnPush, ngzone
MI
ой, ребят.. что вы там пишете такое тяжелое? =) у меня вроде большое real-time (slack like) приложение и ребята, которые писали, про onPush вообще не слышали, а проблем с перформансом нет..
ну мне кажется перфоманса будет больше как перепишите, посмотрите также примеры про change detection tree, которые я скинул
фактически не выстрелите себе в ногу в будущем, если на ngrx/mobx + onpush - without zone
фактически не выстрелите себе в ногу в будущем, если на ngrx/mobx + onpush - without zone
A
ну да, у меня от eslint в tslint включены
ну наверное это холивар, но на ревью, я все-таки прошу людей писать get/set
ну наверное это холивар, но на ревью, я все-таки прошу людей писать get/set
ну лично меня вот это зацепило, я не понял зачем каждому гетеру сетер 😊