一小段JS代码在火狐下不好使，请哪位高手指点下

Question

function showsubmenu(sid)
{
whichEl = eval("submenu" + sid);
imgmenu = eval("imgmenu" + sid);
if (whichEl.style.display == "none")
{
eval("submenu" + sid + ".style.display=\"\";");
imgmenu.background="../internet/images/nei_17.gif";
}
else
{
eval("submenu" + sid + ".style.display=\"none\";");
imgmenu.background="../internet/images/nei_21.gif";
}
}

这段代码是关于后台左侧竖向二级菜单的隐藏显示的JS，在IE6、7、8下都好使，在火狐下失效了，俺不会写JS，不知怎么弄才能让这段代码在火狐下也好用。请高手指点下。谢谢啦。。。

Answer 1

function showsubmenu(sid)
{
var whichEl = document.getElementById("submenu" + sid)||null, imgmenu = document.getElementById("imgmenu" + sid)||null;
if (!whichEl || !imgmenu) return false;
whichEl.style.display = whichEl.style.display && true ? "" : "none";
imgmenu.background = whichEl.style.display && true ? "../internet/images/nei_17.gif" : "../internet/images/nei_21.gif";
return false;
}

Answer 2

　　您好！很高兴为您答疑！

兼容问题，稍作修改后如下：
function showsubmenu(sid)
{
var whichEl = document.getElementById("submenu" + sid)||null, imgmenu = document.getElementById("imgmenu" + sid)||null;
if (!whichEl || !imgmenu) return false;
whichEl.style.display = whichEl.style.display && true ? "" : "none";
imgmenu.background = whichEl.style.display && true ? "../internet/images/nei_17.gif" : "../internet/images/nei_21.gif";
return false;
}
　　您可以在火狐社区了解更多内容。希望我的回答对您有所帮助，如有疑问，欢迎继续在本平台咨询。