Блог

Я использую ASP.NET Framework 4, IIS 7 и SQL Server 2008 R2.

Я получил ошибку типа: {column} not found in selected DataSource,
SQL Reader is close , ….

Это происходит только тогда, когда:

1. Подключено несколько пользователей.
2. Они выполняют вызовы CRUD (создание, повторный поиск, обновление, удаление) одновременно.

Как ни странно, это ускользает от моего улова:

 try{
    Connexion_D.GetConnected();
    // doing CRUD
}
catch{
    // catching Error, avoid yellow page aspx
}
finally
{
    Connexion_D.CloseConnection();
} 
  

И мой класс подключения:

 public class Connexion_D
{

    static public SqlConnection conn;
    static public SqlConnection GetConnected()
    {
        try
        {
            String strConnectionString = ConfigurationManager.ConnectionStrings["xxxxx"].ConnectionString;
            conn = new SqlConnection(strConnectionString);
        }
        catch (Exception excThrown)
        {
            conn = null;
            throw new Exception(excThrown.InnerException.Message, excThrown);
        }

        // Ouverture et restitution de la connexion en cours
        if (conn.State == ConnectionState.Closed) conn.Open();
        return conn;
    }
    static public Boolean IsConnected
    {
        get { return (conn != null) amp;amp; (conn.State != ConnectionState.Closed) amp;amp; (conn.State != ConnectionState.Broken); }
    }

    static public void CloseConnection()
    {
        // Libération de la connexion si elle existe
        if (IsConnected) conn.Close();
    }
}
  

Поэтому я не думаю, что код неправильный / содержит ошибку.

Я думаю, что это может быть конфигурация IIS и SQL server.

Есть идеи?

Заранее благодарю.

Если я правильно понимаю, что вы делаете, то это выглядит очень сомнительно:

 static public SqlConnection conn;     
static public SqlConnection GetConnected() {         
    try         
    {             
        String strConnectionString = ConfigurationManager.ConnectionStrings["xxxxx"].ConnectionString;             
        conn = new SqlConnection(strConnectionString);
    } 
}

static public void CloseConnection() {           
    // Libération de la connexion si elle existe           
    if (IsConnected) conn.Close();       
}   
  

Вы работаете со статической переменной соединения, что означает, что когда вы закрываете ее, вы закрываете последнюю, которая была открыта.

В многопользовательском сценарии это может произойти:

• Пользователь A: Создайте соединение (и верните соединение 1)
• Пользователь A: Выполнить запрос (выполняется с подключением 1)
• Пользователь B: Создайте соединение (и верните соединение 2)
• Пользователь A: Закрыть соединение (последним открытым было 2, так что оно закрыто)
• Пользователь B: Выполнить запрос (выполняется для подключения 2, оно уже закрыто … взрыв)

Кроме того, вам, вероятно, следует пересмотреть использование вашего соединения в качестве общедоступной переменной-члена:

 static public SqlConnection conn;     
  

Обычно это считается плохой практикой и может привести к неожиданным / трудноотслеживаемым ошибкам в будущем, если какой-либо код за пределами вашего класса начнет возиться с его внутренними переменными.

Редактировать:

Казалось бы, наиболее очевидным решением было бы прекратить статическое соединение. Тогда ваш клиентский код мог бы выглядеть примерно так:

 try{
    // use using block around connection, calls dispose automatically when
    // block ends...
    using(var connectionWrapper = new Connexion_D()) {
        var connectedConnection = connectionWrapper.GetConnected();        
        // do CRUD
    }
}
catch{
    // catching Error, avoid yellow page aspx
    // Really you should probably be doing something with the exception (logging?)
    // particularly since you go to the effort of throwing it from your Connection_D
    // class.
}
  

Код вашего класса выглядит как:

 /* Implement IDisposable to cleanup connection */
public class Connexion_D : IDisposable 
{
    public SqlConnection conn;

    public SqlConnection GetConnected()
    {
        try
        {
            String strConnectionString = ConfigurationManager.ConnectionStrings["xxxxx"].ConnectionString;
            conn = new SqlConnection(strConnectionString);
        }
        catch (Exception excThrown)
        {
            conn = null;
            throw new Exception(excThrown.InnerException.Message, excThrown);
        }

        // Ouverture et restitution de la connexion en cours
        if (conn.State == ConnectionState.Closed) conn.Open();
        return conn;
    }
    public Boolean IsConnected
    {
        get { return (conn != null) amp;amp; (conn.State != ConnectionState.Closed) amp;amp; (conn.State != ConnectionState.Broken); }
    }

    public void CloseConnection()
    {
        // Libération de la connexion si elle existe
        if (IsConnected) { 
            conn.Close();
            conn = null;
        }
    }

    // Implement IDisposable.
    // Do not make this method virtual.
    // A derived class should not be able to override this method.
    public void Dispose()
    {
        // Close connection
    }
}
  

Смотрите это для получения дополнительной информации о реализации IDisposable.