Попался на ревью код примерно следующего содержания: searchResult$: Observable; @Input() filters$: Observable; constructor(private

Question

Попался на ревью код примерно следующего содержания: searchResult$: Observable; @Input() filters$: Observable; constructor(private

dataService: DataService) {}

ngOnInit() {
this.searchResult$ = combineLatest(
this.dataService.state$,
this.filters$,
).map(filterData);
}

Ясно, что так делать не стоит, хочу предложить переписать таким образом:

searchResult$: Observable;
@Input() filters;
private filters$ = new Subject();

constructor(private dataService: DataService) {
this.searchResult$ = combineLatest(
this.dataService.state$,
this.filters$,
).map(filterData);
}

ngOnChanges(changes) {
const filters = changes.filters;
if (filters && filters.previousValue !== filters.currentValue) {
this.filters$.next(filters.currentValue);
}
}

Есть у кого-нибудь идеи получше?
Спасибо.

#angular #frontend #javascript #programming #russian

0

07.03.2019

1 ответов

24 просмотра

Arthur Bobrovskih · Accepted Answer

Arthur Bobrovskih

может быть ngOnChanges заменить на @Input() set filters(val) { this.filters$.next(val); }

0

07.03.2019

169 похожих чатов

Попался на ревью код примерно следующего содержания: searchResult$: Observable; @Input() filters$: Observable; constructor(private

1 ответов

Похожие вопросы