调试易

var cls=document.getElementById("menu"+i)if(cls.style.display=="none"){
document.cls.style.display="block";这样写的话就是document.document.getElementById("menu"+i).style.display='block'了。。

最上面的错误代码改为：
var cls=document.getElementById("menu"+i)if(cls.style.display=="none"){
cls.style.display="block";

+
1:cls.style.display="block";
2:document.style.display="block";

var cls=document.getElementById("menu"+i)if(cls.style.display=="none"){
document.cls.style.display="block";
----------------------------
第一句后cls是document.getElementById("menu"+i)对象的引用。
注意是引用传递，而不是值传递！
document.cls.style.display="block";直接写成cls.style.display="block";就行。
----------------
再看这一句
var cls=document.getElementById("menu"+i)
定一个一个变量cls,他的值是一个对象元素的引用。
在JS中定一个变量，默认属于window对象。
document.cls.style.display="block";错误是因为cls不是document的属性。
你可以
var cls=document.getElementById("menu"+i)if(cls.style.display=="none"){
window.cls.style.display="block";
window默认可省略。

js和java一样通过xx.yy来调用xx的yy属性。当yy属性不存在是报undefined对象为定义错误。

这句话var cls=document.getElementById("menu"+i)
已经把值赋给cls了
那么以后要用的话直接写cls就好了
而不是lz写的document.cls

关键误区：js中变量属于windows，而非，document

var cls=document.getElementById("menu"+i)if(cls.style.display=="none"){
cls.style.display="block";
}