需求是这样的,从子节点寻找指定className的父节点,一开始就想到递归(笨!),Dom结构如下:
<div class="layer_1">
<div class="layer_2">
<div class="layer_3">
<div id="layer_4"></div>
</div>
</div>
</div>
先通过id获得layer_4的div,然后逐层向上寻找最外层的layer_1,一开始我试图用如下递归获取:
1 function getNode(){
2 var child = $("#layer_4");
3 var parent = getParent(child);
4 return parent;
5 }
6
7 function getParent(el){
8 var result;
result = el.parentNode;
9 if(!el || el === document.documentElement || el.parentNode === document.documentElement){
10 return;
11 }else if(result && result.className === "layer_1"){
12 return result;
13 }else{
14 getParent(result);
15 }
16 return result;
17 }
18
19 getNode(); //undefined
结果返回的是undefined!
本来是一最基本的递归,为什么会出现这种结果?
其实修改这个问题很简单,目前我只想到一个办法:将result声明为全局变量!
当然这个方法的缺点是造成了memory leak,折中的解决办法是在获取到result后将result =null。
可能有朋友看到这里就已经知道这个问题的原因了,那就是:JavaScript中function的作用域问题-闭包!下面详细解释一下。
如果按照上面的写法,
1、每次递归调用getParent()方法是都会声明一个局部变量result,同时因为闭包的缘故,每次的gerParent()的运行作用域又保留着上次getParent()的作用域,所以每次都会覆盖上层同名的result,作为一个当前函数域的局部变量;
2、当找到layer_1后,result更新,return result得到了我们想要的结果,跳出本次函数域,进入上层函数域,但此时的上层函数域中result并未更新(因为被下层函数域的同名result屏蔽了),所以此时最外层的result仍然是undefined!
所以最终将的到undefined!
这个问题同样引出了以前遇到的关于return的bug,当时把return想象的太强大了,以为return会跳出整条作用域链,上述问题证明了return只能跳出当前作用域,以后注意!
补充:谢谢亮哥的指导,用全局变量解决确实是最笨的法子了,以下是改进办法:
function getParent(el){
var result;
result = el.parentNode;
if(!el || el === document.documentElement || el.parentNode === document.documentElement){
return;
}else if(result && result.className === "layer_1"){
return result;
}else{
return getParent(result);
}
}
在每次递归调用时用return跳出当前函数域,之后进入下层函数时result获取后直接返回,而不用回到最外层函数域。避免了全局变量,同时优化了递归运算。