在构造函数或声明中初始化类字段?

我最近一直在用c#和Java编程，我很好奇初始化我的类字段的最佳位置在哪里。

我应该在申报时申报吗?：

public class Dice
{
    private int topFace = 1;
    private Random myRand = new Random();

    public void Roll()
    {
       // ......
    }
}

或者在构造函数中?：

public class Dice
{
    private int topFace;
    private Random myRand;

    public Dice()
    {
        topFace = 1;
        myRand = new Random();
    }

    public void Roll()
    {
        // .....
    }
}

我很好奇你们这些老兵认为最好的做法是什么。我想保持一致，坚持一种方法。

当前回答

有许多不同的情况。

我只需要一个空列表

情况很清楚。我只需要准备我的列表，并防止在有人向列表添加项时抛出异常。

public class CsvFile
{
    private List<CsvRow> lines = new List<CsvRow>();

    public CsvFile()
    {
    }
}

我知道价值观

我确切地知道在默认情况下我想要什么值，或者我需要使用一些其他逻辑。

public class AdminTeam
{
    private List<string> usernames;

    public AdminTeam()
    {
         usernames = new List<string>() {"usernameA", "usernameB"};
    }
}

public class AdminTeam
{
    private List<string> usernames;

    public AdminTeam()
    {
         usernames = GetDefaultUsers(2);
    }
}

可能值的空列表

有时我希望默认情况下是一个空列表，并可以通过另一个构造函数添加值。

public class AdminTeam
{
    private List<string> usernames = new List<string>();

    public AdminTeam()
    {
    }

    public AdminTeam(List<string> admins)
    {
         admins.ForEach(x => usernames.Add(x));
    }
}

2015-08-14 13:26:06

其他回答

如果我告诉你，这要看情况?

我通常初始化所有东西，并以一致的方式进行。是的，它过于明确，但它也更容易维护。

如果我们担心性能，那么我只初始化必须要做的事情，并将其放在性价比最高的地方。

在实时系统中，我甚至怀疑我是否需要变量或常数。

在c++中，我经常在这两个地方都不初始化，而是将它移动到Init()函数中。为什么?好吧，在c++中，如果你初始化的东西可能会在对象构造过程中抛出异常，你就会导致内存泄漏。

2008-08-23 22:09:35

我通常尝试构造函数不做任何事情，只是获取依赖项并初始化相关的实例成员。如果您想对类进行单元测试，这将使您的工作更加轻松。

如果要分配给实例变量的值不受要传递给构造函数的任何参数的影响，则在声明时分配它。

2008-08-24 15:59:11

保持一致很重要，但你要问自己这个问题: “我还有其他的构造函数吗?”