jQuery為我的顏色應用程序提供更好的方法？

Question

我是jQuery的新手，我必須承認我不喜歡它！

我到處都在尋找信息，這就是我用我的小技巧所做的！

我承認我的代碼可以正常工作，但是效果很差，我懷疑它是否經過優化。 請您能幫我改善和優化此jQuery代碼！

如您所見，該代碼旨在在用戶選擇其中一種顏色時在多個標簽（div，li，h2 ...）中顯示各種信息。

這是我的jQuery代碼：

$(document).ready(function() {

$("div.color-value.hex p").text("pick a color");
$("div.color-value.rgba p").text("pick a color");
$("div.color-value.hsla p").text("pick a color");

$("ul.color-list li").click(function() {

    //var value;

    switch ( $("ul.color-list.one li").index(this) ) {
        case 0 :
            $(this).addClass("ckd");
            $("div.color-type-infos").text("color with alpha");
            $("div.color-value.hex p").text("#1E2224");
            $("div.color-value.rgba p").text("#rgba");
            $("div.color-value.hsla p").text("#hsla");
            break;
        case 1 :
            $(this).addClass("ckd");
            $("div.color-type-infos").text("color with alpha");
            $("div.color-value.hex p").text("#2");
            $("div.color-value.rgba p").text("#rgba2");
            $("div.color-value.hsla p").text("#hsla2");
            break;
        case 2 :
            $(this).addClass("ckd");
            $("div.color-type-infos").text("color without alpha");
            $("div.color-value.hex p").text("#3");
            $("div.color-value.rgba p").text("#rgba3");
            $("div.color-value.hsla p").text("#hsla3");
            break;
        case 3 :
            $(this).addClass("ckd");
            $("div.color-type-infos").text("simple color");
            $("div.color-value.hex p").text("#4");
            $("div.color-value.rgba p").text("#rgba4");
            $("div.color-value.hsla p").text("#hsla4");
            break;
        case 4 :
            $(this).addClass("ckd");
            $("div.color-type-infos").text("color with alpha");
            $("div.color-value.hex p").text("#5");
            $("div.color-value.rgba p").text("#rgba5");
            $("div.color-value.hsla p").text("#hsla5");
            break;
        case 5 :
            $(this).addClass("ckd");
            $("div.color-type-infos").text("color with alpha");
            $("div.color-value.hex p").text("#6");
            $("div.color-value.rgba p").text("#rgba6");
            $("div.color-value.hsla p").text("#hsla6");
            break;

        case 6 :
            $(this).addClass("ckd");
            $("div.color-type-infos").text("color with alpha");
            $("div.color-value.hex p").text("#7");
            $("div.color-value.rgba p").text("#rgba7");
            $("div.color-value.hsla p").text("#hsla7");
            break;
        }

    // second swatch
    switch ( $("ul.color-list.two li").index(this) ) {
        case 0 :
            $(this).addClass("ckd");
            $("div.color-type-infos").text("color with alpha");
            $("div.color-value.hex p").text("#1E2224");
            $("div.color-value.rgba p").text("#rgba");
            $("div.color-value.hsla p").text("#hsla");
            break;
        case 1 :
            $(this).addClass("ckd");
            $("div.color-type-infos").text("color with alpha");
            $("div.color-value.hex p").text("#2");
            $("div.color-value.rgba p").text("#rgba2");
            $("div.color-value.hsla p").text("#hsla2");
            break;
        case 2 :
            $(this).addClass("ckd");
            $("div.color-type-infos").text("color with alpha");
            $("div.color-value.hex p").text("#3");
            $("div.color-value.rgba p").text("#rgba3");
            $("div.color-value.hsla p").text("#hsla3");
            break;
        case 3 :
            $(this).addClass("ckd");
            $("div.color-type-infos").text("color with alpha");
            $("div.color-value.hex p").text("#4");
            $("div.color-value.rgba p").text("#rgba4");
            $("div.color-value.hsla p").text("#hsla4");
            break;
        case 4 :
            $(this).addClass("ckd");
            $("div.color-type-infos").text("color with alpha");
            $("div.color-value.hex p").text("#5");
            $("div.color-value.rgba p").text("#rgba5");
            $("div.color-value.hsla p").text("#hsla5");
            break;
        case 5 :
            $(this).addClass("ckd");
            $("div.color-type-infos").text("color with alpha");
            $("div.color-value.hex p").text("#6");
            $("div.color-value.rgba p").text("#rgba6");
            $("div.color-value.hsla p").text("#hsla6");
            break;

        case 6 :
            $(this).addClass("ckd");
            $("div.color-type-infos").text("color with alpha");
            $("div.color-value.hex p").text("#7");
            $("div.color-value.rgba p").text("#rgba7");
            $("div.color-value.hsla p").text("#hsla7");
            break;
        }


    // third swatches
    switch ( $("ul.color-list.three li").index(this) ) {
        case 0 :
            $(this).addClass("ckd");
            $("div.color-type-infos").text("color with alpha");
            $("div.color-value.hex p").text("#1E2224");
            $("div.color-value.rgba p").text("#rgba");
            $("div.color-value.hsla p").text("#hsla");
            break;
        case 1 :
            $(this).addClass("ckd");
            $("div.color-type-infos").text("color with alpha");
            $("div.color-value.hex p").text("#2");
            $("div.color-value.rgba p").text("#rgba2");
            $("div.color-value.hsla p").text("#hsla2");
            break;
        case 2 :
            $(this).addClass("ckd");
            $("div.color-type-infos").text("color with alpha");
            $("div.color-value.hex p").text("#3");
            $("div.color-value.rgba p").text("#rgba3");
            $("div.color-value.hsla p").text("#hsla3");
            break;
        case 3 :
            $(this).addClass("ckd");
            $("div.color-type-infos").text("color with alpha");
            $("div.color-value.hex p").text("#4");
            $("div.color-value.rgba p").text("#rgba4");
            $("div.color-value.hsla p").text("#hsla4");
            break;
        case 4 :
            $(this).addClass("ckd");
            $("div.color-type-infos").text("color with alpha");
            $("div.color-value.hex p").text("#5");
            $("div.color-value.rgba p").text("#rgba5");
            $("div.color-value.hsla p").text("#hsla5");
            break;
        case 5 :
            $(this).addClass("ckd");
            $("div.color-type-infos").text("color with alpha");
            $("div.color-value.hex p").text("#6");
            $("div.color-value.rgba p").text("#rgba6");
            $("div.color-value.hsla p").text("#hsla6");
            break;

        case 6 :
            $(this).addClass("ckd");
            $("div.color-type-infos").text("color with alpha");
            $("div.color-value.hex p").text("#7");
            $("div.color-value.rgba p").text("#rgba7");
            $("div.color-value.hsla p").text("#hsla7");
            break;
        }
    });

    $("ul.color-list li").click(function(){
            if($(this).hasClass("ckd")){
                $(this).removeClass("ckd");
            } else {
                $(this).addClass("ckd");
        }
    });


});

Answer 1

大多數時候，不需要使用switch語句，您可以將信息存儲在data-*屬性中，並使用.data()方法讀取它：

<li  ... data-info="the info body"></li>

要切換類，可以使用.toggleClass()方法，因此不必使用第二個單擊處理程序，也可以將其主體移動到第一個處理程序，我沒有看到標記，但是根據您的代碼，應如下所示幫助縮小腳本：

$(document).ready(function() {

    var $hex = $("div.color-value.hex p").text("pick a color"),
        $rgba = $("div.color-value.rgba p").text("pick a color"),
        $hsla = $("div.color-value.hsla p").text("pick a color"),
        $info = $("div.color-type-infos");

    $("ul.color-list li").on('click', function() {
        var i = $("ul.color-list.one li").index(this) + 1;
        $("ul.color-list li").not(this).removeClass('ckd');
        var info = $(this).toggleClass("ckd").data('info');

        $info.text(info);
        $hex.text( "#" + (i === 1 ? "1E2224" : i) );
        $rgba.text( "#rgba" + (i !== 1 ? i : "") );
        $hsla.text( "#hsla" + (i !== 1 ? i : "") );
    });

});