Review

[Mairo07]

Ревью на проект ФильмПоиск

Привет! Мне кажется у тебя классная работа - почти все требования 1 части выполнены.

• Четкая и понятная структура.
• Внешний вид приложения соответствует макету
• Использована типизация. Здорово, что использованы типизированные useAppDispatch и useAppSelector https://github.com/alexkor1303/FilmPoiskGH/blob/26d1b94b2ca85b73f67a4b3e83961562e55f6696/src/hooks/hooks.ts#L5-L6 Из недочетов не влияющих на баллы - UI элементы - не вынесены в отдельные компоненты с возможностью переиспользования.

Базовые требования:

Базовые требования все выполнены — 15 баллов

Функциональные требования:

Шапка Позиционируется липко (стики) — 2 балла.

Комментарий.

https://github.com/alexkor1303/FilmPoiskGH/blob/26d1b94b2ca85b73f67a4b3e83961562e55f6696/src/components/Header/styles.module.css#L4-L5

В твоем варианте заявленный функционал есть - поэтому баллы не снимаю. Но судя по заданию, лучше здесь использовать position: sticky. https://doka.guide/css/position-sticky/ Также по значению z-index у меня есть такая информация : 'Стоит избегать сильно больших чисел. Обычно достаточно значений -1/0/1. Но если надо, чтобы элемент точно-точно был выше всех, то можно указать 10. Использование значений выше - чрезмерно, так делать не нужно'

Авторизация Здесь все требования то ТЗ выполнены — 16 баллов

Реализована страница списка фильмов Здесь также требования выполнены — 14 баллов

Комментарий На главной странице без заполненных фильмов не отрисовывается список фильмов . По макету на главной странице, даже без заполненных фильтров есть список карточек фильмов. Считаю это небольшим багом - баллы не снимаю.

Страница фильма Требования выполнены — 17 баллов

Комментарий Страница фильма открывается только при клике на постер фильма. Мое мнение - т. к. у нас в проекте выделенные карточки фильмов - мне кажется, пользователю было бы понятнее если можно было кликнуть в любое место карточки и перейти на полную страницу фильма.

Общий функционал Требования выполнены — 8 баллов

Стор

• Используется rtk и rtk-query — 2 балла

Комментарий Не используется rtk-query, поэтому минус 2 балла Для запросов нужно было использовать createApi из rtk-query https://redux-toolkit.js.org/rtk-query/api/createApi

• Данные корректно разбиты на модули (пример — авторизация, searchParams из фильтров) — 0

Комментарий https://github.com/alexkor1303/FilmPoiskGH/blob/26d1b94b2ca85b73f67a4b3e83961562e55f6696/src/redux/store.ts#L4-L8 Нет разбиений на модули. Один слайс данных - app

Селекторы написаны оптимально (нет кучи дублирования) — 2 балла.

Миграция на Next

Не выполнена

[alexkor1303]

Спасибо за ревью!🫶