• 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中的问题,如果说的不对的地方,望指正。

  • 相关阅读:
    K短路 (A*算法) [Usaco2008 Mar]牛跑步&[Sdoi2010]魔法猪学院
    [Noi2015]软件包管理器 BZOJ4196
    [SDOI2011]染色 BZOJ2243 树链剖分+线段树
    序列操作 BZOJ2962 线段树
    斜率优化入门学习+总结 Apio2011特别行动队&Apio2014序列分割&HZOI2008玩具装箱&ZJOI2007仓库建设&小P的牧场&防御准备&Sdoi2016征途
    BZOJ1854: [Scoi2010]游戏 二分图
    BZOJ3613: [Heoi2014]南园满地堆轻絮
    BZOJ4590: [Shoi2015]自动刷题机
    [JSOI2008]星球大战starwar BZOJ1015
    Rmq Problem/mex BZOJ3339 BZOJ3585
  • 原文地址:https://www.cnblogs.com/ASPNET2008/p/2002005.html
Copyright © 2020-2023  润新知