说来很惭愧,今天因为form表单的一个小问题困扰了一下午。虽然最终得以解决,但花费的时间实在是令人汗颜,现在总结一下遇到的问题。
先说一下背景,之前帮一个朋友的项目写了一个原生js的表单验证,今天突然被找到说表单验证无效。虽然很震惊我写的代码有这种事(毕竟当时学js时间不长,而且距离当时近五个月了),但作为一个有始有终的人,还是要 义不容辞的调bug。得到的反馈是说表单里有两项判空验证是失效的,无论对错都会提交表单,我的第一感觉是判空的函数被改坏了,想着改回来就好了。看到源代码的情况,比我想象的要复杂,html页面中夹杂着很多后台同学自己写得js代码。当然,问题还是得解决。找到我写的表单验证代码,逐行检查,反复调试,都没问题,再看调用函数关系,得到的返回值正常显示。
下面是代码结构:
window.οnlοad=function(){
odiv.οnsubmit=function(){
if(!validate()){ //validate()函数包含了多个表单验证方法
return false;//阻止表单默认提交
}
else return true;
};
}
<input οnclick="myajax()" type="submit" value="提交" /></span>
然而,即使在明确得到return值为false时,还可以正常提交,经过反复测试发现问题最终出现在<input>中,代码如下:
这里写了onclick事件,myajax()是ajax异步传输数据,那么onsubmit事件和onclick事件的执行顺序呢?
简单的测试下:
<script>
function X(){
alert(11);
return false;
}
function Y(){
alert(1);
}
</script>
<body>
<form action="#" method="post" name="form1" οnsubmit="return X();">
<input type="text" value="" />
<input οnclick="Y()" type="submit" value="提交" />
</form>
</body>
得出结论:
onclick先于onsubmit执行,若onsubmit返回值为true,执行submit;
所以合理的执行顺序很重要,否则会有不必要的错误;当然就解决先前bug来说,可以这样写:
window.οnlοad=function(){
odiv.οnsubmit=function(){
if(!validate()){ //validate()函数包含了多个表单验证方法
return false;//阻止表单默认提交
}
else myajax();
};
};
虽然最终还是解决了问题,但是花费了很长时间,这是很得不尝失的,所以,代码规范化很重要。
当然,不仅仅是代码看起来更加优雅、高效,更重要的是避免事件冲突,减少错误发生的可能性。