search

VasilyKonnov / mercury-tz-react

0 stars 0 forks source link

alt и src, type="button" — не лучший набор свойств для компонента UserProfile #11

Open akuznetsov-md opened 5 years ago

akuznetsov-md commented 5 years ago

Как у профиля пользователя может быть тип “кнопка”?

VasilyKonnov commented 5 years ago

Тип "кнопка" у профиля пользователя лишний, убрал его. Свойства alt и src переименовал в altAvatar и srcAvatar, если я правильно понял названия свойств были слишком абстрактные, если нужно иначе передавать свойства, подскажи как правильно делать, пожалуйста)

akuznetsov-md commented 5 years ago

UserProfile — это некоторый компонент, который выводить данные о пользователе. Значит, чтобы сделать его рабочим, нужно передать данные пользователя.

Чтобы оставить пространство для маневра, я обычно передаю не каждое свойство по отдельности, а объект с некотрым набором свойств:

<UserProfile user={user} />

UserProfile это некоторая абстракция. Абстракции нужны для упрощения кода. У компонентов должен быть минимально необходимый набор свойств. Чем меньше свойств — тем проще абстракция, тем проще код, тем легче его поддерживать.

VasilyKonnov commented 5 years ago

Я понял, спасибо! Применил это