0

我一直在编辑一个脚本,以便在单击菜单图标时将其变为蓝色。原始脚本有 8 个 if 语句。这很有效,但是当我将 if 语句更改为 switch 时,它停止了工作。我已经对语法错误、拼写错误等进行了三次检查,但菜单图标仍然是灰色的!

谁能给我一个解释?

window.onload = function() {

    [].forEach.call(document.querySelectorAll('.navico'), function(el) { 
        el.addEventListener('click', imageButtonClickHandler); 
    });


    function imageButtonClickHandler() 
    {  

        switch(this.id) 
        {
           case(this.id.match("aboutnav")):

                grey();
                $('#abouttitle').css('color', 'blue');
                $('#a').css('color', 'blue');
            break;

           case(this.id.match("routenav")):

                grey();
                $('#routetitle').css('color', 'blue');
                $('#b').css('color', 'blue');

            break;

           case(this.id.match("enternav")):

                grey();
                $('#entertitle').css('color', 'blue');
                $('#c').css('color', 'blue');

            break;

           case(this.id.match("racedaynav")): 

                grey();
                $('#racedaytitle').css('color', 'blue');
                $('#d').css('color', 'blue');

            break;

           case(this.id.match("gallerynav")):

                grey();
                $('#e').css('color', 'blue');
                $('#gallerytitle').css('color', 'blue');

            break;

           case(this.id.match("newsnav")):

                grey();
                $('#f').css('color', 'blue');
                $('#newstitle').css('color', 'blue');

            break;

           case(this.id.match("resultsnav")): 

                grey();
                $('#g').css('color' , 'blue');
                $('#resultstitle').css('color', 'blue');

            break;

           case(this.id.match("contactnav")): 

                grey();
                $('#contacttitle').css('color', 'blue');
                $('#h').css('color', 'blue');
            break;

            }
        }
    }

};

谢谢!

4

2 回答 2

5

如果您正在查找确切的 ID 值,请将this.id.match("something")更改为case "something" 。如果您尝试应用正则表达式来匹配字符串的某些部分,我认为您不能使用 switch。

开关在您的情况下正在做什么:对于每个case,它调用 match 方法并确定函数的结果(数组或 null)是否等于您的字符串。这当然不会发生。

于 2013-08-25T21:49:37.953 回答
0

这是您的问题的工作示例。 JsFiddle 链接

HTML:

<div id="aboutnav" class="navico">Click 1</div>
<div id="routenav" class="navico">Click 2</div>
<div id="enternav" class="navico">Click 3</div>
<div id="racedaynav" class="navico">Click 4</div>

Javascript:

 $(document).ready(function () {

    [].forEach.call(document.querySelectorAll('div.navico'), function (el) {
        el.addEventListener('click', imageButtonClickHandler);
    });


    function imageButtonClickHandler() {
        switch (this.id) {
            case "aboutnav":

                $('#aboutnav').css('color', 'blue');
                $('#a').css('color', 'blue');
                break;

            case "routenav":

                //grey();
                $('#routenav').css('color', 'red');
                $('#b').css('color', 'blue');

                break;

            case "enternav":

                //grey();
                $('#enternav').css('color', 'yellow');
                $('#c').css('color', 'blue');

                break;

            case "racedaynav":

                //grey();
                $('#racedaynav').css('color', 'orange');
                $('#d').css('color', 'blue');

                break;
        }

    }
});
于 2013-08-25T22:13:48.683 回答