JS Onclick事件重复执行竟然是因为label

最近在做一个政企满意度调查问卷类的项目 问卷中的单选、多选框要求用美化的图标图片取代默认单选复选框 所以我又用原生JS封装了一个小功能。。。是的 又是原生JS 。
这个功能具备点击渲染图标 改变input的checked属性值 并能够把已选选项的ID存取到隐藏变量内 先看一下HTML结构和CSS样式:

<!--复选框-->
    <div  onclick=javascript:change_icon(this,"x",2); >
        <span >
            <a class="checkbox_bg" href="#"></a>
            <input type="checkbox" id="x_a1" style="display:none;"  />
        </span>
        <label for="x_a1">AAAAAAAA</label>
    </div>
    <!--隐藏变量-->
    <input type="hidden" id="x_ansid" />

    <!--单选框-->
    <div  onclick=javascript:change_icon(this,"x",1);>
        <span>
            <a class="radio_bg" href="#"></a>
            <input type="radio" id="r_a1" style="display:none;"  />
        </span>
        <label for="r_a1">000000000</label>
    </div>
    
<style>
        .checkbox_bg{
         display:inline-block;
        width:20px;height:20px;
        background:url("img/checkbox_false.png") no-repeat left center; 
        background-size:20px 20px;
          }

          .radio_bg{
            display:inline-block;
            width:20px;height:20px;
            background:url("img/checkradio_false.png") no-repeat left center; 
            background-size:20px 20px;
          }
</style>

当然结构不一定要这样写 也可以把label包住input 。。。接下来就是给大父级DIV添加点击事件 在这里我把单选和复选用一个标识参数区分开并融合到一个函数里 并且因为我们后台要求的数据格式是“,ID1,ID2,ID3,ID4,...,"所以在这里存储的时候做了一下字符串处理 请看函数

<script>
function change_icon(_this,mainid,bs){
    var now_input=_this.getElementsByTagName('input')[0]; 
    var now_a=_this.getElementsByTagName('a')[0];
    var nowid=now_input.id;
    var checked_value=var addid=document.getElementById(mainid+'_ansid');//隐藏变量
    var addid=checked_value.value;
        if(bs==2){ //多选
            if(now_input.checked==false){
                now_a.style.background='url(img/checkbox_true.png) no-repeat left center';
                now_a.style.backgroundSize='20px 20px';
                now_input.checked=true;
                if (addid == null || addid == ''){
                     checked_value.value=addid+','+nowid.split("_")[1]+',';
                }else{  
                     checked_value.value=addid+nowid.split("_")[1]+',';     
                } 
            }else {
                now_a.style.background='url(img/checkbox_false.png) no-repeat left center';
                now_a.style.backgroundSize='20px 20px';
                now_input.checked=false;
                checked_value.value=addid.replace(nowid.split("_")[1]+',',"");
            }
            if(checked_value.value==','){
                checked_value.value='';
            }
        }else{ // 单选
            if(now_input.checked==false){
                var all_a=_this.parentNode.getElementsByTagName('a');
                //先清后加
                for(var i=0;i<all_a.length;i++){
                    all_a[i].style.background='url(img/checkradio_false.png) no-repeat left center';
                    all_a[i].style.backgroundSize='20px 20px';
                    var a_input=all_a[i].nextElementSibling ||all_a[i].nextSibling;
                    a_input.checked=false;
                }
                now_a.style.background='url(img/checkradio_true.png) no-repeat left center';
                now_a.style.backgroundSize='20px 20px';
                now_input.checked=true;
                checked_value.value=nowid.split("_")[1];
            }
        }       
    }
</script>

在这里要解释一下,虽然组织在一个方法里面里,但是单选和复选的逻辑不太一样。单选的思路是当点击某一个的时候,给每一组DIV先换成false图片,在另外单独给当前这个DIV换上true的图片。而多选的逻辑是点击时如果当前这个DIV的input是false,则变为true,并换上true的图片,否则则变为false并换上false的图片。另外还有一些开存第一个和删除最后一个后的简单处理,不在赘述。

功能介绍完毕,接下来就开始填坑了,我发现多选的时候,点击某一个后总是会重复执行click事件,导致存储的数据每一个都会存两遍。但是如果把事件名换成了onmouseup,那么事件只会执行一次,数据也不会重复存储了。但是。。。测试的过程中发现onmouseup事件没有onclick事件灵敏,onmouseup事件点击取消选择时需要多次点击才会触发,用户体验很差,这样肯定过不了。查了很多帖子,有的说onclick事件包括onmousedown和onmouseup组合事件,所以会出发两次,但是拿一个很简单的例子click试一下也不会执行两遍,我都开始怀疑三观了。。。结果在我准备静下心来重新写一遍这个功能的时候,我不小心忘记了给label用for绑定input的ID,所以,就很乌龙了。。。然后才查到了问题的核心,“原因应该就是label与input绑定了的缘故,点击label等效于点击input。点击label,冒泡触发一次父元素的click,然后label默认又会触发绑定input的click事件,又一次触发父元素的事件。”

当然,解决label的问题的正经方法肯定是像我一样忘记写for,查到了几种解决方法的思路:
一、最简单粗暴的方法就是阻止input的点击事件:
<input type="checkbox" id="x_a1" style="display:none;" onclick=javascript:return false; />

二、转换布局思路,让input覆盖在label标签上,使点中对象只能是input标签

三、在label>input结构下,为了避免重复点击事件,将点击事件直接加到input中即可。对应的CSS:
label{cursor: pointer;background-color: #f00;}
label input{margin-right: -13px;width:13px;height: 13px;opacity: 0;filter: alpha(opacity=0);}
input标签的css规则,是为了保证不隐藏input的情况下,又能使input不占位,并且在页面上透明显示,从而保证ie7/8也能像chrome一样能正常响应input的点击事件!ie7下默认的input宽高是20px,标准浏览器是13px,因要兼容处理,margin-right才能统一写-13px。

反反复复终于解决了这个小Bug,虽然是一个小功能,但是我发现在用原生JS开发可以接触最底层的逻辑和算法,每一个细小的功能,都是自己组织起来的,遇到问题时,应该是第一时间找出解决办法,但是对于一名有着高远追求的程序员,最欣慰的不是找到一段能够解决bug的方法,而是自己能够有时间静下来去钻研摸索其中的问题核心所在,依然痛并快乐着。

最后编辑于
©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 162,050评论 4 370
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 68,538评论 1 306
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 111,673评论 0 254
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 44,622评论 0 218
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 53,047评论 3 295
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 40,974评论 1 224
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 32,129评论 2 317
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 30,893评论 0 209
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 34,654评论 1 250
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 30,828评论 2 254
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 32,297评论 1 265
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 28,619评论 3 262
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 33,326评论 3 243
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 26,176评论 0 8
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 26,975评论 0 201
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 36,118评论 2 285
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 35,909评论 2 278

推荐阅读更多精彩内容