关于最佳实践的 C# 神话？

Question

我的同事一直告诉我 cmets 中列出的东西。

我很困惑。 有人可以为我揭开这些东西的神秘面纱吗？

class Bar
{
    private int _a;

    public int A
    {
        get { return _a; }
        set { _a = value; }
    }

    private Foo _objfoo;

    public Foo OFoo
    {
        get { return _objfoo; }
        set { _objfoo = value; }
    }

    public Bar(int a, Foo foo)
    {
        // this is a bad idea
        A = a;
        OFoo = foo;
    }

    // MYTHS
    private void Method()
    {
        this.A    //1 -
        this._a   //2 - use this when inside the class e.g. if(this._a == 2)
        A         //3 - use this outside the class e.g. barObj.A
        _a        //4 - 
                  // Not using this.xxx creates threading issues.
    }
}
class Foo
{
    // implementation
}

Answer 1

如果没有名称冲突，this. 是多余的。只有在需要引用当前对象或有与字段同名的参数时才需要它。

线程问题与此无关。混淆可能来自这样一个事实，即大多数静态成员都是线程安全的，而静态成员不能（！）使用this. 调用，因为它们没有绑定到实例。

Answer 2

"不使用 this.xxx 创建线程 问题”

是一个完整的神话。只需让您的同事检查生成的 IL，并让他解释为什么无论您是否添加 this，它们都是相同的。

"在课堂上使用这个，例如 if(this._a == 2)"

取决于您想要实现的目标。您的同事似乎在说总是引用私有字段，这在我看来并不明智。通常你想访问 public 属性，即使是在一个类中，因为 getter 可能会修改值（例如，当列表为 null 时，List 类型的属性可能会返回一个新的 List 实例以避免访问属性时出现空引用异常）。

Answer 3

我个人的“最佳实践”是始终使用它。是的，它是多余的，但是当您考虑多线程应用程序时，它是从第一次查看实例的状态被更改或检索的位置来识别的好方法。

Answer 4

询问您的同事为什么他认为这些建议是最佳实践可能会有所帮助？人们经常引用他们在某个地方找到的最佳实践“规则”，而没有真正了解这些实践背后的原因。

正如 Lucero 所说，“this”不是必需的，除非 () 存在名称冲突。但是，有些人喜欢在没有严格要求时包含“this”，因为他们认为它增强了可读性/更清楚地显示了程序员的意图。在我看来，这是个人喜好问题，而不是其他问题。

至于“Bar”方法中的“坏主意”：您的同事可能会考虑这种不好的做法，原因如下：如果将“A”的 setter 方法更改为产生一些副作用，那么 A=a; 也会产生这种副作用，而 _a = a; 只会设置私有变量。在我看来，最佳实践是意识到差异而不是偏爱一个而不是另一个。

最后，“线程问题”是无稽之谈 - AFAIK “this”与线程无关。

Answer 5

数字 2 是一个神话，很容易通过提及自动属性来揭穿。 Automatic properties 允许您定义一个没有编译器自动生成的支持字段的属性。所以问问你的同事他对自动属性的看法。