Комментарии пользователя Андрей Костяев

Есть ли сайты для выполнения кода (html,js, и пр)?

Андрей Костяев @mrdestroii

Плюсую. Всё как в стандартном node.js проекте

Оценить простое задание на react?

Андрей Костяев @mrdestroii

1. Структура проекта ужасная, именно про то, что даже App лежит просто в src, не и т.д., найди нормальный проект, где хорошая структура.
2. Вместо useState лично я бы заюзал useReducer, код получится намного чище.
3. Ну определение компонетов, как по мне для функциональных компонентов лучше делать так:

import React from "react";
import logo from "./logo.png";

const Logo = () => {
  return (
    <div className="logo">
      <img src={logo} alt="" />
    </div>
  );
}

export default Logo

4. Избегать дублирования, для примера инпут чекбокса можно было вынести как отдельный ui компонент.
5. Для таких моментов

className={props.sort === "price" ? "tabs__item active" : "tabs__item"}

лучше юзать либу classnames
6.

const duration = value => {
    // Функция выводит форматированное время полета
    let hour = parseInt(value / 60);
    let minute = value - hour * 60;
    return (
      <span>
        {hour} ч {minute} м
      </span>
    );
  };

названия таких функций желает лучшего, читаемость кода нужно держать на уровне. тут бы лучше подошло renderDuration, ну и можно было вообще мини компонент сделать, куда прокидывали бы value.
7. Я конечно не углублялся в код, но зачем юзать for? даже у ванильного js есть такая штука как array.map и т.д.
Это так, чисто мое мнение, по коду просто бегло пробежался.

P.S. еще почитайте про useCallback.
Ну и много моментов по поводу наименования переменных, констант и т.д.
Например

const getSearchIdUrl = "https://front-test.beta.aviasales.ru/search";

Если какой то сторонний разработчик будет читать код, и увидет, что где то используется эта константа, то исходя из названия он подумает, что это функция, хотя на деле это просто константа, и это может вызвать некоторые трудности, точнее даже не трудности, а сторонний программист потеряет некоторое время на разбор этого момента.

Войдите на сайт