alexpol19 / adminui Goto Github PK

Дата приводится к string вручную, вместо того чтобы использовать какую-нибудь библиотеку для работы с датой

export function formatDate(dateStr){
    const initDate = new Date(dateStr);
    const date = (Number(initDate.getMonth())+1) + '/' + initDate.getDate()   + '/' + initDate.getFullYear()
    return date
}

Много мест, где можно сразу вернуть результат, без создания переменных:

Пока не нажать на "Cauta", форма пуста:

Не понятно для чего были добавлены следующие компоненты: UseDemoConfig.js, MyChartCustom.js, ChartDemo1/MyChart.js. Кода много, но нигде не используется

Таймер для запуска update графика со счетами, запускается каждый раз при выборе интервала дат. Следовательно, если выбрать 10 раз период времени из dataPicker, то будут запущены параллельно 10 timeout которые обновляют форму каждые 10 секунд:

Вместо того чтобы рекурсивно вызывать setTimeout, можно было просто при запуске приложения запустить setInterval, который вызывает указанную функцию с необходимой периодичностью.
Так как используется Redux, можно сохранить ключ запущенного setInterval, чтобы потом можно было остановить его.

Чтобы разделить число на разряды используется рекурсия:

function numberFormat(num) {
    if (typeof num !== "string") {
      return numberFormat(num.toString())
    } else {
      if (num.length < 4) {
        return num
      } else {
        return numberFormat(num.slice(0, num.length - 3)) + "," + num.slice(num.length - 3)
      }
    }
  }

Не используется маршрутизация React для навигации. Вместо этого запоминается индекс выбранного раздела в Redux и отображается соответствующая страницу в зависимости от этого:

Из-за этого не работает браузерная навигация. URL при переходе на страницы остается неизменным и если нажать "Назад" в браузере, то пользователя просто выкидывает с сайта. Иконка для меню также определяется по индексу

Для определения названия месяца, определена константа и название определяется по индексу месяца из даты: