shevchuknine
commented
2 years ago список актеров и ревью вываливается за высоту страницы https://github.com/kristof22122/Ex2-React-Program1/blob/17d0cc561dd0b56e1b96b52c2d0e4be7ff6ed200/movie-reviews-6-1/src/components/CustomPagination/CustomPagination.js#L20 похожие действия со страницей имеет смысл выполнять в родительском копмоненте, паганция выступает просто формой, которая изменяет страницу. а в родительском компоненте уже имеет смысл сделать скролл вверх на основании изменения страницы https://github.com/kristof22122/Ex2-React-Program1/blob/17d0cc561dd0b56e1b96b52c2d0e4be7ff6ed200/movie-reviews-6-1/src/components/MovieCastInfo/MovieCastInfo.js#L53 запросы предпочтительнее разделять не по переиспользованию кода, а логически. т.е. запросы на актеров и ревью должны быть разными, несмотря на то, что имеют много общего кода. переиспользуй колбек обработки данных для этих запросов, чтобы уменьшить дублирование https://github.com/kristof22122/Ex2-React-Program1/blob/17d0cc561dd0b56e1b96b52c2d0e4be7ff6ed200/movie-reviews-6-1/src/components/MovieReviewInfo/MovieReviewInfo.js#L56 потенциальное место для ошибки, если в результате придет не массив https://github.com/kristof22122/Ex2-React-Program1/blob/17d0cc561dd0b56e1b96b52c2d0e4be7ff6ed200/movie-reviews-6-1/src/components/MovieAllInfo/MovieAllInfo.js#L9-L10 зачем приводить к числу? используй деструктуризацию для доступа к id
в URL перед идентификатором не должно быть двоеточия. в задании таким образом указан динамический параметр
:idотдельные страницы должны выглядеть как полноценные страницы, а не как попапы с крестиком https://github.com/kristof22122/Ex2-React-Program1/blob/561ba27c2e61abf880bb7b3ba960080e77ebcac0/movie-reviews-6-1/src/api.js#L8 асинхронный код строим через промисы, baseURL этоhttps://api.themoviedb.org/3, все остальное конкретный url в каждом запросе, параметры через params, единый обработчик ответа (чтобы не дублировать вездеres.data.results). ты можешь изначально создать апи клиент через axios.create, передать туда baseUrl, а потом дополнять его в отдельных запросах идея динамического роутинга в том, что ты строишь path с параметромpath={"/movies/:id"}. когда выполняешь переход на эту страницу черезLink, то определяешь конкретный id для подстановки. в дальнейшем идентификатор выбранного фильма ты уже читаешь из URL (useParams в react-router). в таком случае состояниеselectMovieIdтебе в принципе не нужно. редирект нужно реализовать, но не средствами