有两个for循环语句，里面都对数据库查询，速度很慢，如何优化？

[fxs320]

如题，

  if (ds.Tables.Count > 0)
                    {
                        int cpidcount;
                        int cpcount;
                        for (int m = 0; m < ds.Tables[0].Rows.Count; m++)
                        {
                         
                            if (int.TryParse(ds.Tables[0].Rows[m][1].ToString(),out cpcount))
                            {
                                if (cpcount > 1)
                                {
                                    for (int j = 0; j < cpcount; j++)
                                    {
                                        TableRow tbcpsec = new TableRow();
                                        TableCell tbcpidsec = new TableCell();
                                        TableCell tbcpvlsec = new TableCell();
                                        cpidcount = j + 1;
                                        tbcpidsec.Text = ds.Tables[0].Rows[m][0].ToString() + cpidcount + ":";
                                        tbcpvlsec.Text = Production.GetCompentIDBySNAndPD(txtSN.Text, ds.Tables[0].Rows[m][0].ToString() ,j);
                                        tbcpsec.Cells.Add(tbcpidsec);
                                        tbcpsec.Cells.Add(tbcpvlsec);
                                        Table1.Rows.Add(tbcpsec);
                                        cpID.Add(ds.Tables[0].Rows[m][0].ToString() + cpidcount);
                                        cpValue.Add (Production.GetCompentIDBySNAndPD(txtSN.Text, ds.Tables[0].Rows[m][0].ToString(), j));
                                    }

                                }
                                else
                                {
                                    TableRow tr = new TableRow();
                                    TableCell tcID = new TableCell();
                                    TableCell tcIDVl = new TableCell();
                                    tr.Cells.Add(tcID);
                                    tr.Cells.Add(tcIDVl);
                                    tcID.Text = ds.Tables[0].Rows[m][0].ToString() + ":";
                                    tcIDVl.Text = Production.GetCompentIDBySN(txtSN.Text, ds.Tables[0].Rows[m][0].ToString());
                                    Table1.Rows.Add(tr);
                                    cpID.Add(ds.Tables[0].Rows[m][0].ToString());
                                    cpValue.Add(Production.GetCompentIDBySN(txtSN.Text, ds.Tables[0].Rows[m][0].ToString()));
                                }
                               
                            }
                          

                        }

[丰云]

依照你的代码体现的业务来看，

没有办法优化。。。。

[antiking]

查询显示用用sqldatareader

[demoooo]

不知道业务情况

如果数据是关联的,可以一次性查询出来,在数据集中for, 不要for着去查询数据.一旦cpcount很大,速度必定就很慢

[李班头]

很难优化。

[Justin-Liu]

如果优化的话就从查询下功夫吧 进而减少一个循环 但不知你的业务需求是否可行

[ckl881003]

把数据一次查出来。。别查这么多次。。

[cuike519]

根本的方法是优化设计方案吧或者算法。

如果只是修改现在的代码，数据库部分你应该缓存到哈希表中，如果量太大还是考虑第一种方法吧。

[烈火晴天]

是你机器慢吧!

[jianshao810]

如果优化的话就从查询下功夫吧 进而减少一个循环 但不知你的业务需求是否可行

[sxiaohui8709253]

可能需要转变思路

[zzxap]

 

if (ds.Tables.Count > 0)
                    {int m = 0;
                        int cpidcount;
                        int cpcount;
                         int a=ds.Tables[0].Rows.Count;
                                   
                                 
                        for ( m = 0; m <a; m++)
                        {
                         
                            if (int.TryParse(ds.Tables[0].Rows[m][1].ToString(),out cpcount))
                            {
                                if (cpcount > 1)
                                {
                                    for (int j = 0; j < cpcount; j++)
                                    {TableRow tbcpsec = new TableRow();
                                        TableCell tbcpidsec = new TableCell();
                                        TableCell tbcpvlsec = new TableCell();
                                        cpidcount = j + 1;
                                        tbcpidsec.Text = ds.Tables[0].Rows[m][0].ToString() + cpidcount + ":";
                                        tbcpvlsec.Text = Production.GetCompentIDBySNAndPD(txtSN.Text, ds.Tables[0].Rows[m][0].ToString() ,j);
                                        tbcpsec.Cells.Add(tbcpidsec);
                                        tbcpsec.Cells.Add(tbcpvlsec);
                                        Table1.Rows.Add(tbcpsec);
                                        cpID.Add(ds.Tables[0].Rows[m][0].ToString() + cpidcount);
                                        cpValue.Add (Production.GetCompentIDBySNAndPD(txtSN.Text, ds.Tables[0].Rows[m][0].ToString(), j));
                                    }

                                }
                                else
                                {
                                     TableRow tr = new TableRow();
                                    TableCell tcID = new TableCell();
                                    TableCell tcIDVl = new TableCell();
                                    tr.Cells.Add(tcID);
                                    tr.Cells.Add(tcIDVl);
                                    tcID.Text = ds.Tables[0].Rows[m][0].ToString() + ":";
                                    tcIDVl.Text = Production.GetCompentIDBySN(txtSN.Text, ds.Tables[0].Rows[m][0].ToString());
                                    Table1.Rows.Add(tr);
                                    cpID.Add(ds.Tables[0].Rows[m][0].ToString());
                                    cpValue.Add(Production.GetCompentIDBySN(txtSN.Text, ds.Tables[0].Rows[m][0].ToString()));
                                }
                               
                            }
                          

                        }

尽量不要再for循环内定义变量

[mngzilin]

用foreach代替for

[段传涛]

可以试试 foreach

[Ray_Yang]

加内存 升级CPU

[moonwrite]

Production.GetCompentIDBySN(txtSN.Text, ds.Tables[0].Rows[m][0].ToString());
cpValue.Add(Production.GetCompentIDBySN(txtSN.Text, ds.Tables[0].Rows[m][0].ToString()));

想比这两个总是Open()+Close() 这是很耗
可以尝试把Open和Close()放在外面 反正就是尽量减少Open()+Close() 的次数

[Teng_s2000]

1.使用SqlDataReader读取数据,然后绑定
2.对数据进行分页操作,在优化,如果你一下将100W的纪录一起显示到web上也会超时