ap1726 / practicum_react Goto Github PK

          Можно лучше

Необходимо так же подсвечивать активный таб в хедере

Originally posted by @VovaGayduk42 in #8 (comment)

• Скрин https://disk.yandex.ru/i/3aTBu5m50n19kw Нужно правильно центрировать контент на странице, чтобы не деформировалась верстка, если экран сужается. Либо скролл нужно показывать, либо сдвигать влево правильно до упора, а потом уже скролл горизонтальный.

Originally posted by @gennady-bars in #11 (review)

          **Можно лучше**

Можно сделать универсальный кастомный хук для контроля любого количества инпутов в любых формах:

export function useForm(inputValues={}) {
  const [values, setValues] = useState(inputValues);

  const handleChange = (event) => {
    const {value, name} = event.target;
    setValues({...values, [name]: value});
  };
  return {values, handleChange, setValues};
}

Этот код помещают в отдельный файл useForm.js в папке hooks и импортируют функцию туда, где нужно контролировать инпуты

И Вам не нужно будет теперь вручную создавать функции обработки инпутов и т д. Все будет в одной строчке кода:

  const {values, handleChange, setValues} = useForm({});

Originally posted by @gennady-bars in #11 (comment)

          **Можно лучше**

В проекте уже есть модалка, ошибку можно показывать в ней.

Originally posted by @andryxxa93 in #4 (comment)

          Здравствуйте. (**Нужно развернуть общий комментарий ↓**)

Посмотрите на гитхабе все комментарии к коду (нужно прокрутить вниз страницу там)

Работа проделана огромная:

• Отлично типизировали хуки useSelector и useDispatch
• Отлично типизировали стейт, диспатч
• Отлично, что закрываете соединение сокетов при покидании страниц
• Хорошо сделали socketMiddleware

но есть некоторые недочеты:

Можно лучше

• Вёрстка не должна ломаться, если открыть консоль в бразуере https://disk.yandex.ru/i/i7pn3S5r4NeOSA
• Желательно очищать конструктор после успешного получения номера заказа с сервера в блоке then, чтобы пользователь мог следующий заказ сделать, не удаляя старые ингредиенты
• лучше не давать нажать на кнопку Оформить заказ, пока пользователь не добавил булку в заказ.
• лучше отображать лоадер (загрузку) при ожидании ответа от сервера с номером заказа
• Вам нужно сделать верстку по макету. Это презентационный проект, чтобы показать навыки и верстки тоже

Originally posted by @gennady-bars in #14 (review)

          Можно лучше

Не совсем понимаю зачем делать сложность N^2
Попробуйте придумать другое решение, если это возможно

Originally posted by @VovaGayduk42 in #8 (comment)

          **Можно лучше**

Если вы добавили localStorage и cookie то их после каждого теста нужно удалять

  afterEach(function () {
     cy.clearLocalStorage();
     cy.clearCookies();
  })

Originally posted by @andryxxa93 in #16 (comment)

          Можно лучше: Вычисления и преобразование данных (на стороне компонента), полученных из селектора (хук useSelector), можно делать в селекторе, используя библиотеку reselect. Данная библиотека нужна для создания мемоизированных, пригодных для компоновки селекторных функций. Примеры взаимодействия reselect с useSelector можно посмотреть здесь: https://react-redux.js.org/api/hooks#useselector-examples

Originally posted by @vadmas in #5 (comment)

          **Можно лучше**

Удобно когда для одного теста используется один expect. Так если тест упадёт будет проще определить из-за чего именно.

Originally posted by @andryxxa93 in #16 (comment)