asp.net中的递归循环问题！急

[mastermanager]

asp.net中的递归循环问题！急

string newstr = "";
public string showtreenume_danye(int tids, string tbname, string objurl, int steeps)
{
string sql_danye = "select tname,id,tn from " + tbname + " where tn=" + tids + " order by px asc";
fx_DataReader dr_danye = new fx_DataReader(sql_danye);
while (dr_danye.Read())
{
if (tids == 0)
{
newstr = newstr + "a.add(" + (Convert.ToInt32(dr_danye["id"]) + Convert.ToInt32(steeps)) + ",0,'" + dr_danye["tname"] + "','" + objurl + dr_danye["id"] + "');\n";
}
else
{
newstr = newstr + "a.add(" + (Convert.ToInt32(dr_danye["id"]) + Convert.ToInt32(steeps)) + "," + (Convert.ToInt32(dr_danye["tn"]) + Convert.ToInt32(steeps)) + ",'" + dr_danye["tname"] + "','" + objurl + dr_danye["id"] + "');\n";
}
this.showtreenume_danye(Convert.ToInt32(dr_danye["id"].ToString()), tbname, objurl, steeps);
}
//dr_danye.Close();
return newstr;
}


这个是代码，请问这个关闭dr_danye.Close();应该在什么位置？

[mastermanager]

好了，问题已经解决了，虽然大家都很是帮忙但是一点作用都没有起，
原因出在datareader不能用嵌套循环，可以使用表格datatable。

echeng192说的对！

谢谢大家，接分去！

[草根醉秋意]

数据库操作放在递归里面是一种很危险的写法。

[mastermanager]

危险？

[飞不动]

为什么一定要用DataReader呢，用DataTable不就行了吗？

[mastermanager]

有没有高人啊！可以再加分的！急等

[fengjian_428]

dr_danye.Close();
this.showtreenume_danye(Convert.ToInt32(dr_danye["id"].ToString()), tbname, objurl, steeps);


放在递归调用的上面

[草根醉秋意]

你这个是一张表里面的树吧？代码写的太乱了，建议你去修改下表的结构或者是寻找另外的解决方法。

[指间的风]

递归需要一个出口，就是在判断是否继续执行递归的地方。

[SK_Aqi]

    public string showtreenume_danye(int tids, string tbname, string objurl, int steeps)
    {
        Response.Write(DateTime.Now.ToString() + "</br>");
        //记录时间你测试一下时间差
        string sql_danye = "select tname,id,tn from " + tbname + " where tn=" + tids + " order by px asc";
        fx_DataReader dr_danye = new fx_DataReader(sql_danye);
        while (dr_danye.Read())
        {
            if (tids == 0)
            {
                newstr = newstr + "a.add(" + (Convert.ToInt32(dr_danye["id"]) + Convert.ToInt32(steeps)) + ",0,'" + dr_danye["tname"] + "','" + objurl + dr_danye["id"] + "');\n";
            }
            else
            {
                newstr = newstr + "a.add(" + (Convert.ToInt32(dr_danye["id"]) + Convert.ToInt32(steeps)) + "," + (Convert.ToInt32(dr_danye["tn"]) + Convert.ToInt32(steeps)) + ",'" + dr_danye["tname"] + "','" + objurl + dr_danye["id"] + "');\n";
            }
            this.showtreenume_danye(Convert.ToInt32(dr_danye["id"].ToString()), tbname, objurl, steeps);
            dr_danye.Close(); //马上去关闭不影响效率好像,放这里
        }
        //dr_danye.Close();最后放这里测试一下时间是多少?
        Response.Write(DateTime.Now.ToString() + "</br>");
        return newstr;
    } 

[mastermanager]

有道理。详细点行吗?现在我脑子一片乱麻，有急需要！

[Quote=引用 5 楼 fengjian_428 的回复:]
dr_danye.Close();
this.showtreenume_danye(Convert.ToInt32(dr_danye["id"].ToString()), tbname, objurl, steeps);
       

放在递归调用的上面
       
[/Quote]

[mastermanager]

有道理。详细点行吗?现在我脑子一片乱麻，有急需要！

[Quote=引用 7 楼 freesarge 的回复:]
递归需要一个出口，就是在判断是否继续执行递归的地方。
[/Quote]

上一个引用错了，不好意思！

[fengjian_428]

出口就是while (dr_danye.Read())

代码乱 但是功能应该正常
问题是你到底想问什么

[mastermanager]

代码我简化了下

string newstr = "";
public string showtreenume_danye(int tids)
{
string sql_danye = "select * from aaa";
fx_DataReader dr_danye = new fx_DataReader(sql_danye);
while (dr_danye.Read())
{

if (tids == 0)
{
newstr = newstr + "111";
}
else
{
newstr = newstr + "222";
}
string myid = dr_danye["id"].ToString();
this.showtreenume_danye(Convert.ToInt32(myid));
//dr_danye.Close();
}
return newstr;
}

我就像问下这个dr_danye.Close();应该怎么放在那个位置才能达到最正确的关闭

[liaoyukun111]

JF

[bbsxun]

感觉你写的代码很乱，其实你可以引用SQLhelp.cs里边的ExcutReader()在调用前采用USing不就成了，何必自己写数据连接块呢？

[fengjian_428]

    string newstr = "";
    public string showtreenume_danye(int tids)
    {
        string sql_danye = "select * from aaa";
        fx_DataReader dr_danye = new fx_DataReader(sql_danye);
        while (dr_danye.Read())
        {

            if (tids == 0)
            {
                newstr = newstr + "111";
            }
            else
            {
                newstr = newstr + "222";
            }
            string myid = dr_danye["id"].ToString();
            

            this.showtreenume_danye(Convert.ToInt32(myid));
           }
           dr_danye.Close();
        return newstr;
    }
刚才发的错了 想想应该放这里

用完一个关一个

[a76477140]

建议不要这样子写啊！

[lishunli86]

用完一个关一个
这个安全