javaオブジェクトにコンストラクタの変更がある場合のベストプラクティス

Question

新しいAPIの統合ごとに多くのパラメータが追加されているクラスがあります。例えば

は、以前の私は4つのパラメータを持つクラスを持っていた：

Integer a; 
String b; 
Map<String, String> c; 
List<Integer> e. 

ので、コンストラクタは：

public SampleClass(Integer a, 
        String b, 
        Map<String, String> c,  
        List<Integer> e) 
{ 
    this.a = a; 
    this.b = b; 
    this.c = c; 
    this.e = e; 
} 

いくつかのチームは自分のコードでこのコンストラクタを使用して、私のAPIと統合しています。 しばらくしてから、このクラスに新しいパラメータが追加されました。すなわち

Double d; 

だから私は、新しいコンストラクタを追加しました：

public SampleClass(Integer a, 
        String b, 
        Map<String, String> c, 
        List<Integer> e, 
        Double d) 
{ 
    this.a = a; 
    this.b = b; 
    this.c = c; 
    this.e = e; 
    this.d = d; 
} 

を非推奨として、私は、以前のコンストラクタをマーク。以前のコンストラクタを削除しなかったのは、削除した場合、クライアントのコードが破損するためです。

新しいパラメータが追加されるにつれ、5つのパラメータを持つコンストラクタが追加されました。

このようなシナリオが発生しないように、コンストラクタを非推奨/削除する方法に関するベストプラクティスはありますか？

Answer 1

なぜ可変引数のコンストラクタを使用しないでください。この方法で、コンストラクタに多くの引数を渡すことができます。例えば

：

公共平均の2倍（ダブル...数字）{

 double total = 0.0; // initialize total 

     // calculate total using the enhanced for statement 
     for (double d : numbers)    
     total += d;       

     return total/numbers.length; 
    } // end method average 

Answer 2

から変更古いコンストラクタ：

public SampleClass(Integer a, 
        String b, 
        Map<String, String> c,  
        List<Integer> e) 
{ 
    //Zero is passed as a default value, but you can pass anything you want 
    this(a,b,c,e,0); 
} 

これに

public SampleClass(Integer a, 
        String b, 
        Map<String, String> c,  
        List<Integer> e) 
{ 
    this.a = a; 
    this.b = b; 
    this.c = c; 
    this.e = e; 
} 

それはcaフードの下に新しいものがあります。

まだ、あなたは、古いものをどの程度サポートしているかに関する十分な情報を提供していません。まったく必要ない場合はコードから削除する必要があります。この方法は、あなたは新しいコンストラクタを変更し、ワイヤー何を分析するためにAPIのユーザーを強制します。これを行わない場合

は、彼らは古いものを、使用し続けるプログラマは怠惰

:-)をしているので、

Answer 3

Open/closed principleに従うと有益です。最初に書いたクラスは、新しい機能が必要なときに変更すべきではなく、その機能を拡張するために別のクラスを派生させるべきです。