ДЗ по лекции #11 #117
ДЗ по лекции #11 #117
Conversation
| priceRange: { | ||
| min: number; | ||
| max: number; | ||
| name?: string; |
There was a problem hiding this comment.
Немного духоты по семантике от Егора: судя по коду - поля не опциональные - они будут иметь либо указанное в них значение, либо undefinde, но они точно будут определены в объекте. По этому корректней указать name: string | undefined, если я все правильно понял
There was a problem hiding this comment.
Привет! Не смог разобраться, почему поля не могут быть опциальными. Ведь если юзер вручную отредактирует URL в браузере с параметрами и удалит часть параметров или например перейдет на путь /products-list без параметров, то объект filter в FilterComponent не будет иметь какое-то поле или будет вообще пустым.
| } | ||
|
|
||
| updateFilterForm() { | ||
| this.updateBrandsControl(); |
There was a problem hiding this comment.
Кажется, что в каком то кейсе мы будем вызывать updateBrandsControl за один ngOnChanges несколько раз, лучше в таком случае немного поменять условие в ngOnChnges:
ngOnChanges({brands, filter}: SimpleChanges) {
if (brands || filter) {
this.updateBrandsControl();
}А в updateFilterForm updateBrandsControl не вызывать
Кстати, а почему метод решил сделать публичным?
There was a problem hiding this comment.
Спасибо! Понял, что ошибся в логике. Исправил. Метод сделал публичным не специально, забыл указать private.
| map(filterFormValues => { | ||
| return this.processFilterFormValues(filterFormValues); | ||
| }), |
There was a problem hiding this comment.
| map(filterFormValues => { | |
| return this.processFilterFormValues(filterFormValues); | |
| }), | |
| map(filterFormValues => this.processFilterFormValues(filterFormValues)), |
| const productsFilter = { | ||
| ...filterFormValues, | ||
| brands: sanitizedBrands, | ||
| } as IProductsFilter; | ||
|
|
||
| return productsFilter; |
There was a problem hiding this comment.
| const productsFilter = { | |
| ...filterFormValues, | |
| brands: sanitizedBrands, | |
| } as IProductsFilter; | |
| return productsFilter; | |
| return productsFilter = { | |
| ...filterFormValues, | |
| brands: sanitizedBrands, | |
| } as IProductsFilter; |
| trackById(_index: number, {_id}: IProduct) { | ||
| return _id; | ||
| } | ||
|
|
||
| listenQueryParamsChange() { | ||
| this.activatedRoute.queryParams.subscribe(params => { |
There was a problem hiding this comment.
Почему метод публичный? И почему не захотел использовать AsyncPipe?)
There was a problem hiding this comment.
По AsyncPipe у меня как-то отложилось в голове, что | async лучше использовать для вывода каких-то данных в компоненте, а не для передачи между компонентами, т.е. я в принципе, изначально и не рассматривал такой вариант. Переделал. Думаю, лучше с async. Правда теперь у меня сомнения, не слишком ли громоздким получается шаблон компонента из-за дополнительного ng-container'а.
Публичным метод сделал опять же не специально, забыл. Исправил.
|
|
||
| updateProductsFilter(productsFilter: IProductsFilter) { | ||
| this.router.navigate([''], { | ||
| relativeTo: this.activatedRoute, |
No description provided.