Блог

В общедоступном классе у меня есть частный статический словарь. Поскольку словарь статичен, означает ли это, что он является общим для всех других экземпляров того же объекта (см. Пример ниже).

 public class Tax
{
    private static Dictionary<string, double> TaxBrakets = new Dictionary<string, double>(StringComparer.OrdinalIgnoreCase)
    {
        { "Individual",   0.18 },
        { "Business",     0.20 },
        { "Other",        0.22 },
    };

    public string Type { get; set; }
    public double ComputeTax(string type, double d)
    {
        return d * TaxBrakets[this.Type];
    }
}
  

Допустимо ли использовать словарь таким образом (как статическую переменную)?

Ваша статическая переменная TaxBrakets не связана с экземпляром. this.TaxBrakets не будет компилироваться. Все вхождения TaxBrakets будут ссылаться на один и тот же словарь. В общем, использование статических словарей вполне приемлемо. Хотя это конкретное использование кажется немного забавным, но мне нужно было бы увидеть больше кода, чтобы предложить какие-либо изменения.

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

Является ли это правильным решением вашей бухгалтерской проблемы, является спорным, IANAA, но я понимаю, что налоговые скобки (примечание: правильное написание «скобок» имеет c ) приравнивают уровни дохода к предельным налоговым ставкам, знание типа объекта само по себе недостаточно для определения налоговой ставки.

Да, этот статический элемент будет общим для всех экземпляров. (Visual Basic фактически делает это явным, используя Shared ключевое слово для статических членов, а не static .)

Вполне допустимо использовать словарь таким образом, с одной оговоркой: если ваш код многопоточный, вы должны убедиться, что доступ к статическому элементу потокобезопасен. (Конечно, это относится ко всем элементам, к которым можно обращаться одновременно, а не только к статике.)

Не имеет прямого отношения к вопросу, но вам, вероятно, следует использовать decimal , а не double для всего, что связано с денежными значениями или финансовыми расчетами.

Да, она будет общей, и да это должно быть нормально, это будет потокобезопасно, а также без обновления. Я бы просто добавил readonly к объявлению:

 private static readonly Dictionary<string, double> TaxBrakets = new Dictionary<string, double>(StringComparer.OrdinalIgnoreCase)
    {
        { "Individual",   0.18 },
        { "Business",     0.20 },
        { "Other",        0.22 },
    };
  

Статическое свойство на самом деле не является общим для экземпляров как таковых, но вы используете его правильно.

Да, static указывает, что это единственная переменная в пределах класса Tax.

Вам нужно будет получить к ней доступ следующим образом

 return d * Tax.TaxBrackets[this.Type]
  

http://msdn.microsoft.com/en-us/library/98f28cdx.aspx

ДА. На самом деле, если бы она не была скрыта private , то она была бы доступна даже без какого-либо экземпляра.

Например:

 public class X
{
    public static String y = "abc";
}

// other code, even without instances of X alive:
Console.WriteLine(X.y);