Да, это нормальная практика, если вы внимательны. У кода вызываемого из конструктора, есть особенности.
Метод может быть вызван для не полностью сконструированого объекта:
public class MethodCallInConstructor1 {
public static void main(String... args) {
new Class().method();
}
public static class Class {
private final int value1;
private final int value2;
public Class() {
System.out.println("Class.Class");
value1 = 10;
method();
value2 = 100;
}
public void method() {
System.out.println("Class.method: value1 " + value1 + ", value2 " + value2);
}
}
}
$ javac MethodCallInConstructor1.java && java MethodCallInConstructor1
Class.Class
Class.method: value1 10, value2 0
Class.method: value1 10, value2 100
Поля в классе Class
обозначены final
. Нормально предполагать что эти значения неизменны во время работы программы. И это так после того как конструктор завершился. А во время конструирования значение final
-поля может измениться и это видно всему миру.
Если вы вызываете метод из конструктора учтите что объект может быть в необычном состоянии.
Метод производного класса вызывается до конструктора производного класса:
public class MethodCallInConstructor2 {
public static void main(String... args) {
new Derived();
}
public static class Base {
public Base() {
System.out.println("Base.Base");
method();
}
public void method() {
System.out.println("Base.method");
}
}
public static class Derived extends Base {
public Derived() {
System.out.println("Derived.Derived");
}
@Override
public void method() {
System.out.println("Derived.method");
}
}
}
$ javac MethodCallInConstructor2.java && java MethodCallInConstructor2
Base.Base
Derived.method
Derived.Derived
В Java конструирование объекта происходит в две стадии: объект распределяется в памяти и заполняется нулями, затем вызывается конструктор.
Если вы вызываете метод из конструктора базового класса и метод переопределен в наследнике, учтите что объект будет полностью или частично не инициализирован.
Если при конструировании метод бросит исключение, оператор new Class(...)
не завершится нормально и объект, например, не будет никуда присвоен. Если вы бросаете исключение из вызваного метода, то вся сделанная работа по созданию объекта пропадает.
Ни один из этих пунктов не запрещает вам вызывать методы из конструкторов. Будьте внимательны, пишите длинные комментарии и всё будет хорошо.