Уменьшите количество итераций JavaScript

Сейчас я разрабатываю программу таймлайна с помощью javascript. Когда я щелкаю каждый класс, он должен быть раскрашен на временную шкалу с 2 блоками, потому что каждый класс - это 2-часовой класс.

каждая таблица временной шкалы имеет идентификатор (например, первый класс понедельника имеет = "mon1", а первый класс второго дня имеет значение "fri2"

Вот мой код. и это хорошо работает. но я хочу знать, как уменьшить код.

let timelineColor = getRamdomColor();
let clickDay;
let clickDay1;

for (i = 0; i < dayArray.length; i++) {
    if (dayArray[i] === 'monday') {
        clickDay = 'mon' + timeArray[i];
        let table = document.getElementById(clickDay);
        table.style.backgroundColor = timelineColor;
        clickDay1 = 'mon' + (Number(timeArray[i]) + 1);
        console.info(clickDay1);
        let table1 = document.getElementById(clickDay1);
        table1.style.backgroundColor = timelineColor;
    } else if (dayArray[i] === 'tuesday') {
        clickDay = 'tue' + timeArray[i];
        let table = document.getElementById(clickDay);
        table.style.backgroundColor = timelineColor;
        clickDay1 = 'tue' + (Number(timeArray[i]) + 1);
        let table1 = document.getElementById(clickDay1);
        table1.style.backgroundColor = timelineColor;
    } else if (dayArray[i] === 'wednesday') {
        clickDay = 'wed' + timeArray[i];
        let table = document.getElementById(clickDay);
        table.style.backgroundColor = timelineColor;
        clickDay1 = 'wed' + (Number(timeArray[i]) + 1);
        let table1 = document.getElementById(clickDay1);
        table1.style.backgroundColor = timelineColor;
    } else if (dayArray[i] === 'thursday') {
        clickDay = 'tur' + timeArray[i];
        let table = document.getElementById(clickDay);
        table.style.backgroundColor = timelineColor;
        clickDay1 = 'tur' + (Number(timeArray[i]) + 1);
        let table1 = document.getElementById(clickDay1);
        table1.style.backgroundColor = timelineColor;
    } else if (dayArray[i] === 'friday') {
        clickDay = 'fri' + timeArray[i];
        let table = document.getElementById(clickDay);
        table.style.backgroundColor = timelineColor;
        clickDay1 = 'fri' + (Number(timeArray[i]) + 1);
        let table1 = document.getElementById(clickDay1);
        table1.style.backgroundColor = timelineColor;
    }
}

Уменьшите количество итераций JavaScript

Можете ли вы предоставить свой html? это даст мне лучшее представление о картине в целом. Определенным улучшением будет переключение с блоков if на оператор switch. Хотя вы могли бы поместить обе таблицы в массив const tables = [document.getElementById('fri' + timeArray[i]), document.getElementById('fri' + (Number(timeArray[i]) + 1))];, тогда для изменения цвета tables[0].style.backgroundColor = timelineColor; tables[1].style.backgroundColor = timelineColor;

— 14.12.2018 02:51

javascript loops iterator

14.12.2018 02:41

Поведение ключевого слова "this" в стрелочной функции в сравнении с нормальной функцией

В JavaScript одним из самых запутанных понятий является поведение ключевого слова "this" в стрелочной и обычной функциях.

Концепция локализации и ее применение в приложениях React ⚡️

Локализация - это процесс адаптации приложения к различным языкам и культурным требованиям. Это позволяет пользователям получить опыт, соответствующий...

Улучшение производительности загрузки с помощью Google Tag Manager и атрибута Defer

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

Безумие обратных вызовов в javascript [JS]

Здравствуйте! Юный падаван 🚀. Присоединяйся ко мне, чтобы разобраться в одной из самых запутанных концепций, когда вы начинаете изучать мир...

Система управления парковками с использованием HTML, CSS и JavaScript

Веб-сайт по управлению парковками был создан с использованием HTML, CSS и JavaScript. Это простой сайт, ничего вычурного. Основная цель -...

JavaScript Вопросы с множественным выбором и ответы

Если вы ищете платформу, которая предоставляет вам бесплатный тест JavaScript MCQ (Multiple Choice Questions With Answers) для оценки ваших знаний,...

Перейти к ответу Данный вопрос помечен как решенный

Ответы 2

Вам вообще не нужны if-операторы. Создайте карту, которая связывает ваше полное название дня с короткими названиями дней, и найдите ее:

let timelineColor = getRamdomColor();
let clickDay;
let clickDay1;

var dayMap = {
    'monday'   : 'mon',
    'tuesday'  : 'tue',
    'wednesday': 'wed',
    'thursday' : 'tur',
    'friday'   : 'fri'
};

for (var i = 0; i < dayArray.length; i++) {
    let shortDayName = dayMap[dayArray[i]];
    clickDay = shortDayName + timeArray[i];
    let table = document.getElementById(clickDay);
    table.style.backgroundColor = timelineColor;
    clickDay1 = shortDayName + (Number(timeArray[i]) + 1);
    console.info(clickDay1);
    let table1 = document.getElementById(clickDay1);
    table1.style.backgroundColor = timelineColor;
}

14.12.2018 02:47

Ответ принят как подходящий

Вы можете использовать оператор switch вместо нескольких операторов if-elseif:

let timelineColor = getRamdomColor();

for (i = 0; i < dayArray.length; i++){
  let day;

  switch (dayArray[i]){
    case 'monday': day = 'mon'; break;
    case 'tuesday': day = 'tue'; break;
    case 'wednesday': day = 'wed'; break;
    case 'thursday': day = 'tur'; break;
    case 'friday': day = 'fri'; break;
  }

  // I moved these variables into the loop too
  let clickDay = day + timeArray[i];
  let clickDay1 = day + (Number(timeArray[i]) + 1);

  let table = document.getElementById(clickDay);
  let table1 = document.getElementById(clickDay1);

  table.style.backgroundColor = timelineColor;
  table1.style.backgroundColor = timelineColor;
}

14.12.2018 02:54

Другие вопросы по теме

Как перебирать массив по одной строке за раз?

Как найти самую большую группу чисел в массиве и вернуть их отдельно в javascript?

Я не понимаю, как рассчитывается временная сложность этого алгоритма

Удалить изображение заполнителя категории продукта в Woocommerce

Как динамически вызывать поля формы html в vbscript в цикле

Перебирайте веб-элементы с помощью Selenium и Robot Framework

Несколько курсоров MySQL в процедуре

R: удаление строк в фрейме данных путем индексации

Динамический выбор конкретной ячейки с помощью цикла

Как как можно больше раз заполнить n таблиц содержимым строки?

Уменьшите количество итераций JavaScript

Ответы 2

Другие вопросы по теме

Похожие вопросы