connection这样创建的优缺点是什么
private static SqlConnection con;
/// <summary>
/// 取得连接
/// </summary>
public static SqlConnection Connection
{
get
{
string connectstring = @"Data Source=.\SQLEXPRESS;AttachDbFilename=|DataDirectory|\GameCard_data.mdf;Integrated Security=True;User Instance=True";
if (con == null)
{
con = new SqlConnection(connectstring);
con.Open();
}
else if (con.State == ConnectionState.Broken)
{
con.Close();
con.Open();
}
else if (con.State == ConnectionState.Closed)
{
con.Open();
}
return con;
}
}
[解决办法]
using System;using System.Data;using System.Data.SqlClient;using System.Threading;public partial class _Default : System.Web.UI.Page{ protected void Page_Load(object sender, EventArgs e) { for (int i = 1; i <= 10; i++) { Response.Write("start:" + i.ToString()); new Thread(new ThreadStart(MyThread)).Start(); } for (int i = 1; i <= 10; i++) { Response.Write("start:" + i.ToString()); new Thread(new ThreadStart(TestThread)).Start(); } } private static string connectstring = @"Data Source=127.0.0.1;Initial Catalog=XXXXXX;Persist Security Info=True;User ID=sa;Password=sa"; private static SqlConnection con; /// <summary> /// 取得连接 /// </summary> public static SqlConnection Connection { get { if (con == null) { con = new SqlConnection(connectstring); con.Open(); } else if (con.State == ConnectionState.Broken) { con.Close(); con.Open(); } else if (con.State == ConnectionState.Closed) { con.Open(); } return con; } } static void MyThread() { try { var sql = "SELECT * FROM table1"; SqlCommand cmd = new SqlCommand(sql, Connection); SqlDataReader reader = cmd.ExecuteReader(); while (reader.Read()) { Console.WriteLine(reader["mc"].ToString()); } reader.Close(); } catch (Exception ex) { } } static void TestThread() { var sql = "SELECT * FROM table1"; using (SqlConnection myconn = new SqlConnection(connectstring)) { myconn.Open(); SqlCommand cmd =new SqlCommand(sql,myconn); SqlDataReader reader = cmd.ExecuteReader(); while (reader.Read()) { Console.WriteLine(reader["mc"].ToString()); } reader.Close(); } }}
[解决办法]
缺点：
1.使用static，也就是全局能使用，如果有多线程，那么大家都来争这个资源，建议你可以在控制台中模拟多线程来测试一下，特别是两个线程几乎差不多时间进入此代码获取connection，你可以看看会怎样。
2.把connectstring放在代码里面，任何线程（用户）来调用的时候，都会产生新的字符串。这有点浪费内存
3.根据代码逻辑，如果con不为空，说明已经有人用过con了，所以，你判断是否Broken，但是如果不是Broken呢？比如另一个用户正在读取数据库，读到一半，而下一个用户拿着这个con去做别的事情，比如查询或者修改数据库操作或者执行存储过程，很可能会出异常
4..net里面有内置的连接池概念，而你这个思路我觉得是尽量使用一个连接，我觉得比较使用单机版的应用，最多一个人操作的。
优点： 从初学者来说，这种体现了封装思想，值得肯定。如果不是有很特殊的应用，是没必要这么做的，建议研究一下微软在数据库连接方面做了什么事情，连接池原理是怎样的，等你研究明白后，再重新看你这代码。