我有一个静态类数据:
public static class Data
{
public static SqlConnection connexion;
public static bool Connect()
{
System.Data.SqlClient.SqlConnectionStringBuilder builder = new System.Data.SqlClient.SqlConnectionStringBuilder();
builder["Initial Catalog"] = "Upload";
builder["Data Source"] = "base";
builder["integrated Security"] = true;
string connexionString = builder.ConnectionString;
connexion = new SqlConnection(connexionString);
try { connexion.Open(); return true; }
catch { return false; }
}
public static void Disconnect()
{
if (connexion != null) connexion.Close();
connexion = null;
}
}
在动作主页中:
public ActionResult Home()
{
Data.Connect();
if (CompteModels.Connected)
{
ArrayList model = new ArrayList();
ClientModels clients = new ClientModels();
model.AddRange(clients.Client_List());
AdminModels admins = new AdminModels();
model.AddRange(admins.Admin_List());
return View(model);
}
else return RedirectToAction("Login", "Account");
}
客户端类:
public List<ClientModels> Client_List()
{
List<ClientModels> l = new List<ClientModels>();
using (Data.connexion)
{
string queryString = @"select Login, Password, Mail, Name, Tentatives from Compte where User_type_id in ( select Id from User_type where Fonction = 'Client')";
SqlCommand command = new SqlCommand(queryString, Data.connexion);
try
{
SqlDataReader reader = command.ExecuteReader();
do
{
while (reader.Read())
{
ClientModels admin = new ClientModels { Login = reader.GetString(0), Password = reader.GetString(1), Mail = reader.GetString(2), Name = reader.GetString(3), Tentatives = reader.GetInt32(4) };
l.Add(admin);
}
} while (reader.NextResult());
return l;
}
catch { return null; }
}
对于函数AdminList
,实现与Client_List
相同,但在课程Admin
中。
问题在于静态变量connexion
:在第一个函数Client_List
中,它的值是正确的,我得到了客户端列表,但在第二个函数中它变为null
尽管它是静态类中的静态变量!!!
这种改变的原因是什么?我该如何解决?
答案 0 :(得分:4)
您要么在某处将connexion
设置为null
,要么在使用前将其初始化。
最有可能的是,一个类调用Disconnect
将connexion
设置为null,而另一个类则认为它不为null并尝试使用它。
正如评论中所提到的,保持对像SqlConnection
这样的资源的静态引用并不是一个好主意。如果要重用代码,可以创建静态函数,返回新 SqlConnection
实例并生成连接字符串静态,但对整个网站共享的连接进行静态引用会给你带来更多值得的问题(正如你已经看到的那样)。
在静态函数中执行此操作的一种方法是:
public static SqlConnection GetConnection()
{
System.Data.SqlClient.SqlConnectionStringBuilder builder = new System.Data.SqlClient.SqlConnectionStringBuilder();
builder["Initial Catalog"] = "Upload";
builder["Data Source"] = "base";
builder["integrated Security"] = true;
string connexionString = builder.ConnectionString;
connexion = new SqlConnection(connexionString);
return connexion;
}
您的客户端代码将类似于:
using (SqlConnection conn = Data.GetConnection())
答案 1 :(得分:0)
这是一个坏主意,正如其他人已经提到的那样。
您只需在需要时创建并打开连接,然后再将其处理:
using (SqlConnection connection = new SqlConnection(connectionString))
{
connection.Open();
// ... now use it
}
当然,还有其他模式可以隐藏这种机制,但在你的情况下可能会有点过分。
答案 2 :(得分:0)
您的using语句使用IDisposable接口
处理连接 public List<ClientModels> Client_List()
{
List<ClientModels> l = new List<ClientModels>();
using (Data.connexion) <--- here
}
更改此项以创建新连接
public List<ClientModels> Client_List()
{
List<ClientModels> l = new List<ClientModels>();
using (var connection = Data.CreateConnexion())
}
与此相似
public static class Data
{
public static SqlConnection CreateConnection()
{
System.Data.SqlClient.SqlConnectionStringBuilder builder = new System.Data.SqlClient.SqlConnectionStringBuilder();
builder["Initial Catalog"] = "Upload";
builder["Data Source"] = "base";
builder["integrated Security"] = true;
string connexionString = builder.ConnectionString;
var connexion = new SqlConnection(connexionString);
connexion.Open();
return connexion;
}
}