在构造函数中初始化私有变量

Question

我一直在查看此代码作为比较器。我的问题是在构造函数中初始化两个私有变量有一个特定的好处（在默认构造函数中完成）。我知道第二个构造函数可用于创建具有提供值的实例。如果我这样做会有什么不同

private String sortBy = COLUMN_LAST_NAME;
private boolean ascending = true; 

如果这是一个真正简单而笼统的问题，我深表歉意。

public class CustomComparator implements Comparator<StudentList>
{   private String sortBy;
private boolean ascending;

public CustomComparator()
{
    sortBy = COLUMN_LAST_NAME;
    ascending = true;
}

public CustomComparator(String sortBy, boolean ascending)
{
    this.sortBy = sortBy;
    this.ascending = ascending;
}

Answer 1

我在各处看到的最佳实践是拥有一个接受所有参数的构造函数，即使这意味着它必须是私有的，然后只需使用this(..,..,...) 从其他构造函数调用它，同时提供适当的值。

这将使您尽可能多地重用代码，并且未来的修复只在一个地方进行 - 无需对代码进行双重维护。

您的示例如下所示：

public class CustomComparator implements Comparator<StudentList> {
    private String sortBy;
    private boolean ascending;

    public CustomComparator()
    {
        this(COLUMN_LAST_NAME, true);
    }

    public CustomComparator(String sortBy, boolean ascending)
    {
        this.sortBy = sortBy;
        this.ascending = ascending;
    }
}

初始化通常被认为是在构造函数中，以便更容易区分静态成员的静态初始化和实例变量的每个实例初始化。没有性能差异。

Answer 2

实际上，如果您关心对象的不变性，这很重要（而且您应该 :-)）。如果你的两个字段都是final，你必须像这样初始化它们：

public class CustomComparator implements Comparator<StudentList> {
    private final String sortBy;
    private final boolean ascending;

    public CustomComparator() {
        sortBy = COLUMN_LAST_NAME;
        ascending = true;
    }

    public CustomComparator(String sortBy, boolean ascending) {
        this.sortBy = sortBy;
        this.ascending = ascending;
    }
}

甚至更好：

public class CustomComparator implements Comparator<StudentList> {
    private final String sortBy;
    private final boolean ascending;

    public CustomComparator() {
        this(COLUMN_LAST_NAME, true);
    }

    public CustomComparator(String sortBy, boolean ascending) {
        this.sortBy = sortBy;
        this.ascending = ascending;
    }
}

Answer 3

这完全取决于您的需求，两个构造函数都是不错的选择，显然如果您想初始化字段是最好的选择，但是如果您像 Hibernate 上的实体或 Spring 框架上的 bean 一样使用此类，请考虑不要写空的构造函数，什么都不会运行良好......

如果你在 DTO 模式中思考，声明像 @Tomasz Nurkiewicz 这样的不可变字段，那么参数化构造函数是唯一的选择......

但就像我之前说的，这取决于要求...

Answer 4

我想不出有什么区别，我更喜欢在变量声明中进行初始化。