今天在新公司发现这样的代码，是不是有错误
0
悬赏园豆：10
[已解决问题]
解决于 2008-08-24 19:18
<P>public static SqlDataReader ExecuteReader(string connString, CommandType cmdType, string cmdText, params SqlParameter[] cmdParms)<BR> {<BR> SqlCommand cmd = new SqlCommand();<BR> SqlConnection conn = new SqlConnection(connString);<BR> try<BR> {<BR> PrepareCommand(cmd, conn, null, cmdType, cmdText, cmdParms);<BR> SqlDataReader rdr = cmd.ExecuteReader(CommandBehavior.CloseConnection);<BR> cmd.Parameters.Clear();<BR> return rdr;</P>
<P> }<BR> catch (Exception ex)<BR> {<BR> conn.Close();<BR> throw ex;<BR> }<BR> }</P>
<P> 怎样修改好呢，现在发现经常有超时的现象，数据库连接池已满的现象</P>
最佳答案
0
看看是不是在用完SqlDataReader之后有没有即时关闭。
另外看看超时的sql语句是不是可以优化。
其他回答(9)
0
全局用一个SqlConnection不行吗.SqlCommand.CommandTimeout 设置长一点就不超时了.
0
把数据库链接池设置大一点.
0
conn open后没有Close
0
Please check the cmdText
0
这不就是SqlHelper么……看看外面用的时候有没有SqlDataReader给Dispose掉，其他不会有问题的
0
在使用SqlDataReader的时候使用Using块括起来就不会忘记关闭了，养成这个习惯就好了
0
1:using 和try catch 双剑合一
2:优化sqlserver[查询计划]
3:优化连接池..................
0
在finally{} 里面判断是否关闭了reader和connection。如果没有，则关闭，就不会出现异常了
0
SqlDataReader rdr;
try
{
......
}
catch (Exception ex)
{
throw ex;
}
finally
{
conn.Close();
}
return rdr;
