Код оптимизирован, как его хорошо выглядеть

-2

Сегодня я код JavaScript, но он действительно выглядит не очень хорошо. Так как я могу оптимизировать его с хорошим расширить и отыскать вкусныеКод оптимизирован, как его хорошо выглядеть

if($("#active-flg").val() == 'clubInfo') { 
    $("[name='local-li']:eq(0)").addClass("active"); 
    $("[name='local-li']:eq(1)").removeClass("active"); 
    $("[name='local-li']:eq(2)").removeClass("active"); 

    $("#manage-club-info").addClass("active"); 
    $("#manage-club-config").removeClass("active"); 
    $("#manage-club-phone").removeClass("active"); 

} else if($("#active-flg").val() == 'clubSeo') { 
    $("[name='local-li']:eq(0)").removeClass("active"); 
    $("[name='local-li']:eq(1)").addClass("active"); 
    $("[name='local-li']:eq(2)").removeClass("active"); 

    $("#manage-club-info").removeClass("active"); 
    $("#manage-club-seo").addClass("active"); 
    $("#manage-club-phone").removeClass("active"); 

} else if($("#active-flg").val() == 'clubPhone') { 
    $("[name='local-li']:eq(0)").removeClass("active"); 
    $("[name='local-li']:eq(1)").removeClass("active"); 
    $("[name='local-li']:eq(2)").addClass("active"); 

    $("#manage-club-info").removeClass("active"); 
    $("#manage-club-seo").removeClass("active"); 
    $("#manage-club-phone").addClass("active"); 
}

источник

2016-03-22 NattyQ

Если код работает, и вы хотите, чтобы он рассмотрел, [codereview.se] это место, чтобы спросить. – Tushar

@NattyQ вы также можете поделиться html – brk

Хотя это не обзор кода,

var flags = ['clubInfo', 'clubSeo', 'clubPhone'] 
var targets = ['#manage-club-info','#manage-club-seo','#manage-club-phone'] 
var val = $("#active-flg").val(); 
var index = flags.indexOf(val); 

if (index >= 0){ 
    $("[name='local-li']").removeClass("active"); 
    $(targets.join(',')).removeClass("active"); 

    var activeLi = "[name='local-li']:eq("+ index +")"; 
    $(activeLi).addClass("active"); 
    $(targets[index]).addClass("active");  
}

источник

2016-03-22 05:01:41 softwarenewbie7331

Попробуйте это ..

if($("#active-flg").val() == 'clubInfo') { 
    removeClass(["[name='local-li']:eq(1)","[name='local-li']:eq(2)","#manage-club-config","#manage-club-phone"]); 
    addClass(["[name='local-li']:eq(0)","#manage-club-info"]) 

} else if($("#active-flg").val() == 'clubSeo') { 
    removeClass(["[name='local-li']:eq(0)","[name='local-li']:eq(2)","#manage-club-info","#manage-club-phone"]); 
    addClass(["[name='local-li']:eq(1)","#manage-club-seo"]) 

} else if($("#active-flg").val() == 'clubPhone') { 
    removeClass(["[name='local-li']:eq(0)","[name='local-li']:eq(1)","#manage-club-info","#manage-club-seo"]); 
    addClass(["[name='local-li']:eq(2)","#manage-club-phone"]); 
} 


function addClass(arraySelectors){ 
    for (selector in arraySelectors){ 
     $(arraySelectors[selector]).addClass("active"); 
    } 
} 

function removeClass(arraySelectors){ 
    for (selector in arraySelectors){ 
     $(arraySelectors[selector]).removeClass("active"); 
    } 
}

источник

2016-03-22 04:53:14

Что об этом

var actions = { 
    clubInfo: function(){ 
     $("[name='local-li']:eq(0)").addClass("active"); 
     $("[name='local-li']:eq(1)").removeClass("active"); 
     $("[name='local-li']:eq(2)").removeClass("active"); 

     $("#manage-club-info").addClass("active"); 
     $("#manage-club-config").removeClass("active"); 
     $("#manage-club-phone").removeClass("active"); 

    }, 
    clubSeo: function(){ 
     $("[name='local-li']:eq(0)").removeClass("active"); 
     $("[name='local-li']:eq(1)").addClass("active"); 
     $("[name='local-li']:eq(2)").removeClass("active"); 

     $("#manage-club-info").removeClass("active"); 
     $("#manage-club-seo").addClass("active"); 
     $("#manage-club-phone").removeClass("active"); 
    }, 
    clubPhone: function(){ 
     $("[name='local-li']:eq(0)").removeClass("active"); 
     $("[name='local-li']:eq(1)").removeClass("active"); 
     $("[name='local-li']:eq(2)").addClass("active"); 

     $("#manage-club-info").removeClass("active"); 
     $("#manage-club-seo").removeClass("active"); 
     $("#manage-club-phone").addClass("active"); 
    } 
} 
var action = $("#active-flg").val(); 
if(actions[action]) { 
    actions[action](); 
} else { 
    sexyDefaultFunction(); 
}

источник

2016-03-22 04:59:43 stalin

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

Что вы хотите делать, это найти способы перебора вещей, которые у вас есть, и оставить избыточность на компьютере или еще лучше, но просто избавиться от избыточности и использовать только те значения, которые вам нужны. AKA вместо:

if(var == "bread") { 
    $("#bread").addClass("active"); 
} 

if(var == "milk") { 
    $("#milk").addClass("active"); 
} 

if(var == "eggs") { 
    $("#eggs").addClass("active"); 
}

Вы хотите что-то вроде:

$("#" + var).addClass("active");

источник

2016-03-22 05:06:05 JackHasaKeyboard

Спасибо за ваши идеи. – NattyQ

Не столько мои идеи, сколько фундаментальные аспекты информатики, не проблема! Эти вещи вступают в контекст, тем больше вы делаете. – JackHasaKeyboard

Код оптимизирован, как его хорошо выглядеть

ответ

Смежные вопросы