1

Имеется старый проект на asp.net (web forms). Появилась необходимость добавить вторую строку подключения (в web.config) и через select тег переключать, когда необходимо запросы выполнить к другой БД. Как это лучше и правильнее сделать подскажите? Где хранить значение выбранной БД? Я попробовал делать через куки:

Из Js отправляю на сервер данные о выбранном подключении и записываю в свойство:

[System.Web.Services.WebMethod]
[System.Web.Script.Services.ScriptMethod(ResponseFormat = System.Web.Script.Services.ResponseFormat.Json)]
public static string SetConnection(string connectionName)
{
    if (!String.IsNullOrEmpty(connectionName))
    {        
        Handlers.ConnectionString = ConfigurationManager.ConnectionStrings[connectionName].ConnectionString;
    }
    return Handlers.ConnectionString;
}

В Handlers.cs:

private static string connectionString = ConfigurationManager.ConnectionStrings[HttpContext.Current.Request.Cookies["nameConn"].Value].ConnectionString;

public static string ConnectionString
{
    get 
    {              
        return connectionString;
    }
    set
    {
        connectionString = value;
    }                              
}

Далее обращаюсь к свойству ConnectionString из других методов и делаю запрос к БД (new SqlConnection(ConnectionString)). Это работает, но через какое то время (например через 2ч) обновлю страницу и данные не приходят с sql сервера, пишет "System.TypeInitializationException" Message : "Инициализатор типа "ProjectSO.Handlers" выдал исключение. В dev режиме не могу воспроизвести проблему..

Вот метод из которого обращаюс к БД:

public static List<object> GetData()
{
    List<object> result = new List<object>();           
    SqlConnection conn = new SqlConnection(ConnectionString);
    SqlCommand cmd = new SqlCommand(@"exec dbo.getData", conn);
        
    SqlDataReader reader = null;
    try
    {
        conn.Open();
        if (conn.State.ToString() == "Open")
        {
            reader = cmd.ExecuteReader();
            if (reader.HasRows)
            {
                while (reader.Read())
                {
                   
                   result.Add(new DynaTreeNode()
                   {                               
                        key = reader.GetValue(1).ToString(),
                        title = reader.GetValue(2).ToString(),
                        children = null
                   });                                                  
                   
                }
            }
        }
     }       
     catch (Exception ex)
     {
         WriteToEventLog(ex, "GetData");
     }
     finally
     {
         if (reader != null) { reader.Close(); }
         conn.Close();
     }

     return result;
    
}
9
  • Упал конструктор класса ProjectSO.Handlers, смотрите трассировку стека.
    – aepot
    8 ноя 2022 в 6:24
  • @aepot, смотрел. Вызывается метод который подгружает данные из БД. В нем создается новое подключение к бд через обращение к свойству new SqlConnection(ConnectionString), строка указанная в трассировке стека указывает на key = reader.GetValue(1).ToString() - но какая тут проблема не понятна.
    – Odyssey
    8 ноя 2022 в 6:37
  • В этой строке в целом может быть только одна проблема, reader.GetValue(1) вернул null, ну или сам reader null. А вообще работа с сетью или другими внешними источниками данных в конструкторе - плохая история. Конструктор должен инициализировать свойства класса, а не запускать активные процессы. Задача конструктора - рабочий экземпляр класса, в котором функционируют все методы корректно, согласно описанному в документации поведению. Все остальное надо выполнять в методах, вызывамых отдельно от конструктора. Создание экземпляров для работы с бд - ок, а вот само подключение - нет.
    – aepot
    8 ноя 2022 в 6:47
  • Так же убедитесь, что IDisposable объекты для работы с БД уничтожаются своевременно.
    – aepot
    8 ноя 2022 в 6:53
  • @aepot, спасибо. А как в моем случае лучше определять строку подключения?
    – Odyssey
    8 ноя 2022 в 7:15

1 ответ 1

2

Всё, понял, падает здесь ConfigurationManager.ConnectionStrings[HttpContext.Current.Request.Cookies["nameConn"].Value].ConnectionString; то есть либо ConnectionStrings не содержит то что возвращает HttpContext.Current.Request.Cookies["nameConn"].Value, либо падает само вычисление этого выражения, унесите в настоящий конструктор и оберните в try-catch.

public string ConnectionString { get; set; }

public Handlers()
{
    try
    {
        ConnectionString = ConfigurationManager.ConnectionStrings[HttpContext.Current.Request.Cookies["nameConn"].Value].ConnectionString;
    }
    catch (Exception ex)
    {
        WriteToEventLog(ex, "ConnectionString");
    }
}

static убрать.

Таким образом можно узнать, что же на самом деле произошло, а убирание static может реально вылечить проблему.

GetData() - тоже убрать статику.

public List<DynaTreeNode> GetData()
{
    List<DynaTreeNode> result = new List<DynaTreeNode>();           
    try
    {
        using (SqlConnection conn = new SqlConnection(ConnectionString))
        using (SqlCommand cmd = new SqlCommand(@"exec dbo.getData", conn))
        {
            conn.Open();

            using (SqlDataReader reader = cmd.ExecuteReader())
            {
                while (reader.Read())
                {
                    result.Add(new DynaTreeNode()
                    {                               
                        key = reader.GetValue(1).ToString(),
                        title = reader.GetValue(2).ToString(),
                        children = null
                    });                                                  
                }
            }
        }
    }       
    catch (Exception ex)
    {
        WriteToEventLog(ex, "GetData");
    }

    return result;
    
}

Создавайте экземпляр var handlers = new Handlers() и через экземпляр работайте. Если потребуется, сделайте синглтон, но никак не статику.

2
  • спасибо, попробую, отмечу в любом случае как ответ. Там кстати все методы в классе Handlers статичные и из других мест вызов идет не посредственно через прямое обращение Handlers.NameMethod. Я так понимаю надо везде статику убрать и сначала создавать, как вы сказали новый экземпляр класса и через эту переменную обращаться к нужному методу
    – Odyssey
    8 ноя 2022 в 8:12
  • @Odyssey верно, для начала такой способ должен сработать. В будущем для экономии ресурсов можете синглтон сделать.
    – aepot
    8 ноя 2022 в 8:15

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.