为了账号安全,请及时绑定邮箱和手机立即绑定

请问为什么我的代码这样写鼠标可以控制,可是键盘就控制不了呢?代码错在哪?怎么改

<!doctype html>

<html>


<head>

<meta charset="UTF-8">

<title>下拉菜单</title>

<style type="text/css">

body,

ul,

li {

margin: 0;

padding: 0;

font-size: 13px;

}


ul,

li {

list-style: none;

}


#divselect {

width: 186px;

margin: 80px auto;

position: relative;

z-index: 10000;

}


#divselect cite {

width: 150px;

height: 24px;

line-height: 24px;

display: block;

color: #807a62;

cursor: pointer;

font-style: normal;

padding-left: 4px;

padding-right: 30px;

border: 1px solid #333333;

background: url(xjt.png) no-repeat right center;

}


#divselect ul {

width: 184px;

border: 1px solid #333333;

background-color: #ffffff;

position: absolute;

z-index: 20000;

margin-top: -1px;

display: none;

}


#divselect ul li {

height: 24px;

line-height: 24px;

}


#divselect ul li a {

display: block;

height: 24px;

color: #333333;

text-decoration: none;

padding-left: 10px;

padding-right: 10px;

}

</style>

<script type="text/javascript">

window.onload = function() {

var box = document.getElementById('divselect'),

title = box.getElementsByTagName('cite')[0],

menu = box.getElementsByTagName('ul')[0],

as = box.getElementsByTagName('a'),

index = -1,

btn = false;


// 点击三角时

title.onclick = function(event) {

event = event || window.event;

if(event.stopPropagation) {

event.stopPropagation();

} else {

event.cancelBubble();

}

if(btn == false) {

menu.style.display = "block";

btn = true;

} else {

menu.style.display = "none";

btn = false;

}


// 键盘事件

document.onkeyup = function(event) {

e = event || window.event;

if(e.keyCode == 40) {

index++;

if(index >= as.length) {

index = 0;

}

changebgd();

}

if(e.keyCode == 38) {

if(index <= 0) {

index = as.length;

}

index--;

changebgd();

}

}


}


// 鼠标滑过、离开、点击每个选项时

for(var i = 0; i < as.length; i++) {

as[i].index = i;//得这样传值否则匿名函数获取不到i的值

as[i].onmouseover = function() {

for(var j = 0; j < as.length; j++) {

as[j].style.background = "none";

}

this.style.background = "#ccc";

index = this.index;

keyenter();

}

//   as[i].onmouseout=function(){

//       this.style.background="#fff";

//   }上面那样写、键盘上下移动后鼠标移动,LI不存在两个“被选”。

as[i].onclick = function() {

title.innerHTML = this.innerHTML;

reset();

}

// 执行脚本

}


// 执行脚本


function changebgd() {

for(var i = 0; i < as.length; i++) {

as[i].style.background = "none";

}

as[index].style.background = "#ccc";

keyenter();

}


function keyenter() {

document.onkeyup = function(event) {

event = event || window.event;

if(event.keyCode == 13 && index != -1) {

title.innerHTML = as[index].innerHTML;

reset();

document.onkeyup=null;

}

}

}


// 点击页面空白处时

document.onclick = function() {

reset();

}


//复位函数

function reset() {

if(btn == true) {

if(index != -1) {

as[index].style.background = "none";

index = -1;

}


menu.style.display = "none";

btn = false;

}


}

// 执行脚本

}


</script>

</head>


<body>

<div id="divselect">

<cite>请选择分类</cite>

<ul>

<li id="li">

<a href="javascript:;" selectid="1">ASP开发</a>

</li>

<li>

<a href="javascript:;" selectid="2">.NET开发</a>

</li>

<li>

<a href="javascript:;" selectid="3">PHP开发</a>

</li>

<li>

<a href="javascript:;" selectid="4">Javascript开发</a>

</li>

<li>

<a href="javascript:;" selectid="5">Java特效</a>

</li>

</ul>

</div>

</body>


</html>

如果改,怎么个改法最好最少呢

正在回答

3 回答

设置index值就是为了鼠标和键盘可以共同利用这个值控制选中项,每次鼠标经过或上下键按动时index值都在刷新,所以没必要到处调用keyenter()函数,只需要判断按键的键值的三种情况进行处理或者调用处理函数,你调用函数,各种嵌套,函数内又触发事件,反而有点乱了。

还有个小问题,如果点开下拉列表,不选选项的话点空白处,选项隐藏不了,因为不选选项的话index值为-1,reset()函数第一句话走不通。建议:

1、将判断键值的三种情况写到一起,document.onkeyup=function(){} 里面;

2、将keyenter()中只留处理的两行代码;

3、去掉鼠标划过和函数changeBgd()中的keyenter()函数调用;

4、在判断回车键条件中调用keyenter()函数;

5、将reset()函数第一句话改为遍历列表全部设置背景色;

<!doctype html>
<html>
<head>
    <meta charset="UTF-8">
    <title>下拉菜单</title>
    <style type="text/css">
body,ul,li{ margin:0; padding:0; font-size:13px;}
ul,li{list-style:none;}
#divselect{width:186px; margin:80px auto; position:relative; z-index:10000;}
#divselect cite{width:150px; height:24px;line-height:24px; display:block; color:#807a62; cursor:pointer;font-style:normal;
padding-left:4px; padding-right:30px; border:1px solid #333333; 
background:url(xjt.png) no-repeat right center;}
#divselect ul{width:184px;border:1px solid #333333; background-color:#ffffff; position:absolute; z-index:20000; margin-top:-1px; display:none;}
#divselect ul li{height:24px; line-height:24px;}
#divselect ul li a{display:block; height:24px; color:#333333; text-decoration:none; padding-left:10px; padding-right:10px;}
    </style>
   <script type="text/javascript">
window.onload=function(){
    var box=document.getElementById('divselect'),
        title=box.getElementsByTagName('cite')[0],
        menu=box.getElementsByTagName('ul')[0],
        as=box.getElementsByTagName('a'),
        index=-1,
        btn=false;
    // 点击三角时
    title.onclick=function(event){
        event=event||window.event;
        if(event.stopPropagation){
            event.stopPropagation();
        }else{
            event.cancelBubble();
        }
        if(btn==false){
           menu.style.display="block";
           btn=true;
       }else {
           menu.style.display="none";
           btn=false;
       }
       
   }
    // 鼠标滑过、离开、点击每个选项时
   for(var i=0;i<as.length;i++){
       as[i].index=i;
       as[i].onmouseover=function(){
           for(var j=0;j<as.length;j++){
               as[j].style.background="#fff";
           }
           this.style.background="#ccc";
           index=this.index;
           //keyenter();
       }
    //   as[i].onmouseout=function(){
    //       this.style.background="#fff";
    //   }上面那样写、键盘上下移动后鼠标移动,LI不存在两个“被选”。
       as[i].onclick=function(){
           title.innerHTML=this.innerHTML;
           reset();
       }
      // 执行脚本
}  
   // 键盘事件
   document.onkeyup=function(event){
       event=event||window.event;
       if(event.keyCode==40){
           index++;
           if(index>=as.length){
               index=0;
           }
           changeBgd();
       }
       if(event.keyCode==38){
           if(index<=0){
               index=as.length;
           }
           index--;
           changeBgd();
       }
       if(event.keyCode==13){
           keyenter();
       }
   }
      // 执行脚本
   function keyenter(){
        title.innerHTML=as[index].innerHTML;
        reset();
       
   }
   function changeBgd(){
       //alert("1");
       for(var i=0;i<as.length;i++){
           as[i].style.background="none";
       }
       as[index].style.background="#ccc";
       //keyenter();
   }
   // 点击页面空白处时
   document.onclick=function(){
       reset();
   }
   //复位函数
   function reset(){
       for(var i=0;i<as.length;i++){
          as[i].style.background="none";
       }
          index=-1;
          menu.style.display="none";
          btn=false; 
   } 
 }
   </script>
</head>
<body>
    <div id="divselect">
      <cite>请选择分类</cite>
      <ul>
         <li id="li"><a href="javascript:;" selectid="1">ASP开发</a></li>
         <li><a href="javascript:;" selectid="2">.NET开发</a></li>
         <li><a href="javascript:;" selectid="3">PHP开发</a></li>
         <li><a href="javascript:;" selectid="4">Javascript开发</a></li>
         <li><a href="javascript:;" selectid="5">Java特效</a></li>
      </ul>
    </div>
</body>
</html>


0 回复 有任何疑惑可以回复我~
#1

jazia 提问者

非常感谢!大神
2016-09-22 回复 有任何疑惑可以回复我~
#2

jazia 提问者

这样改条理清晰多了,原来不用调用keyenter函数的,按回车键它就自动调用了,不用我们写,谢谢仔细认真的回答
2016-09-22 回复 有任何疑惑可以回复我~

求教一下,你的“// 鼠标滑过、离开、点击每个选项时”这段代码中,这个as[i].index=i;是什么意思??

0 回复 有任何疑惑可以回复我~

建议你下载源码看下吧,封装函数有问题,各种函数嵌套反而更乱了

0 回复 有任何疑惑可以回复我~
#1

jazia 提问者

谢谢回答,不用了,楼上解决了。
2016-09-22 回复 有任何疑惑可以回复我~

举报

0/150
提交
取消

请问为什么我的代码这样写鼠标可以控制,可是键盘就控制不了呢?代码错在哪?怎么改

我要回答 关注问题
意见反馈 帮助中心 APP下载
官方微信