利用Cache解决连接池占满的问题

 

  今天在实现一个功能时候发现，没过多久页面就抛出异常，提示连接池已用完，仔细查看了连接数据库的代码，发现connection也都close了，找了半天，发现还是跟我的应用模式有关，分析如下：

 

  功能描述：页面动态生成一个table，并用JS控制每一行的onmouseover事件，动态设置<image>的src属性，根据不同的条目显示不同的图片，因为我的图片都是存在数据库里的，所以需要一个页面（readImg.aspx）来动态生成

 

  原因分析：每一页大约20行数据，这样用户在表格上来回移动鼠标的时候，就不停的在加载readImg.aspx页面，造成了不停的打开数据库连接，即使我显示的关闭了，但用sp_who查看有很多连接处于sleeping的状态。没过多久就把连接池耗尽了。

 

  解决方法：因为这部分数据都是公用的，并且极少变化，所以采用Cache似乎是个不错的方法，更改了下readImg.aspx的部分代码如下：

 

1. private void Page_Load(object sender, System.EventArgs e)
2.         {
3.             // 在此处放置用户代码以初始化页面
4.             string userName = Request.QueryString["username"].ToString();
5.             if(Cache["photo" + userName]!=null)
6.             {
7.                 Response.BinaryWrite((byte[])Cache["photo" + userName]);
8.             }
9.             else
10.             {
11.                 ReadImage(userName);
12.             }
13.             
14.         }
15. private void ReadImage(string userName)
16.         {
17.             SqlConnection conn = new SqlConnection(Class.Common.getDBConnStr());
18.             
19.             SqlCommand cmd = new SqlCommand();
20.             cmd.Connection = conn;
21.             cmd.CommandText = "select photo from  RL_ITBC_HM_Employees where UserName='"+userName+"' and photo is not null";
22.             conn.Open();
23.             SqlDataReader sr = cmd.ExecuteReader();
24.             
25.             string cStr = "photo" + userName;
26.             
27.             if(sr.Read())
28.             {
29.                 //bool kk = sr.HasRows;
30.                 
31.                 Cache[cStr] = (byte[])sr["photo"];
32.                 
33.                 Response.BinaryWrite((byte[])sr["photo"]);
34.                 
35.                 
36.             }
37.             else
38.             {
39.                 byte[] l = {00000000};
40.                 Cache[cStr] = (byte[])l;
41.             }
43.             conn.Close();
44.             conn.Dispose();
45.             sr.Close();
46.             
47.         }

这样每次载入前，先判断是否存在Cache，存在的话直接显示，没有的话再读取数据库。

 

经测试，修改后的代码数据库连接占用很少，基本解决了问题。

 

总结如下：

 

1，Cache属于全局变量，适合于公共数据且变化很少的部分。

 

2，SqldataReader对象一定要显示的关闭，而不仅是关闭SqlConnection，例如 sr.Close();