asb_cloud_front/CODE_STANDART.md

12 KiB
Raw Permalink Blame History

1. Общие положения

  1. Все несамостоятельные компоненты должны быть написаны на TypeScript. Для самостоятельных компонентов (использующихся как страницы) (далее страницы) допускается использование JavaScript для ускорения написания;

1.1. Файловая структура проекта

  1. Компоненты должны распределяться по директориям в соответствии со своим назначением:
    • src/context - Для контекстов приложения;
    • src/components - Для несамостоятельных компонентов, применяющихся многократно;
    • src/pages - Для страниц и компонентов, использующихся исключительно в единственном экземпляре;
    • src/images - Для компонентов-изображений.
  2. Если страница описывается 1 файлом она должна именоваться в соответствии с содержимым, в ином случае должна быть создана директория с соответствующим названием, внутри которой будут находиться файлы страницы. Основной файл в таком случае должен быть переименован в index.jsx;
  3. Файлы именуются в соответствии с таблицей:
    Тип содержимого файла Расширение Стиль именования
    Компонент или страница jsx/tsx PascalCase
    Файл стилей css/less snake_case
    Вспомогательные методы или константы js/ts snake_case
    Описательные документы md SCREAMING_SNAKE_CASE

1.2. Стилизация кода

  1. Все строки должны по возможности описываться одинарными кавычками или при необходимости обратными:
        const name = 'world'
        const msg = 'Hello, \'' + name + '\'!'
        const toPrint = `Message: ${msg}`
    
  2. Все переменные по возможности должны инициализироваться как const, применение var не допускается;
  3. Переменные именуются в соответствии с таблицей:
    Тип переменной Стиль именования
    Метод, переменная camelCase
    Константы SCREAMING_SNAKE_CASE
    Компонент PascalCase

1.3. Импортирование / Экспортирование

  1. Импортированные файлы (в том числе lazy import) необходимо указывать в самом верху документа в следующем порядке с разделением пустой строкой:
    1. Внешние зависимости (react, antd, webpack и т.д.);
    2. Локальные компоненты по порядку:
      1. Контексты (@asb/context);
      2. Компоненты (@components/Table);
      3. Вспомогательные методы (@utils);
      4. Сервисы API (@api).
    3. Изображения и компоненты-изображения (@images);
    4. Стили (@styles);
    5. Lazy import (const page = React.lazy(() => import('./page'))).
  2. При импорте локальных файлов стоит пользоваться alias'ами:
    Путь Alias
    src/components @components
    src/context @asb/context
    src/images @images
    src/pages @pages
    src/services/api @api
    src/styles @styles
    src/utils @utils
  3. По возможности импортировать из пакетов и файлов только использующиеся сущности:
    // вместо
    import React from 'react'
    const page: React.ReactNode = React.lazy(() => import (...))
    
    // стоит использовать
    import { lazy, ReactNode } from 'react'
    const page: ReactNode = lazy(() => import (...))
    

1.4. Работа с репозиторием

1.4.1. Подготовка к публикации работы по заданию

При получений задания необходимо создать для неё ветку, наследуемую от dev. Ветка должна именоваться в kebab-case и иметь префикс соответствующий типу задачи:

  • "feature/" - для нового функционала или визуала;
  • "fix/" - для багов и любых исправлений.

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

Далее необходимо создать pull request на ветку dev от новосозданной и сразу отметить его как WIP. При завершении задания метку WIP необходимо снять.

1.4.2 Оформление коммита

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

    * На странице "Мониторинг" и "Архив" сокращено колличество запросов;
    * Страница "Сообщения" удалена.

2. JS

  1. Методы, константы и переменные документируются в соответствии с JSDoc;
  2. При документации страниц необходимо указать её название, краткое описание и описание получаемых параметров:
    import { memo } from 'react'
    
    import LoaderPortal from '@components/LoaderPortal'
    
    /**
     * Тестовая страница
     *
     * @description Данная страница не имеет смысла и просто выводит переданное название и контент
     * @param title - Название страницы
     * @param content - Контент страницы
     * @param loading - Отображать ли оверлей загрузки над блоком страницы
     */
    export const TestPage = memo(({ title, content, loading }) => (
        <LoaderPortal show={loading}>
            <div className={'dd-test-page'}>
                <div className={'dd-test-page-title'}>{title}</div>
                <div className={'dd-test-page-content'}>{content}</div>
            </div>
        </LoaderPortal>
    ))
    
    export default TestPage
    

3. TS

  1. Методы, константы и переменные документируются в соответствии с TSDoc;
  2. При документации компонентов необходимо указать их название, краткое описание, а также описать параметры в типе:
    import { memo } from 'react'
    
    import LoaderPortal from '@components/LoaderPortal'
    
    export type TestPageProps = {
        /** Название страницы */
        title: ReactNode
        /** Контент страницы */
        content: ReactNode
        /** Отображать ли оверлей загрузки над блоком страницы */
        loading: boolean
    }
    
    /**
     * Тестовая страница
     *
     * @description Данная страница не имеет смысла и просто выводит переданное название и контент
     */
    export const TestPage = memo<TestPageProps>(({ title, content, loading }) => (
        <LoaderPortal show={loading}>
            <div className={'dd-test-page'}>
                <div className={'dd-test-page-title'}>{title}</div>
                <div className={'dd-test-page-content'}>{content}</div>
            </div>
        </LoaderPortal>
    ))
    
    export default TestPage
    
  3. Использование any в типах допустимо только, если значение используется только в параметрах компонентов, обозначенных типом any. Если метод предполагает работу с разными типами значений стоит описать его как обобщённый.

4. JSX/TSX

4.1. Стилизация кода

  1. Все указываемые к компоненту параметры должны быть обёрнуты в фигурные скобки, кроме параметров флагов со значением true:
    <Button disabled title={'Hello, world!'} type={'ghost'}>Click me!</Button>
    
  2. Если описание параметров компонента не укладывается в ширину в 120 строк стоит перенести их в соответствии с шаблоном:
    <Button
        disabled
        title={'Hello, world!'}
        type={'ghost'}
    >
        Click me!
    </Button>
    
  3. Если JSX код передаётся как значение стоит обернуть его в круглые скобки:
        const a = (
            <Button disabled title={'Hello, world!'} type={'ghost'}>Click me!</Button>
        )
    

4.2. Логика поведения

  1. Не допускается создание значений ссылочных типов в области рендера. Они должны быть вынесены в переменные или константы;
  2. Не допускается создание переменных в функциональных компонентов без использования хуков useMemo/useCallback/useState;
  3. Если переменные или методы не имеют зависимостей и не вызывают методы, доступные исключительно внутри компонента, они должны быть вынесены выше кода компонента.

5. LESS

  1. Использование id должно быть сведено к минимуму;
  2. Все классы именуются с префиксом компании "dd-";
  3. Слова в классах разделяются тире ("-");
  4. Файлы именуются в соответствии с компонентом, к которому относятся;
  5. В одном файле описываются стили либо к конкретному компоненту, либо к странице;
  6. Файл со стилями должен подключаться не более чем к одному компоненту (странице);
  7. Файлы поделены на директории виду компонента, к которому применяются стили:
    • styles/components - для компонентов, не использующихся самостоятельно;
    • styles/pages - для компонентов, использующихся как страница;
    • styles/widgets - для компонентов, применяющихся как виджеты в дашбордах.