CA1500 против SA1309 - Кто победит?

Выключаем SA1309. Причина этого довольно слабая.

Наша команда считает, что общепринятая практика частных участников, начинающихся с подчеркивания, намного перевешивает идею о том, что кто-то может использовать другой редактор для кода, чего никогда не бывает в нашем магазине. Что касается обеспечения «немедленной дифференциации», подчеркивание тоже делает это.

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

Вот мое обычное решение:

class SomeClass
{
    int SomeField{get;set;}

    public SomeClass(int someField)
    {
        SomeField = someField;
    }
}

Основываясь на том, что я видел от самих Microsoft, я считаю, что CA1500 побеждает.

Если вы посмотрите на BCL, большая часть кода добавляет к локальным полям знак подчеркивания.

Просто используйте суффикс Field для частных полей, когда есть класс:

 private Int32 counterField;

 public Int32 Counter
 {
     get
     {
          return this.counterField;
     }

     set
     {
           if (this.counterField != value)
           {
                this.counterField = value;
                this.OnPropertyChanged("Counter");
            }
      }

И вы можете удовлетворить оба правила. Украшать переменные любыми символами или венгерскими префиксами - это племя. Каждый может найти правило, которое ему не нравится в StyleCop или FXCop, но стандарт работает только тогда, когда его используют все. Преимущества автоматического скруббера кода намного перевешивают ваш личный «художественный» вклад в язык.

Единственная альтернатива, о которой я могу думать, которая, похоже, удовлетворяет обоим правилам, и которую я действительно видел где-либо, - это что-то вроде следующего. Я сам не соблюдаю эту конвенцию, поскольку она кажется неуклюжей.

public class Class1
{
    // prefix private fields with "m"
    private int mValue1;

    public int Value1
    {
        get { return mValue1; }
        set { mValue1 = value; }
    }

    private string mValue2;

    public string Value2
    {
        get { return mValue2; }
        set { mValue2 = value; }
    }

    // prefix parameters with "p"
    public bool PerformAction(int pValue1, string pValue2)
    {
        if (pValue1 > mValue1)
        {
            mValue2 = pValue2;
            return true;
        }
        else
        {
            return (mValue2 == pValue2);
        }
    }
}

Нет конфликта. Измените имя параметра.

public class SomeClass
{
    private int namedField { get; set; }

    public SomeClass(int differentlyNamedField)
    {
        this.namedField = differentlyNamedField;
    }
}