Начинаем программировать#2
Conversation
| offersCount: number; | ||
| } | ||
|
|
||
| function App({ offersCount }: AppScreenProps): JSX.Element { |
There was a problem hiding this comment.
JSX.Element - не обязательно
There was a problem hiding this comment.
src/components/app/app.tsx
upd - правильней src/app/app.tsx, потому что app это не переиспользуемый компонент, а отдельная смысловая сущность
There was a problem hiding this comment.
Не исправлено:
Должно быть src/app/app.tsx
There was a problem hiding this comment.
offer-card.tsx
| @@ -0,0 +1,42 @@ | |||
|
|
|||
|
|
|||
| function Card(): JSX.Element { | |||
| @@ -0,0 +1,171 @@ | |||
| function LoginPage(): JSX.Element { | |||
There was a problem hiding this comment.
FavoritesPage
| <li className="places__option" tabIndex={0}>Top rated first</li> | ||
| </ul> | ||
| </form> | ||
| <div className="cities__places-list places__list tabs__content"> |
There was a problem hiding this comment.
[...new Array(offersCount)].map((offer) => <Card />)
Почему такая запись лучше
offersCount в дальнейшем сменится offers у которыйх тип Array<Offer>
offers.map((offer) => <Card offer={offer} />)
| @@ -1,12 +1,23 @@ | |||
| import React from 'react'; | |||
| import ReactDOM from 'react-dom/client'; | |||
| import App from './components/app/app'; | |||
There was a problem hiding this comment.
Надо дирректорию для App объявить с маленькой буквы
Переместить в корень src
И тогда путь будет "./app/app"
There was a problem hiding this comment.
Не исправлено:
Переместить в корень src
И тогда путь будет "./app/app"
| import App from './components/app/app'; | ||
|
|
||
| const Setting = { | ||
| offersCount: 312, |
There was a problem hiding this comment.
Это предупреждение связано с правилом react-refresh/only-export-components: константа Setting названа с заглавной буквы, из-за чего линтер принимает её за компонент.
Можно на него забить - потом будет отдельный файл с константами
| @@ -1,12 +1,23 @@ | |||
| import React from 'react'; | |||
| import ReactDOM from 'react-dom/client'; | |||
| import App from './components/app/app'; | |||
There was a problem hiding this comment.
Не исправлено:
Переместить в корень src
И тогда путь будет "./app/app"
| @@ -0,0 +1,5 @@ | |||
| const setting = { | |||
There was a problem hiding this comment.
Новая ошибка:
Константные объекты с большой буквы (смотри критерий Б9)
https://up.htmlacademy.ru/react-individual/3/criteries#b9
| offersCount: 312, | ||
| } as const; | ||
|
|
||
| export default setting; |
There was a problem hiding this comment.
Новая ошибка:
В данном случае как default не экспортируем
Про то как устроены экспорты и какие когда используются, можно почитать тут: https://learn.javascript.ru/import-export
There was a problem hiding this comment.
Не исправлено:
Должно быть src/app/app.tsx
3 participants
🎓 Начинаем программировать