Давайте с простого) С чего начинается ревью кода?

Какая цель вообще у ревью кода?

емм, проверить если качество кода нормальное?)

емм, проверить если качество кода нормальное?)

угу) Как насчет того, чтобы как раз с качества кода и начать?

Что вообще такое качество кода?

Я вот только недавно прогаю, как мне понять, что мой код качественный?

читаемость, производительность и минимальное количество зависимостей, которые мешают расширять приложение?

читаемость, производительность и минимальное количество зависимостей, которые мешают расширять приложение?

Ну как вариант. А ошибки синтаксиса ?

относятся к читаемости

отлично) По читаемости код сверху качественный?

относятся к читаемости

вообще нет на самом деле, ну да лдано)

ну и могут мешать расширяемости тоже, если будет в нескольких местах написано по разному, скажем где-то с ошибкой, а где-то нет

Если он ошибку в имени переменной сделает, у него же не будет работать

Или будет не так работать

Там инишлПродуктс кладётся в константу, а где используется?

Зачем кладётся, если Н где не используется, и зачем тогда вообще функция

относятся к читаемости

По рзеузльаттам илссоевадний одонго анлигсйокго унвиертисета, не иеемт занчнеия, в каокм проякде рсапжоолены бкувы в солве. Галовне, чотбы преавя и пслонедяя бквуы блыи на мсете. осатьлыне бкувы мгоут селдовтаь в плоонм бсепордяке, все-рвано ткест чтаитсея без побрелм. Пичрионй эгото ялвятеся то, что мы не чиаетм кдаужю бкуву по отдльенотси, а все солво цлиеком. Так то читаемо))

Но работать не будет :DDD

а, я пропустил слово в синтаксисе, думал про ошибки в наименованиях переменных)

по читаемости более менее ок, не считая 2 похожих функций

ладно) Го по порядку) компонент описан правильно?

Декларация компонента)

наверно да, если это правда тестовый компонент)

да не важно) Тестовый или боевой)

Тут как минимум 2 ошибки уже в описании компонента

В его декларации

с большой буквы?

да, это первая) Если название типа элемента начинается с маленькой буквы, он ссылается на встроенный компонент, к примеру <div> или <span>, что в результате приведёт к тому, что в React.createElement будет передана строка 'div' или 'span'. Типы, начинающиеся с заглавной буквы, такие как <Foo />, компилируются в React.createElement(Foo) и соответствуют компоненту, который был объявлен или импортирован в вашем JavaScript-файле.

Все компоненты с большой буквы)

В целом без деталей - это прям в начале доки по реакту написано. Просто) Что все компоненты должны быть с большой буквы =)

да, но это больше похоже на проверку внимательности 😄

А она так работает) Если знаешь, то найдешь)

Мы и не требуем, чтобы сразу все нашли. Естественно мы ведем собес и направляем)

Просто банальная просьба) Пройдись построчно и найди ошибки по коду построчно.

Без вникания в суть происходящего

Первая часть ревью - проверить, что код работает)

Вот вопрос) Оно сейчас будет работать?

https://codefile.io/f/KGkfL0W4L3ZOuUkbbuKW

Можешь поредактировать, если охота)

не уверен, что initProducts в useState правильно написан, а с другой стороны вроде и правильно)

Поправь как считаешь нужным)

с большой буквы?

Кстати да

Можешь удалить то, что считаешь не нужным) В рамках компонента свобода действий)

На собесе мы просим рассуждать в слух =) Чтобы понимать как мыслит кандидат) Тут с этим сложнее, ну да и ладно)

кажется больше не вижу возможных проблем

они есть) Сейчас будет ошибка падать)

Что возвращает функциональный компонент?

JSX

а сейчас он что возвращает?

Promise?

угу)

Компоненты могут быть асинхронными?

видимо нет :D

угу) Реакт умеет в JSX, но не умеет в промисы)

Это рендер функция и она должна jsx возвращать)

смотрел несколько раз на этот async, думал никогда так не делал, выглядит странно)

да, там дибильный редактор кода -_-

А зачем тебе getProducts и setProducts в зависимостях?

у меня бы линтер жаловался, привычка :D

можно и без этого

линтер на такое не жалуется))

Ни в одном проекте так настроен не был, да и не к чему))

у меня на проекте он жаловался :D причем правило какое-то тупое

Ок) Тогда для чего вообще нужны эти зависимости?

Что вообще такое эффект и как он работает?

от них зависит вызов useEffect, если эти зависимости меняются, то он вызывается

но в данном случае нет разницы, т.к. они не будут меняться

getProducts там точно не нужен, тк он вне компонента и реакт понятия не будет иметь, изменилась функция или нет)

Нифига у вас тут диалог ;D

окей, но изменение функций звучит как-то неправильно

Но в целом сейчас уже будет работать)

даже не хочу предполагать, что в целом функция может измениться)

а можешь на async await переписать?

Чтобы не then был)

ну он тебе вечные варнинги писать будет)

useEffect синхронный

Ну всмысле, что на вход ему нужны синхронная функция)

т.е. useEffect не особо поддерживает async await? или я не так написал?

https://ultimatecourses.com/blog/using-async-await-inside-react-use-effect-hook

А дело не в useeffecte)