私有成员实例化/初始化的最佳实践是什么？

Question

可能的重复：
C# 成员变量初始化； 最佳实践？

，这有什么好处

public class RemotingEngine
{
    uint m_currentValueId;
    object m_lock;

    public RemotingEngine()
    {
        m_currentValueId = 0;
        m_lock = new object();
    }

与此相比

public class RemotingEngine
{
    uint m_currentValueId = 0;
    object m_lock = new object();

：我一直在避免使用第二个，只是因为它感觉“肮脏”。 它显然减少了打字，所以这对我很有吸引力。

Answer 1

它可以改变继承情况。 有关对象初始化顺序的信息，请参阅此链接：
http://www.csharp411.com/c-object-initialization/

1. 派生静态fields
2. 派生静态构造函数
3. 派生实例字段
4. 基类静态字段
5. 基类静态构造函数
6. 基类实例字段
7. 基类实例构造函数
8. 派生实例构造函数

因此，如果这是派生类，则整个基对象会在派生字段初始化和构造函数运行之间初始化。

Answer 2

没有太大区别，我会保留第一个，因为它更具可读性。 通常变量是在类的顶部定义的，我可以去那里检查它们的默认值，而不是寻找构造函数并查看它是否设置它。 就编译器而言，没有什么区别，除非你有多个构造函数。

Answer 3

由于这个原因，我总是使用第一个：初始化变量是构造函数的责任。 不同的构造函数可能会以不同的方式初始化变量。 所以，是的，你应该觉得用第二种方法很脏=)。

Answer 4

我更喜欢第二种，在某些情况下它取决于您的编码标准。 但考虑一下处理顺序：

目标类字段初始化-> 基类字段初始化-> 基类构造函数 -> 目标类构造函数

如果基类或目标类在创建对象时出现异常并且字段已预初始化，则它将在终结时具有值，并可能会导致一些意外问题。

另请参阅 Bill Simser 的博客 C# 中的最佳实践和成员初始化

Answer 5

对于 int，你根本不必这样做，内在类型被初始化为 default() - 对于 int 来说，它是 0。

至于对象，在我看来这是一个品味问题。 我更喜欢前者。 如果有人重写我的构造函数，我希望他们调用 base() 以正确的状态构造它。

Answer 6

您希望避免在构造函数范围之外进行实例化，因为它显示了意图，特别是如果您要重写构造函数，则您有更多的灵活性

Answer 7

就 IL 而言，它们是相同的。

编译器会将这个：

class Foo
{
    int bar = 1;
}

变成这样：

class Foo
{
    int bar;

    public Foo()
    {
        this.bar = 1;
    }
}

即使您自己添加一个构造函数，如下所示：

class Foo
{
    int bar = 1;

    public Foo(int bar)
    {
        this.bar = bar;
    }
}

编译器也会将其变成这样：

class Foo
{
    int bar;

    public Foo(int bar)
    {
        this.bar = 1;
        this.bar = bar;
    }
}

Answer 8

我对此有不同的看法。 我认为你应该抽象属性并在构造函数中设置它们。 使用自动属性基本上可以消除这个问题，因为您无法初始化它们（默认值以外的任何值）。

public class RemotingEngine {
    private uint CurrentValueID { get; set; }
    private object Lock { get; set; }

    public RemotingEngine()
    {
        this.CurrentValueID = 0; // not really necessary...
        this.Lock = new object();
    }
}