定数クラスを作成するためのベストプラクティスを提案する

定数クラスの宣言については、私のチームメンバーの間で議論があります。定数変数を以下のような別のクラスに移動しています。

public class Constants
{
      public const string StateId = "ST";
      public const string CountryId = "CI";
}

私のチームメンバーの何人かは、オーバーライドオプションを回避するためにクラスを封印済みとして宣言することを提案しました。また、定数クラスのインスタンス作成を回避するために静的としてマークすることを提案する人もいます。
ただし、将来の必要に応じて読み取り専用変数を初期化するのに役立つため、静的コンストラクターでSealedとして使用することを好みます。これに関するアドバイスをお願いします。

あなたの質問が何であるかは完全には明らかではありませんが、値が本当に一定であれば、次の単純なオプションに問題は見られません。

    public static class LocationConstants
    {
        public const string StateId = "ST";
        public const string CountryId = "CI";
    }

staticクラス宣言で使用すると、このクラスの目的に対する意図が示されます。

Marc Gravellは、このStack Overflowの回答で、定数に関する潜在的な問題について説明しています。これらがコードベースの問題であるかどうかを知るのはあなただけですが、値が変更される可能性がある場合は、のpublic static readonly代わりに使用してconstください。そうでない場合、値が変更されると定数を参照するコードを再構築する必要があります。