Перегруженный конструктор, вызывающий другой конструктор, но не как первый оператор - PullRequest
Новая
       7

Перегруженный конструктор, вызывающий другой конструктор, но не как первый оператор

16 голосов
/ 02 августа 2010

У меня возникли проблемы с использованием нескольких конструкторов в Java.

Я хочу сделать что-то вроде этого: 

public class MyClass {

 // first constructor
 public MyClass(arg1, arg2, arg3) {
  // do some construction
 }

 // second constructor
 public MyClass(arg1) {
      // do some stuff to calculate arg2 and arg3
      this(arg1, arg2, arg3);
    }
}

, но я не могу, так как второй конструкторне может вызвать другой конструктор, если это не первая строка.

Какое общее решение для такой ситуации?Я не могу вычислить arg2 и arg3 "в строке".Я подумал, может быть, создание вспомогательного метода построения, который будет выполнять фактическую конструкцию, но я не уверен, что это так "красиво" ...

EDIT : Использование вспомогательного метода такжепроблематично, так как некоторые из моих полей являются окончательными, и я не могу установить их, используя вспомогательный метод.

Ответы [ 9 ]

23 голосов
/ 02 августа 2010

Как правило, используйте другой распространенный метод - «помощник по построению», как вы предлагали. 

public class MyClass { 

    // first constructor 
    public MyClass(arg1, arg2, arg3) { 
      init(arg1, arg2, arg3); 
    } 

    // second constructor 
    public MyClass(int arg1) { 
      // do some stuff to calculate arg2 and arg3 
      init(arg1, arg2, arg3); 
    } 

    private init(int arg1, int arg2, int arg3) {
      // do some construction 
    }
}

Альтернативой является подход в заводском стиле, в котором у вас есть MyClassFactory, который дает вам MyClass instance, а у MyClass есть только один конструктор: 

public class MyClass { 

    // constructor 
    public MyClass(arg1, arg2, arg3) { 
      // do some construction 
    } 
} 

public class MyClassFactory { 

    public static MyClass MakeMyClass(arg1, arg2, arg3) { 
      return new MyClass(arg1, arg2, arg3);
    } 

    public static MyClass MakeMyClass(arg1) { 
      // do some stuff to calculate arg2 and arg3 
      return new MyClass(arg1, arg2, arg3);
    } 
}

Я определенно предпочитаю первый вариант.

9 голосов
/ 02 августа 2010

Следующее возможное решение - Заводской метод .Эти статические методы могут быть перегружены, и после вычисления они могут вызвать приватный / защищенный конструктор 

public class MyClass {

    private MyClass( arg1, arg2, arg3 ) {
         // do sth
    }

    public static MyClass getInstance( arg1 ) {
         // calculate arg2,3
        return new MyClass( arg1, arg2, arg3 );
    }

    public static MyClass getInstance( arg1, arg2, arg3 ) {
        return new MyClass( arg1, arg2, arg3 );
    }
}

РЕДАКТИРОВАТЬ: Этот метод также идеально подходит, когда у вас есть окончательные поля

8 голосов
/ 02 августа 2010

Хотя я предпочитаю вариант фабричного метода, на который указывают несколько других ответов, я хотел бы предложить другой вариант: вы можете использовать статические методы для вычисления других ваших параметров: 

public class MyClass {
    public MyClass(int arg1, int arg2, int arg3) {
        // do some construction
    }

    public MyClass(int arg1) {
      //call to this() must be the first one
      this(arg1, calculateArg2(arg1), calculateArg3());
      //you can do other stuff here
    }

    private static int calculateArg2(int arg1) {
      //calc arg2 here
    }

    private static int calculateArg3() {
      //calc arg3 here
    }
}
4 голосов
/ 02 августа 2010

Хелпер и заводские опции очень хороши.

Есть еще один: 

public MyClass(int arg1) {
    this(arg1, calculateArg2(), calculateArg3());
}

private static int calculateArg2() {..}
private static int calculateArg3() {..}
3 голосов
/ 02 августа 2010

Используйте значения маркера для «отсутствующих» 

public class MyClass {
 public MyClass(arg1, arg2, arg3) {
  // do some stuff to calculate arg2 and arg3 if they are the missing values
  // do some construction
 }
 public MyClass(arg1) {
   this(arg1, null, null);
 }
}

Для достижения наилучших результатов используйте «общий» конструктор protected или private.

0 голосов
/ 02 августа 2010

В качестве альтернативы данным ответам, самый простой способ - это рефакторинг вычисления аргумента для конструктора с тремя аргументами; 

public class MyClass {

    // first constructor
    public MyClass(arg1, arg2, arg3) {
        if (null == arg2) {
            // calculate arg2
        }
        if (null == arg3) {
            // calculate arg3
        }
        // do some construction
    }

    // second constructor
    public MyClass(arg1) {
        this(arg1, null, null);
    }
}
0 голосов
/ 02 августа 2010

Другой способ заключается в следующем: 

public class MyClass {

  // first constructor
  public MyClass(arg1, arg2, arg3) {
   // do some construction
   doSomeStuffToArg3Arg3(arg2, arg3)
  }

  // second constructor
  public MyClass(int arg1) {
      this(arg1, arg2, arg3);
  }

  private void doSomeStuffToArg3Arg3(int arg2, int arg3) {
     // do some stuff to calculate arg2 and arg3
  }
}
0 голосов
/ 02 августа 2010

Вы можете создать фабричный метод , который вызывает конструктор: 

public class MyClass { 

    // first constructor 
    public MyClass(arg1, arg2, arg3) { 
    // do some construction

    } 

    // second constructor as factory method
    public static createMyClassAndDoFunkyStuff(int arg1) { 
      // do some stuff to calculate arg2 and arg3 
      return new MyClass(arg1, arg2, arg3); 
    } 

}
0 голосов
/ 02 августа 2010

Вы можете переместить код MyClass(arg1, arg2, arg3) в вспомогательный метод (назовите его Init или как-нибудь еще), а затем вызвать этот метод в обоих конструкторах.

...