Codematic的mysqlhelper问题

      Codematic,我想有一部分朋友应该用过，我最近使用CodematicDemoS3这个项目中的mysql数据库帮助类，但发现在处理数据库连接以及异常上有比较明显的错误，不知道是我下载的这个版本问题还是Codematic一直都这么写数据库操作类。
  
     大家看下类似如下代码，代码并不一定完成相同，但类似的问题是相同的(可以去下载一个来查找类似代码)，能看出问题来吗？
    

  ///   <summary>
         ///  执行一条计算查询结果语句，返回查询结果（object）。
         ///   </summary>
         ///   <param name="SQLString"> 计算查询结果语句 </param>
         ///   <returns> 查询结果（object） </returns>
         public   static   object  GetSingle( string  SQLString,  params  MySqlParameter[] cmdParms)
        {
             using  (MySqlConnection connection  =   new  MySqlConnection(connectionString))
            {
                 using  (MySqlCommand cmd  =   new  MySqlCommand())
                {
                     try
                    {
                        PrepareCommand(cmd, connection,  null , SQLString, cmdParms);
                         object  obj  =  cmd.ExecuteScalar();
                        cmd.Parameters.Clear();
                         if  ((Object.Equals(obj,  null ))  ||  (Object.Equals(obj, System.DBNull.Value)))
                        {
                             return   null ;
                        }
                         else
                        {
                             return  obj;
                        }
                    }
                     catch  (MySql.Data.MySqlClient.MySqlException e)
                    {
                         throw  e;
                    }
                }
            }
        }

        
        我刚开始使用这个帮忙类时，也没看它的源码，但发现有些情况下，数据库连接不自动关闭，调试之后，决定定位源码查看原因，就发现类似上面的代码，在程序发生异常时，帮助类直接向外抛异常，这就导致打开的数据库连接不能关闭，如果是一个轮询查询，每次都不关闭数据库连接，不出一会，你的数据库就too many conn了。我实在是不能理解，在这个开源项目中为什么会出现这种代码。正确的做法是在throw之前，进行数据库连接的关闭，这个做法在DbHelperMySQL.cs中也存在，但不统一，即有的是按这种标准，有的不是。

 

       我再列举一个平时可能会忽视的一个错误写法：这种错误和上面的处理方式应该是一样的，当while循环发生错误时，如果不在抛出异常前关闭dr，此数据库连接就不会即时释放。

      

   using  (MySqlDataReader dr  =  DbHelperMySQL.ExecuteReader(strSql.ToString(),  null ))
            {
                 try
                {
                     while  (dr.Read())
                    {
                         // ...

                    }
                    dr.Close();
                }
                 catch  (Exception ex)
                {
                     throw  ex;
                }

            }

 

    
        思考：上面在throw前不手动关闭数据库连接，是不是受到了using的影响，认为用了using就肯定关闭连接，但在throw后，实际上程序就不会进入finall块，也就不会关闭连接了。
           
        总结：针对数据库的连接一定要仔细对待，在thorw之前保证有效关闭数据库连接，针对Codematic中的问题，如果说的不对的地方，望指正。

转载于:https://www.cnblogs.com/ASPNET2008/archive/2011/04/01/2002005.html