Это плохая практика, чтобы изменить значение статической переменной?

Question

У меня есть статическая строковая переменная, которую мне нужно изменить, возможно, в зависимости от протокола HTTP.

Это плохая практика, чтобы изменить статическую строковую переменную>

static string QuoteWebServiceUrl = CommonFunctions.ReadAppSetting("QuoteWebServiceUrl");

if(url == "https")
{
  QuoteWebServiceUrl = CommonFunctions.ReadAppSetting("QuoteWebServiceUrlSecure");
}
else
{
  QuoteWebServiceUrl = CommonFunctions.ReadAppSetting("QuoteWebServiceUrl");
}

Спасибо

Answer 1

Нет.Конечно, вы можете изменить значение статической строковой переменной.Почему вы думаете, что это плохая практика?

Answer 2

В этом случае это выглядит как разовая установка, но вы должны помнить о состоянии гонки в многопоточной среде, включая ASP.NET.

Answer 3

Я имею в виду, изменение статической переменной не является проблемой.Это переменная.Это может варьироваться.Итак, почему изменение (то есть изменение его) было бы плохой практикой?Да, есть ситуации, когда вы не должны или должны быть осторожны, если это так, но в целом это не так.

Большой проблемой здесь является чтение настроек приложения в глубине души вашего приложения.Это убивает ремонтопригодность и тестируемость.Это ужасно плохая практика, и я призываю вас немедленно остановиться.

Answer 4

Вы можете использовать статическое свойство вместо статической переменной.

private static string QuoteWebServiceUrl 
{ 
    get 
    { 
        if(url == "https") 
        {   
            return CommonFunctions.ReadAppSetting("QuoteWebServiceUrlSecure"); 
        } 
        else 
        {   
            return CommonFunctions.ReadAppSetting("QuoteWebServiceUrl"); 
        } 
    } 
}

Статическое свойство

Answer 5

Само по себе изменение статической переменной не является плохой вещью.

Однако я не вижу мер предосторожности, касающихся нескольких экземпляров этого класса, изменяющих одну и ту же статику.Это потенциальная проблема параллелизма.

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

Answer 6

Нет, это не плохая практика в общем . В вашем конкретном случае это ужасная идея.

Answer 7

Программно говоря, от языка POV все в порядке, чтобы изменить его.

В противном случае зависит от логики переменной, значения, которое она имеет в общей бизнес-логике.