本文共 1689 字，大约阅读时间需要 5 分钟。

需求是这样的，从子节点寻找指定className的父节点，一开始就想到递归（笨！），Dom结构如下：

先通过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获取后直接返回，而不用回到最外层函数域。避免了全局变量，同时优化了递归运算。