JS-鼠标经过显示二级菜单

在css处添加了border样式为了看得更清楚——源代码有一个程序漏洞，存在一个很烦人的大bug。

 1 <ul class="nav">
 2         <li class="li">
 3             <a href="#">一级菜单</a>
 4             <ul class="subNav">
 5                 <li>
 6                     <a href="#">二级菜单</a>
 7                 </li>
 8                 <li>
 9                     <a href="#">二级菜单</a>
10                 </li>
11                 <li>
12                     <a href="#">二级菜单</a>
13                 </li>
14                 <li>
15                     <a href="#">二级菜单</a>
16                 </li>
17             </ul>
18         </li>
19         <li class="li">
20             <a href="#">一级菜单</a>
21             <ul class="subNav">
22                 <li>
23                     <a href="#">二级菜单</a>
24                 </li>
25                 <li>
26                     <a href="#">二级菜单</a>
27                 </li>
28                 <li>
29                     <a href="#">二级菜单</a>
30                 </li>
31                 <li>
32                     <a href="#">二级菜单</a>
33                 </li>
34             </ul>
35         </li>
36         <li>
37             <a href="#">一级菜单</a>
38         </li>
39         <li>
40             <a href="#">一级菜单</a>
41         </li>
42         <li>
43             <a href="#">一级菜单</a>
44         </li>
45     </ul>

html源码

 1 <style type="text/css">
 2 * {
 3 margin: 0;
 4 padding: 0;
 5 font-size: 14px;
 6 }
 7 
 8 a {
 9 color: #333;
10 text-decoration: none
11 }
12 
13 ul {
14 list-style: none;
15 }
16 
17 .nav {
18 height: 30px;
19 border-bottom: 5px solid #F60;
20 margin-left: 50px;
21 width: 600px;
22 }
23 
24 .nav li {
25 float: left;
26 position: relative;
27 height: 30px;
28 width: 120px
29 }
30 
31 .nav li a {
32 display: block;
33 height: 30px;
34 text-align: center;
35 line-height: 30px;
36 width: 120px;
37 background: #efefef;
38 margin-left: 1px;
39 }
40 
41 .subNav {
42 position: absolute;
43 top: 30px;
44 left: 0;
45 width: 120px;
46 /*height:120px;*/
47 height: 0;
48 overflow: hidden;
49 border: 1px solid #4169E1;/*先找准height的值*/
50 }
51 
52 .subNav li a {
53 background: #ddd
54 }
55 
56 .subNav li a:hover {
57 background: #efefef
58 }
59 </style>

style样式

//alert('+20');//弹出+20，以判断在这里执行了，几次+20.经判断，他是执行了6次，即加到了120，在120的时候，因为if判断，执行了clear
This.style.height = This.offsetHeight + 20 + "px";
if(This.offsetHeight >= 120) {

sub.style.height = 120+'px';//解决2
clearInterval(This.timer)

//alert('跳出来了');//clear执行后，跳出本次循环

}
}, 30)

//alert(sub);//本来以为放在这里弹窗。会在加完120后再执行，没想到他先执行的这个，并且弹出ullistElement，即说明他找到了subNav的ul，
　　问题来了，当加载完成后，请你尝试把鼠标再放到二级菜单上移动一下，你没选择一个二级菜单，他就是把这三个alert再执行一遍，然后你就会看到ul的长度又加了20，当你不停的上下晃动鼠标，二级菜单就会不停的添加。最后加的老长。为了明显，我加了border，效果一目了然。
　　原因：这里，通过这个代码：alert(aLi.length);原因就明显了，因为开头的aLi，获得的是body中所有的li，包括二级菜单的。就是说你在二级菜单的每一个li上晃一下，他都会认为你是在重新执行了aLi.onmouseover这个代码，流程就再走了一遍。但是因为在高度加20那里，他高度先增加了20，然后一判断，发现高度大于120了，就赶紧跳出了。所以就会每次你晃一下，只增加了20。
　　解决1，从源头，只找对应的li，试过以后，我还没发现真谛。难道是要给ul一个id，然后通过id获得ul下的li集合，再判断谁有二级菜单进行显示
　　解决2：高度处，在判断那里，如果高度大于120了，我们就直接让高度等于120，不就得了，以后他再长，也会被这一条限制住。sub.style.height = 120+'px';
　　解决3：判断处，再增加前我先判断是不是大于120行不行，是的话你就停止，不是就继续。问题也就可以解决了。代码：

if(This.offsetHeight >= 120){
clearInterval(This.timer)；
}else{
This.style.height = This.offsetHeight + 20 + "px";
}

　　最后mouseout那里，就不用管了，毕竟ul的长度不会出格了，就不需要加强防备了

}
}
</script>

    <script>
window.onload = function() {
            var aLi = document.getElementsByClassName('li');
            for(var i = 0; i < aLi.length; i++) {
                aLi[i].onmouseover = function() {
                        var sub = this.getElementsByClassName('subNav')[0];
                        if(sub) {
                            clearInterval(sub.timer);
                            sub.timer = setInterval(function() {
                                sub.style.height = sub.offsetHeight + 20 + "px";
                                if(sub.offsetHeight > 120) {
                                    sub.style.height = 120+'px';
                                    clearInterval(sub.timer)
                                }
                            }, 30)
                        }
                    }        
                aLi[i].onmouseout = function() {
                    var sub = this.getElementsByTagName('ul')[0];
                    if(sub) {
                        clearInterval(sub.timer);
                        sub.timer = setInterval(function() {
                            sub.style.height = sub.offsetHeight - 20 + "px";
                            if(sub.offsetHeight <= 0) {
                                clearInterval(sub.timer)
                            }
                        }, 30)
                    }
                }
            }
        }
    </script>

调整好的js源码

　　后来我也发现了收缩不干净，总是多出外面几像素，但是我把border删掉他就收缩干净了，我不知道在js处是不是还存在问题导致的收缩不干净。

　　幕友是这么回答的：收缩不干净是因为宽高不会小于零，比如你在某一刻高度为3，但是要减去4，这个时候高度的值不会等于-1，也不会等于0，而是选择不执行。也就是说，高度会一直为3，所以if里面的将高度设置为零的语句根本就没有执行，同时定时器也没有清除。然后没收缩干净的就是很小的高度为3的部分。关键是定时器还一直在占用系统的资源。希望对认真做练习和思考的小伙伴们有帮助。

来自慕课练习题的问题：http://www.imooc.com/code/1737

posted @ 2016-09-19 09:58 xing.org1^ 阅读(969) 评论(0) 编辑收藏举报

刷新页面返回顶部

xing.org1^

前端界的小学生~

JS-鼠标经过显示二级菜单

公告