Соединения SQL Server в приложении ASP.NET

Question

Я создал веб-приложение с asp.net, примерно 100 users используют его.

Однако время от времени люди получают сообщение об ошибке, что соединение по-прежнему открыто. Указывая, что он не был закрыт должным образом.

Он появляется в случайных местах, а не в одном конкретном месте и никаких других ошибок перед ним.

Я знаю, что когда у меня есть ошибка в приложении, и он сбой без моего изящества, связанного с ошибкой, соединение остается открытым, и в основном все из-за него будут разбиваться. Это заставило меня подумать, что каждый использует тот же объект соединения, возможно ли, что 2 пользователя могут иметь идеальное время и вызвать функцию, используя DB connection, в то же время вызывая ошибку? Есть ли способ убедиться, что все используют свои собственные объекты соединения, например, помещают их в свою сессию или что-то еще?

Надеюсь, вы поймете, что я имею в виду, я не думаю, что публикация моего кода поможет, поскольку это происходит в случайных местах моего проекта.

Это подключение к SQL Server с использованием System.Data.SqlClient.

Найдите ниже функцию, которая генерирует ошибку. Эта функция вызывается Page_Load, перед ней ничего нет.

public static SqlConnection conn = new SqlConnection("Data Source=Server00\\SQLEXPRESS;Initial Catalog=r2;Integrated Security=true;Connect Timeout=0"); 

private void populateGameDrop() 
{ 
    try 
    { 
     conn.Open(); 
     drop_game.Items.Clear(); 

     SqlCommand cmd = conn.CreateCommand(); 
     Access ac = (Access)Session["Access"]; 
     cmd.CommandText = "Select * from dbo.Games where " + ac.GameQuery; 

     SqlDataReader r = cmd.ExecuteReader(); 

     while (r.Read()) 
     { 
      drop_game.Items.Add(new ListItem(r["name"].ToString(), r["Abbr"].ToString())); 
     } 

     conn.Close(); 
    } 
    catch (Exception exc) 
    { 
     conn.Close(); 
     Log.Error(exc.ToString()); 
     Session["Error"] = exc.ToString(); 
     Response.Redirect("~/YouBrokeIt.aspx"); 
    } 

    populateServers(); 
    SetSplitScreen(); 
} 

Answer 1

Не пытайтесь разделить SqlConnection объекты.

Попробуйте вместо этого:

private static string connString = "Data Source=Server00\\SQLEXPRESS;Initial Catalog=r2;Integrated Security=true;Connect Timeout=0"; 
private void populateGameDrop() 
{ 
    try 
    { 
     using (var conn = new SqlConnection(connString)) 
     { 
      conn.Open(); 
      drop_game.Items.Clear(); 
      using (var cmd = conn.CreateCommand()) 
      { 
       Access ac = (Access)Session["Access"]; 
       //TODO 
       //TODO - Introduce parameters to avoid SQL Injection risk 
       //TODO 
       cmd.CommandText = "Select name,Abbr from dbo.Games where " + ac.GameQuery; 
       using(SqlDataReader r = cmd.ExecuteReader()) 
       { 
        while (r.Read()) 
        { 
         drop_game.Items.Add(new ListItem(r["name"].ToString(), 
              r["Abbr"].ToString())); 
        } 
       } 
      } 
     } 
    } 
    catch (Exception exc) 
    { 
     Log.Error(exc.ToString()); 
     Session["Error"] = exc.ToString(); 
     Response.Redirect("~/YouBrokeIt.aspx"); 
    } 
    populateServers(); 
    SetSplitScreen(); 
} 

За кулисами, .NET использует концепцию под названием connection pooling, так что фактическое число реальных подключений к SQL Server сведены к минимуму. Но SqlConnection объектов не предназначены для совместного использования несколькими потоками.

Answer 2

Ваши SQL соединения не должны быть статичными, используйте следующие действия для создания их

var connectionString = "YOUR CONNECTION STRING"; 
var queryString = "SQL QUERY"; 

using (SqlConnection connection = new SqlConnection(connectionString)) 
using (SqlCommand command = new SqlCommand(queryString, connection)) 
using (SqlDataReader dateReader = command.ExecuteReader()) { 

} 

ОЧЕНЬ ВАЖНО

Вы должны использовать параметризацию SQL код открыт для SQL-инъекций ,

см Parameterize SQL query

Answer 3

Не размещайте код базы данных непосредственно на страницах ASPX. Создание дополнительного слоя (т. Е. DAL) позволяет протестировать методы БД без использования страницы.

Попробуйте что-нибудь подобное.

//Don't embed database logic directly in the aspx files 
public class GamesProvider 
{   
    //Put the ConnectionString in you configuration file 
    private string ConnectionString 
    { 
     get { return ConfigurationManager.ConnectionStrings["GameDB"].ConnectionString; } 
    } 

    public IEnumerable<Game> LoadGames(string x, string y) 
    { 
     var games = new List<Game>(); 

     const string queryString = "select name, Abbr from dbo.Games where x = @x and y = @y"; 

     using (var connection = new SqlConnection(ConnectionString)) 
     using (var command = new SqlCommand(queryString, connection)) 
     { 
      command.Parameters.AddWithValue("@x", x); 
      command.Parameters.AddWithValue("@y", y); 
      using (var dateReader = command.ExecuteReader()) 
      { 
       while (dateReader.Read()) 
       { 
        var game = new Game 
        { 
         Name = dateReader["name"].ToString(), 
         Abbr = dateReader["Abbr"].ToString(), 
        }; 
        games.Add(game); 
       } 
      } 
     } 

     return games; 
    } 
} 

//Use types 
public class Game 
{ 
    public string Name { get; set; } 
    public string Abbr { get; set; } 
}