Jak obudować zmienne „globalne” w języku C #? /najlepsze praktyki

W języku C # jaka jest najlepsza praktyka w zakresie enkapsulacji zmiennych, których muszę używać na wiele metod? Czy wystarczy zadeklarować je na szczycie mojej klasy powyżej dwóch metod?

Również jeśli używam ustawień aplikacji z mojego pliku konfiguracyjnego, czy powinienem użyć gettera? lubię to...

private string mySetting{ get { return WebConfigurationManager.AppSettings["mySetting"]; } }

Jaka jest najlepsza praktyka?

To nie tylko OK. Według książki Clean Code jest to w rzeczywistości bardzo dobra praktyka, a wujek Bob naprawdę ją zachęca. Zmienna stosowana w wielu metodach może wykazywać wysoki stopień spójności między metodami. Co więcej, wysoki stopień zmiennych obiektowych może również sugerować, że wspomnianą klasę należy podzielić na dwie części, więc zadeklarowanie ich jako zmiennych obiektowych może pomóc w znalezieniu kandydatów na ukryte klasy.

Zmienne na poziomie obiektu nie są zmiennymi globalnymi, więc nie bój się ich używać, jeśli powinny być udostępniane różnymi metodami.

Ciągłe kapsułkowanie ustawień to świetny pomysł.

To, co robię, to tworzenie klasy ustawień albo jednej statycznej globalnej jednej, albo wielu klas instancji, którymi będę zarządzał z wstrzykiwaniem zależności. Następnie ładuję wszystkie ustawienia z konfiguracji do tej klasy podczas uruchamiania.

Napisałem też małą bibliotekę, która wykorzystuje refleksję, aby uczynić to jeszcze łatwiejszym.

Gdy moje ustawienia znajdą się w moim pliku konfiguracyjnym

<?xml version="1.0" encoding="utf-8" ?>
<configuration>   
    <appSettings>
        <add key="Domain" value="example.com" />
        <add key="PagingSize" value="30" />
        <add key="Invalid.C#.Identifier" value="test" />
    </appSettings>
</configuration>

Tworzę klasę statyczną lub instancję w zależności od moich potrzeb. W przypadku prostych aplikacji z kilkoma ustawieniami jedna klasa statyczna jest w porządku.

private static class Settings
{
    public string Domain { get; set; }

    public int PagingSize { get; set; }

    [Named("Invalid.C#.Identifier")]
    public string ICID { get; set; }

}

Następnie za pomocą mojej bibliotece połączenia albo Inflate.Staticczy Inflate.Instancei fajne jest to mogę użyć dowolnego źródła wartości klucza.

using Fire.Configuration;

Inflate.Static( typeof(Settings), x => ConfigurationManager.AppSettings[x] );

Cały kod do tego jest w GitHub na https://github.com/Enexure/Enexure.Fire.Configuration

Istnieje nawet pakiet nuget:

PM> Zainstaluj pakiet Enexure.Fire.Configuration

Kod referencyjny:

using System;
using System.Linq;
using System.Reflection;
using Fire.Extensions;

namespace Fire.Configuration
{
    public static class Inflate
    {
        public static void Static( Type type, Func<string, string> dictionary )
        {
            Fill( null, type, dictionary );
        }

        public static void Instance( object instance, Func<string, string> dictionary )
        {
            Fill( instance, instance.GetType(), dictionary );
        }


        private static void Fill( object instance, Type type, Func<string, string> dictionary ) 
        {

            PropertyInfo[] properties;
            if (instance == null) {

                // Static
                properties = type.GetProperties( BindingFlags.Public | BindingFlags.Static | BindingFlags.DeclaredOnly );
            } else {

                // Instance
                properties = type.GetProperties( BindingFlags.Public | BindingFlags.Instance | BindingFlags.DeclaredOnly );
            }

            // Get app settings and convert
            foreach (PropertyInfo property in properties) {
                var attributes = property.GetCustomAttributes( true );
                if (!attributes.Any( x => x is Ignore )) {

                    var named = attributes.FirstOrDefault( x => x is Named ) as Named;

                    var value = dictionary((named != null)? named.Name : property.Name);

                    object result;
                    if (ExtendConversion.ConvertTo(value, property.PropertyType, out result)) {
                        property.SetValue( instance, result, null );
                    }
                }
            }
        }
    }
}