ДЗ #3 Параметры для дочерних компонентов и событие добавления в корзину #50
Conversation
vakulovv
changed the title
vakulovv
changed the title
|
|
||
| onProductBuy(event: Event) { | ||
| @Output() readonly buyClick: EventEmitter<string> = new EventEmitter<string>(); |
There was a problem hiding this comment.
Не кажется ли тебе, что получается тавтология?)
...: EventEmitter<string> = new EventEmitter<string>();Есть ли в этом сакральный смысл?
Мне кажется, что нет, тут отлично сработала бы неявная типизация свойства, а за счет дженерика можно четко указать тип данных передающихся в Output:
@Output() readonly productBuy = new EventEmitter<string>();Эта запись мне видится лаконичней, без тавтологии, и не менее типизированной, что важно.
Что думаешь по данному поводу?
| @@ -1,13 +1,17 @@ | |||
| import {Component} from '@angular/core'; | |||
| import {IProduct} from 'src/app/shared/products/product.interface'; | |||
| import {productsMock} from 'src/app/shared/products/products.mock'; | |||
There was a problem hiding this comment.
Почему использовал абсолютные пути?
|
|
||
| @Component({ | ||
| selector: 'app-products-list', | ||
| templateUrl: './products-list.component.html', | ||
| styleUrls: ['./products-list.component.css'], | ||
| }) | ||
| export class ProductsListComponent { | ||
| onCardClick() { | ||
| products: IProduct[] = productsMock; |
There was a problem hiding this comment.
Я бы тут рекомендовал убрать явную типизацию, т.к. в ней нет особого смысла: она ни читаемости особой не добавляет, ни типизацию не ужесточает, т.к. через механизм неявной типизации - свойство products сможет отлично унаследовать тип константы productsMock
No description provided.