DATAREADER没有关闭的问题

[fskjb01]

请各位帮我看看为什么说链接没有关闭呢，以确定是//以下有问题和//以上有问题之间的代码有问题，就是不知道该如何解决啊。。

//监控数据库
        private void tmrDB_Tick(object sender, EventArgs e)
        {
            ShowFriendList();       // 刷新好友列表
            int MessageTypeId = 1;  // 消息类型
            int MessageState = 1;   // 消息状态
            string sql = string.Format("select top 1 Id,FromUserId,MessageTypeId,MessageState from Messages where MessageState=0 and ToUserId={0}", UserHelper.loginId);
            try
            {
                DBhelper.connection.Open();
                DBhelper.datareader = DBhelper.command(sql).ExecuteReader();                
                if (DBhelper.datareader.Read())
                {
                    MessageTypeId = (int)DBhelper.datareader["MessageTypeId"];
                    MessageState = (int)DBhelper.datareader["MessageState"];               
                    fromUserId = (int)DBhelper.datareader["FromUserId"];
                    messageId = (int)DBhelper.datareader["Id"];
                    friendFaceId = GetfriendFaceId(fromUserId);

                }
                
                DBhelper.datareader.Close();
            }
            catch (Exception ex)
            {

                MessageBox.Show(ex.Message);
            }
            finally {
                DBhelper.connection.Close();
            }

            // 如果是添加好友消息让小喇叭闪烁
            if (MessageTypeId == 2)
            {
                SoundPlayer player = new SoundPlayer("system.wav");
                player.Play();
                tmrAddFriend.Start();
            }
            // 如果是聊天消息让好友头像闪烁
            else if (MessageTypeId == 1)
            {
                //以下有问题                
                sql = string.Format("select FaceId from users where Id={0}", fromUserId);
                try
                {
                    DBhelper.connection.Open();
                    friendFaceId = (int)DBhelper.command(sql).ExecuteScalar();
                }
                catch (Exception ex)
                {

                    MessageBox.Show(ex.Message);
                }
                finally
                {
                    DBhelper.connection.Close();
                }
                //以上有问题
                
                // 如果发消息的人没有在列表中就添加到陌生人列表中
                if (!isContainInList(fromUserId))
                {
                    AddToStranger(fromUserId);
                }
                SoundPlayer player = new SoundPlayer("msg.wav");
                player.Play();
                tmrface.Start(); 
            }
        }

[一游]

那你一步一步调度，看哪里在打开后没有关闭就可以了

[HDNGO]

SqlDataReader myReader = cmd.ExecuteReader(CommandBehavior.CloseConnection);

[nbgcqlk]

要DBhelper.datareader.dispose();

[fskjb01]

我以锁定有问题的代码了，但问题是connectin已经关闭了啊！！

[Zhanlixin]

重新实例化new DBhelper.connection一次

[qinhl99]

首先DBhelper中的connection作为一个静态全局变量非常的不好,应该作为每一个方法的局部变量!
你这样写,会经常遇到意想不到的问题!

[fskjb01]

晕，问题解决啦，重新生产一下解决方案就搞定了，怎么会事呢？？？