c# - 静的変数の値を変更するのは悪い習慣ですか?

Question

おそらくHTTPプロトコルに応じて変更する必要がある静的文字列変数があります。

静的文字列変数を変更するのは悪い習慣ですか>

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

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

ありがとう

score 12 · Accepted Answer

いいえ。もちろん、静的文字列変数の値を変更できます。なぜそれが悪い習慣だと思いますか？

score 2 · Accepted Answer

つまり、静的変数の変更は問題ではありません。それは変数です。変化する可能性があります。では、なぜ変更する（つまり、変更する）のは悪い習慣なのでしょうか？はい、そうするべきではない、または注意しなければならない状況がありますが、一般的にはそうではありません。

ここでの大きな問題は、アプリケーションの根底にあるアプリケーション設定を読み取ることです。それは保守性とテスト性を殺します。それは恐ろしく悪い習慣であり、すぐにやめることをお勧めします。

score 1 · Accepted Answer

この場合、1 回限りのセットアップのように見えますが、ASP.NET を含むマルチスレッド環境での競合状態に注意する必要があります。

score 0 · Accepted Answer

プログラム的に言えば、POV言語から変更してもかまいません。

それ以外の場合は、変数のロジック、つまりビジネスロジック全体での意味に依存します。

score 0 · Accepted Answer

静的変数を変更すること自体は悪いことではありません。

ただし、このクラスの複数のインスタンスが同じ static を変更することに関して、予防策が取られていないようです。これは潜在的な同時実行の問題です。

さらに、おそらく静的に 2 つの設定を設定し、状態に基づいて適切な方を選択する必要がありますが、静的が初期化されたらそのままにしておく必要があります。

score -1 · Accepted Answer

静的変数の代わりに静的プロパティを使用できます。


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

静的プロパティ

score -2 · Accepted Answer

いいえ、一般的に悪い習慣ではありません。あなたの特定のケースでは、それはひどい考えです。

c# - 静的変数の値を変更するのは悪い習慣ですか?

8 に答える 8

Related

Reference