Элегантный способ переключения назад и принудительного включения нескольких элементов css

У меня есть 2 кнопки с идентификаторами «видео-кнопка» и «фото-кнопка». При нажатии каждой кнопки мне нужно выполнить набор действий для изменения css или удаления/добавления класса для определенных элементов HTML. По сути, для каждого элемента мне нужно переключить их начальное состояние обратно и принудительно. Подробности смотрите в коде:

$('#videos-button').on('click',function(){
            $("#ib-row").css({'display':'none'});
            $("#ib-pagination").css({'display':'none'});
            $("#ib-video-carousel-pagination").css({'display':'block'});
            $("#ib-video-carousel").css({'display':'block'});
            $("#photos-button").removeClass('image-block-media-button-selected');
            $("#videos-button").addClass('image-block-media-button-selected');
});


$('#photos-button').on('click',function(){
            $("#ib-row").css({'display':'block'});
            $("#ib-pagination").css({'display':'block'});
            $("#ib-video-carousel-pagination").css({'display':'none'});
            $("#ib-video-carousel").css({'display':'none'});
            $("#videos-button").removeClass('image-block-media-button-selected');
            $("#photos-button").addClass('image-block-media-button-selected');
        });

Как я могу реорганизовать его более элегантным способом, без дублирования?

Существуют ли какие-либо другие возможности для всех этих элементов, для которых задан свой CSS, кроме комбинаций, показанных в этих двух обработчиках? Например, если у #ib-row есть display: block, всегда ли у #ib-pagination будет display: block, у #ib-video-carousel будет display: none, а у #photos-button будет выбранный класс?

CertainPerformance 28.05.2019 05:14
Поведение ключевого слова "this" в стрелочной функции в сравнении с нормальной функцией
Поведение ключевого слова "this" в стрелочной функции в сравнении с нормальной функцией
В JavaScript одним из самых запутанных понятий является поведение ключевого слова "this" в стрелочной и обычной функциях.
Концепция локализации и ее применение в приложениях React ⚡️
Концепция локализации и ее применение в приложениях React ⚡️
Локализация - это процесс адаптации приложения к различным языкам и культурным требованиям. Это позволяет пользователям получить опыт, соответствующий...
Улучшение производительности загрузки с помощью Google Tag Manager и атрибута Defer
Улучшение производительности загрузки с помощью Google Tag Manager и атрибута Defer
В настоящее время производительность загрузки веб-сайта имеет решающее значение не только для удобства пользователей, но и для ранжирования в...
Безумие обратных вызовов в javascript [JS]
Безумие обратных вызовов в javascript [JS]
Здравствуйте! Юный падаван 🚀. Присоединяйся ко мне, чтобы разобраться в одной из самых запутанных концепций, когда вы начинаете изучать мир...
Система управления парковками с использованием HTML, CSS и JavaScript
Система управления парковками с использованием HTML, CSS и JavaScript
Веб-сайт по управлению парковками был создан с использованием HTML, CSS и JavaScript. Это простой сайт, ничего вычурного. Основная цель -...
JavaScript Вопросы с множественным выбором и ответы
JavaScript Вопросы с множественным выбором и ответы
Если вы ищете платформу, которая предоставляет вам бесплатный тест JavaScript MCQ (Multiple Choice Questions With Answers) для оценки ваших знаний,...
1
1
59
2
Перейти к ответу Данный вопрос помечен как решенный

Ответы 2

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

одним из способов рефакторинга было бы создание функции, которая принимает кнопку видео или кнопку фотографии в качестве входных данных и возвращает объект (карту) идентификаторов и соответствующих классов. то есть функция (назовем ее получитькласскарту), возвращающая

{ '#id-row' : 'display: none', ... } 

когда i/p - кнопка видео и

{ '#id-row' : 'display: block', ... }

когда i/p является фото-кнопкой. Затем переместите логику применения классов, т.е.

function applyStyles(id) {
    const styleIdMap = getClassMap(id);
    $("#ib-row").css(styleIdMap["#ib-row"]);
    $("#ib-pagination").css(styleIdMap["#ib-pagination"]);
    ...
}

Таким образом, в логике нет повторений.

Вы можете попробовать это:

const ibRowAndPagination = ['#ib-row', '#ib-pagination'];
const ibVideoCarouselAndPagination = ['#ib-video-carousel-pagination', '#ib-video-carousel'];
const photosAndVideosButtons = ['#photos-button', '#videos-button'];
const photosAndVideosButtonsToggleClass = () => photosAndVideosButtons.forEach(el => $(el).toggleClass('image-block-media-button-selected'));

$('#videos-button').on('click', function() {
  ibRowAndPagination.forEach(el => $(el).css({'display':'none'}));
  ibVideoCarouselAndPagination.forEach(el => $(el).css({'display':'block'}));
  photosAndVideosButtonsToggleClass();
});

$('#photos-button').on('click', function() {
  ibRowAndPagination.forEach(el => $(el).css({'display':'block'}));
  ibVideoCarouselAndPagination.forEach(el => $(el).css({'display':'none'}));
  photosAndVideosButtonsToggleClass();
});

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