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 : "") );
    });

});