并非所有的代码都有返回值

[superhjtc]

大家好，小弟做一个asp。net项目，有一代码要搜索出菜单的一个节点：
private MenuItem FindMenuItem(MenuItem startMenu, string menuValue)
{
foreach (MenuItem mItem in startMenu.ChildItems)
{
if (mItem.Value == menuValue)
{
return mItem;
}
else if (mItem.ChildItems.Count > 0)
{
return FindMenuItem(mItem, menuValue);
}
}
}
但是代码编译不了，说是并非所有的代码都有返回值，请各位高手帮帮小弟看看该怎么解决。

[_-_-_-_-]

private MenuItem FindMenuItem(MenuItem startMenu, string menuValue)
{
foreach (MenuItem mItem in startMenu.ChildItems)
{
if (mItem.Value == menuValue)
{
return mItem;
}
else if (mItem.ChildItems.Count > 0)
{
return FindMenuItem(mItem, menuValue);
}
}
return null;
}


改成这样
当没有符合条件的..就返回一个空值

[szc21]

并非所有代码都有返回值就是你代码里面没考虑完所有的返回情况，如ＬＳ说的，后面加个return就可以了

[superhjtc]

小弟确实刚出道啊，下学期要读大三，呵呵，趁还有两年时间给小公司写些代码混点小经验。

代码中单单加个return null语法过了，逻辑还不行，今天改了改，要改成
private MenuItem FindMenuItem(MenuItem startMenu, string menuValue)
{
MenuItem temp;

foreach (MenuItem mItem in startMenu.ChildItems)
{
if (mItem.Value == menuValue)
{
return mItem;
}
else if (mItem.ChildItems.Count > 0)
{
temp = FindMenuItem(mItem, menuValue);
if (temp != null)
{
return temp;
}
}
}

return null;
}

就ok了哈。

[exiori]

返回值的层次不对。

[pccc1984]

你把return都写在if中了.不一定保证有返回值.所以要在if外写个return.返回null也好.在接受调用的地方再判断一下

[gzlaspnet]

在 foreach结束后给一个返回值。

[blueice008]

在 foreach结束后给一个返回值。

[GXY2005]

楼主刚出道？？
jf

[Zine_Alone]

该代码并不能保证能执行到return.也就是说该段代码可能出现没有返回值的情况。

foreach 如果其中的项为0呢？则根本不会进到里面去。