for循环 返回值问题

[ayiosi]

代码如下：
------------------------------------------------------
public bool IsAdmin(int emp_id)
{
            DataSet ds = dal.GetList("emp_id=" +emp_id);
            DataTable dt=ds.Tables[0];
            int count =dt.Rows.Count;
            int[] role = new int[count];
                for (int i = 0; i < count; i++)
                {
                    if (dt.Rows[i]["role_id"].ToString() != "")
                    {
                        role[i] = Convert.ToInt32(dt.Rows[i]["role_id"].ToString());
                        if (role[i] == 4)
                        {
                            return true;
                        }
                        else
                        {
                            return false;
                        }
                    }
                    else
                    {
                        return false;
                    }
                }
}
----------------------
为什么报“并非所有路径都有返回值”

[denbes]

[sh_lin]

看完所有贴 该说的大虾们都说 这种东西多写写就知道

[勤劳的渔网工作者]

方法最后是要返回一个Bool值的，而你却只在for循环中返回了，属于局部的，所以报错....

[dafei198607]

for外面加个return ,

[xray2005]

你这代码太不简洁了。

给你简洁点的：

public bool IsAdmin(int emp_id)
{
            DataSet ds = dal.GetList("emp_id=" +emp_id);
            DataTable dt=ds.Tables[0];
            int count =dt.Rows.Count;
            int[] role = new int[count];
                for (int i = 0; i < count; i++)
                {
                    if (dt.Rows[i]["role_id"].ToString() != "")
                    {
                        role[i] = Convert.ToInt32(dt.Rows[i]["role_id"].ToString());
                        if (role[i] == 4)
                        {
                            return true;
                        }
                     }
                }
                return false;

}

[卧_槽]

你的代码count==0的时候，没有返回值

[wclhack]

C# code代码如下：
------------------------------------------------------
public bool IsAdmin(int emp_id)
{
DataSet ds = dal.GetList("emp_id=" +emp_id);
DataTable dt=ds.Tables[0];
int count =dt.Rows.Count;
bool retvalue=false;
int[] role = new int[count];
for (int i = 0; i < count; i++)
{
if (dt.Rows[i]["role_id"].ToString() != "")
{
role[i] = Convert.ToInt32(dt.Rows[i]["role_id"].ToString());
if (role[i] == 4)
{
retvalue=true;
}
}
}
return retvalue;
}

[lhz_dxm]

public bool IsAdmin(int emp_id)
{
DataSet ds = dal.GetList("emp_id=" +emp_id);
DataTable dt=ds.Tables[0];
int count =dt.Rows.Count;
int[] role = new int[count];
for (int i = 0; i < count; i++)
{
if (dt.Rows[i]["role_id"].ToString() != "")
{
role[i] = Convert.ToInt32(dt.Rows[i]["role_id"].ToString());
if (role[i] == 4)
{
return true;
}
else
{
return false;
}
}
else
{
return false;
}
}
return false;

}

[yudi010]

罗锅

[飘渺幽冥]

for循环也有条件判断，也有不执行的可能所以在for循环外也要有return
最好有个bool的变量在方法的最后返回

[蒙戈]

首先你要对你的count判断是否大于0
然后你的代码逻辑上是有错误的，请仔细看下你的代码，那个FOR循环不管怎么样都会只执行一次，因为你FOR循环中第一种可能结果都是return了
最后说句实在话，看了楼主的代码想不通是用来做什么的。。。。。

[蒙戈]

更正下，是每一种可能都Return了

[antony1029]

一般 定义变量。最后return。

[坎离枫]

建议在for(){...}后加一个return：如果for未执行不就没返回值了？
另：
检查你的程序里的其它地方，看是否在其它地方调用了这个函数，但参数不正确。我以前就遇到过。

[walkingmu]

这个是软件工程结构化的问题。
建议楼主在for的上面定义一个要返回的变量
for循环中只是改变变量的值
for循环结束后返回这个变量

[退一步海阔天空l忍一时风平浪静]

在外面套一个if不就解决问题了？
没执行直接返回false

[ymkyniyq]

看你们的贴子有点像是在说自己几厉害一样的
不就是程序在生成的时候if else里的返回值都是有可能不返回的 因为它是分支语句有可能不执行 则会有可能没有返回值
不就这么简单
都不知道扯到那里去咯

[erydemimi]

如果不循环一次 那返回什么？？？

[ytc666]

[Quote=引用楼主 ayiosi 的回复:]
C# code
代码如下：------------------------------------------------------publicbool IsAdmin(int emp_id)
{
DataSet ds= dal.GetList("emp_id="+emp_id);
DataTable dt=ds.Tables[0];int count=dt.Rows.Count;int[] role=newint[count];for (int i=0; i< count; i++)
{if (dt.Rows[i]["role_id"].ToString()!="")
{
role[i]= Convert.ToInt32(dt.Rows[i]["role_id"].ToString());if (role[i]==4)
{returntrue;
}else
{returnfalse;
}
}else
{returnfalse;
}
}
}----------------------
为什么报“并非所有路径都有返回值”
[/Quote]

因为如果不进入for循环，就没有返回值
在for循环外加返回值即可

[小灰狼]

心寒呀

那么多给楼主讲原理的没给分,而提供代码的就给了满分.

看样子,楼主是个喜欢别人都替你做好,自己的不用动脑子的人