Действительно ли это - хорошая или плохая практика для вызова методов экземпляра от конструктора Java?

Существует несколько различных способов, которыми я могу инициализировать сложные объекты (с введенными зависимостями и требуемой установкой введенных участников), все кажутся разумными, но имеют различные преимущества и недостатки. Я дам конкретный пример:

final class MyClass {
  private final Dependency dependency;
  @Inject public MyClass(Dependency dependency) {
    this.dependency = dependency;
    dependency.addHandler(new Handler() {
      @Override void handle(int foo) { MyClass.this.doSomething(foo); }
    });
    doSomething(0);
  }
  private void doSomething(int foo) { dependency.doSomethingElse(foo+1); }
}

Как Вы видите, конструктор делает 3 вещи, включая вызов метода экземпляра. Мне сказали, что вызов методов экземпляра от конструктора небезопасен, потому что он обходит проверки компилятора на неинициализированных участников. Т.е. Я, возможно, звонил doSomething(0) перед установкой this.dependency, который скомпилировал бы, но не работал бы. Что лучший способ состоит в том, чтобы осуществить рефакторинг это?

1. Сделать doSomething статичный и передача в зависимости явно? В моем фактическом случае у меня есть три метода экземпляра и три членских поля, что все зависят друг от друга, таким образом, это походит на большое количество дополнительного шаблона для создания всех трех из них статичными.

2. Переместитесь addHandler и doSomething в @Inject public void init() метод. В то время как использование с Guice будет прозрачно, это требует, чтобы любая ручная конструкция убедилась звонить init() или иначе объект не будет полностью функционален, если кто-то забудет. Кроме того, это выставляет больше API, оба из которых походят на плохие идеи.

3. Перенесите вложенный класс для хранения зависимости, чтобы удостовериться, что она ведет себя правильно, не выставляя дополнительный API:

   class DependencyManager {
     private final Dependency dependency;
     public DependecyManager(Dependency dependency) { ... }
     public doSomething(int foo) { ... }
   }
   @Inject public MyClass(Dependency dependency) {
     DependencyManager manager = new DependencyManager(dependency);
     manager.doSomething(0);
   }

   Это вытаскивает методы экземпляра из всех конструкторов, но генерирует дополнительный слой классов, и когда у меня уже были внутренние и анонимные классы (например, тот обработчик), это может стать сбивающим с толку - когда я попробовал это, мне сказали переместиться DependencyManager в отдельный файл, который также неприятен, потому что это - теперь несколько файлов, чтобы сделать единственную вещь.

Таким образом, что предпочтительный путь состоит в том, чтобы иметь дело с этим видом ситуации?