VS
Size: a a a
2020 June 16
Вキ
приватные станут приватными, когда ты их такими опишешь, а с публичными ничего не меняется
Явное написание красивее и ровнее
E
Самое смешное, что прошлые разрабы сделали на каждый пук общий компонент, а вот то, что в 376 местах у нас используется конструкция:
ngOnChanges(inputs) {
if (inputs.accounts && inputs.accounts.currentValue && inputs.accounts.currentValue !== inputs.accounts.previousValue) {
А сделать по человечески (на скрине) догадалась только я. Теперь это выглядит просто вот так:ngOnChanges(changes: SimpleChanges): void {
if (isOnChanges(changes.document)) {>inputs.accounts && inputs.accounts.currentValue && inputs.accounts.currentValue !== inputs.accounts.previousValue
accounts - это массив?)
accounts - это массив?)
Вキ
а лучше вообще избегать onChanges
А что вместо?
E
Самое смешное, что прошлые разрабы сделали на каждый пук общий компонент, а вот то, что в 376 местах у нас используется конструкция:
ngOnChanges(inputs) {
if (inputs.accounts && inputs.accounts.currentValue && inputs.accounts.currentValue !== inputs.accounts.previousValue) {
А сделать по человечески (на скрине) догадалась только я. Теперь это выглядит просто вот так:ngOnChanges(changes: SimpleChanges): void {
if (isOnChanges(changes.document)) {шо за жесть вообще)
AH
а лучше вообще избегать onChanges
каким мне образом их избежать?) например у меня через директиву берется список каких-нибудь чего-нибудь. Я передаю его в дочерний компонент, а в дочернем компоненте надо с этим списком сделать так this.dataSource.connect().next(this.data)
VS
>inputs.accounts && inputs.accounts.currentValue && inputs.accounts.currentValue !== inputs.accounts.previousValue
accounts - это массив?)
accounts - это массив?)
скорее всего сабджект
E
каким мне образом их избежать?) например у меня через директиву берется список каких-нибудь чего-нибудь. Я передаю его в дочерний компонент, а в дочернем компоненте надо с этим списком сделать так this.dataSource.connect().next(this.data)
сделать @ObservableInput()
AH
ЭП
Явное написание красивее и ровнее
ну для меня это уже наверно больше дело привычки. А еще многие создают компонент ставят везде паблики, а на онИнит не добавляют и уже не ровнее😁
AS
А что вместо?
не то чтобы "вместо". Постоянное использование этого хука как правило следствие плохого разбиение на компоненты и кривой работы с rxjs
Типа например фильтр данных при входе, хотя можно было сделать пайпом
Типа например фильтр данных при входе, хотя можно было сделать пайпом
E
ну для меня это уже наверно больше дело привычки. А еще многие создают компонент ставят везде паблики, а на онИнит не добавляют и уже не ровнее😁
у меня есть модифицированное tslint-правило, которое заставляет проставлять модификаторы у всех методов, кроме хуков ^^
N👩
не то чтобы "вместо". Постоянное использование этого хука как правило следствие плохого разбиение на компоненты и кривой работы с rxjs
Типа например фильтр данных при входе, хотя можно было сделать пайпом
Типа например фильтр данных при входе, хотя можно было сделать пайпом
иногда надо
AS
иногда надо
не спорю, но не так часто
N👩
не спорю, но не так часто
ну вот и все тогда
N👩
давай робототь
ЭП
у меня есть модифицированное tslint-правило, которое заставляет проставлять модификаторы у всех методов, кроме хуков ^^
но я тогда не понимаю, в чем логика, тут ставим а тут нет
E
но я тогда не понимаю, в чем логика, тут ставим а тут нет
я за то чтобы ставить везде)
AH
не то чтобы "вместо". Постоянное использование этого хука как правило следствие плохого разбиение на компоненты и кривой работы с rxjs
Типа например фильтр данных при входе, хотя можно было сделать пайпом
Типа например фильтр данных при входе, хотя можно было сделать пайпом
кстати для некоторых моментов это идея, у меня есть места, где просто идет "обработка" входящих данных
ЭП
