我正在尝试将我们当前的 jQuery 代码转换为 slider 的 ES6。 下面是当前的 jQuery 代码:
$('.slider-nav a').click(function (event) {
$('.slider-nav a').not($(this)).removeClass('current');
$(this).addClass('current');
$('.slider-content').not($(this).data('target')).removeClass('current');
$($(this).data('target')).addClass('current');
event.preventDefault();
});
这是我更新的 javascript 代码
let contentSlidersNavs = document.querySelectorAll('.slider-nav a');
if (contentSlidersNavs.length) {
for (let current_item of contentSlidersNavs) {
current_item.addEventListener('click', function (event) {
let target_element = event.target || event.srcElement;
for(let current_item of contentSlidersNavs){
if(current_item !== target_element){
current_item.classList.remove('current');
}
}
current_item.classList.add('current');
let current_content_id = current_item.dataset.target;
let slider_contents = document.querySelectorAll(`.slider-content:not(${current_content_id})`);
for(let current_content of slider_contents){
current_content.classList.remove('current');
}
document.querySelector(`.slider-content${current_content_id}`).classList.add('current');
event.preventDefault();
});
}
}
如您所见,我的新代码比以前的 jQuery 代码大三倍,而且看起来更复杂。
例如,下面的 jQuery 行:
$('.slider-nav a').not($(this)).removeClass('current');
成为这个 ES6 block :
let target_element = event.target || event.srcElement;
for(let current_item of contentSlidersNavs){
if(current_item !== target_element){
current_item.classList.remove('current');
}
}
所以,我想请教您的建议,有什么方法可以优化我的代码性能或使代码更简洁?
这是 jsfiddle link玩。
谢谢!
最佳答案
您正在观看 jQuery 之美的演示;更短、更精简的代码。代价是加载时间增加和性能稍慢。
要回答您的问题,不,实际上没有任何方法可以优化您拥有的“非”逻辑。无论采用哪种方式,都需要一个循环。
您可以将 :not
选择器与 querySelectorAll()
一起使用,但这依赖于将 target_element
对象转换为有效的选择器字符串。
关于Javascript ES6 代码优化,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/42689353/