javascript - 我的颜色应用程序的 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 : "") );
    });

});