我试图从多个线程调用此方法,试图从同一个字符串中获取ID。我总是在创建SqlDataReader的行中得到此异常:
已经有一个与此命令关联的开放DataReader 必须先关闭。
我不知道问题出在哪里。我正在使用lock()语句,所以我只使用一次命令,然后我将其处理掉。有点新的数据库编程所以我不知道我的错误在哪里。
谢谢!
public int UsernameGetID(string username)
{
using (var command = new SqlCommand("SELECT user_id FROM " + ServerConstants.Database.TableUserInformation + " WHERE username = @Username", connection))
{
lock (command)
{
SqlParameter param = new SqlParameter("@Username", SqlDbType.VarChar, username.Length);
param.Value = username;
command.Parameters.Add(param);
using (SqlDataReader reader = command.ExecuteReader())
{
if (reader.Read())
{
return (int)reader[0];
}
else
{
// username doesn't exists
return 0;
}
}
}
}
}
答案 0 :(得分:9)
锁定command
是没有意义的,因为你在方法中创建了一个新命令 - 没有其他代码可以锁定它。
但是,您在多个命令之间共享连接。不要这样做 - 在每次调用中创建一个新的SqlConnection
(再次,在using
语句中)。不要担心效率方面 - 连接池将负责“真正的”网络连接。
所以你想要:
using (var connection = new SqlConnection(...))
using (var command = new SqlCommand(..., connection))
{
connection.Open();
...
using (var reader = command.ExecuteReader())
{
return reader.Read() ? (int) reader[0] : 0;
}
}