ジェネリックでこのインタフェースをより簡単に定義/実装する方法は？

Question

私は遺伝的アルゴリズムに取り組んでおり、私はGAを再利用できるようにできるだけ抽象的にしたいと思っています。 Population Interfaceを定義して実装してもうまく動作しますが、それを行う最良の方法ではないと確信しています。私はJava Genericsに大きな経験はありません。定義し、人口のインタフェースを実装する簡単な方法があります（例えば、多分キャスト変換を避ける）（getChromosomesで新しいリストを避けるため？）

public interface Population 
{ 
    void addChromosomes(List<? extends Chromosome> chromosomes); 

    List<Chromosome> getChromosomes(); 

    // More code here ... 
} 

public class TSPPopulation implements Population 
{ 
    private List<TSPChromosome> chromosomes; 

    @Override 
    public void addChromosomes(List<? extends Chromosome> chromosomes) { 
     for (Chromosome chromosome : chromosomes) { 
      this.chromosomes.add((TSPChromosome) chromosome); 
     } 
    } 

    @Override 
    public List<Chromosome> getChromosomes() { 
     List<Chromosome> newList = new ArrayList<Chromosome>(); 
     for (TSPChromosome chromosome : chromosomes) { 
      newList.add(chromosome); 
     } 
     return newList; 
    } 
} 

Answer 1

をお使いのインタフェースで有界ワイルドカードを使用します。

public interface Population<T extends Chromosome>{ 
    void addChromosomes(List<T> chromosomes); 

    List<T> getChromosomes(); 
} 

public class TSPPopulation implements Population<TSPChromosome> 
{ 
    private List<TSPChromosome> chromosomes; 

    @Override 
    public void addChromosomes(List<TSPChromosome> chromosomes) { 
... 
    } 

    @Override 
    public List<TSPChromosome> getChromosomes() { 
...  
    } 

} 

Answer 2

最も簡単な解決策は、リスト（そのリストにChromosomsのリストを追加するaddAll(...)を使用）拡張されています

class Population<T extends Chromosome> extends ArrayList<T> { 
} 

をしかし、あなたは同じ目をしたい場合私はPopulationを一般的なリストクラスにします。このようにして、add...とget...メソッドの両方をジェネリックベースクラスで処理できます。あなたが他の機能を無効化したいなら、あなただけのPopulation（class TSPPopulation extends Population<TSPChromosome>を拡張

使用法：。

public static void main(String... args) { 
    Population<TSPChromosome> tspPopulation = new Population<TSPChromosome>(); 
    ... 
} 

実装：インスタンスの

class Population<T extends Chromosome> { 

    private List<T> chromosomes = new ArrayList<T>(); 

    public void addChromosomes(List<T> chromosomes) { 
     this.chromosomes.addAll(chromosomes); 
    } 

    public List<T> getChromosomes() { 
     return new ArrayList<T>(this.chromosomes); 
    } 
} 

Answer 3

はい、：

public interface Population<T extends Chromosome> 
{ 
    void addChromosomes(List<T> chromosomes); 

    List<T> getChromosomes(); 

    // More code here ... 
} 

public class TSPPopulation implements Population<TSPChromosome> 
{ 
    private List<TSPChromosome> chromosomes; 

    @Override 
    public void addChromosomes(List<TSPChromosome> chromosomes) { 
     this.chromosomes.addAll(chromosomes); 
    } 

    @Override 
    public List<TSPChromosome> getChromosomes() { 
     return new ArrayList<TSPChromosome>(chromosomes); 
    } 
} 

Answer 4

あなたが集団そのものを作成した場合はもっと安全です：

public interface Population<T extends Chromosome> { 

    void addChromosomes(List<T> chromosomes); 

    List<T> getChromosomes(); 

} 

public class TspPopulation implements Population<TspChromosome>{ 

    @Override 
    public void addChromosomes(List<TspChromosome> chromosomes){ 
     // 
    } 

    @Override 
    public List<TspChromosome> getChromosomes(){ 
     // 
    } 

} 

これで、クライアントコードにキャスティングする必要はありません。

Answer 5

私はガスを知っている、と私はあなたのPopulation実装が実際にあなたが入れChromosomeの種類を知っておく必要があるかどうかを疑問でしょう。あなたが本当にChromosomeサブクラスに応じて、異なるPopulation実装を持っていますか？または、あなたが本当に望むのは、同じサブクラスのChromosomeがPopulationにあることを確認することです。この最後の場合には他の人が示唆したように、あなたはPopulationインタフェースを定義することができ、および一般的な実装を作る（または完全インタフェースをスキップ）：

public class PopulationImpl implements Population<T extends Chromosome> { 
    private List<T> chromosomes; 

    @Override 
    public void addChromosomes(List<T> chromosomes) { 
     this.chromosomes.addAll(chromosomes); 
    } 

    @Override 
    public List<T> getChromosomes() { 
     return new ArrayList<T>(chromosomes); 
    } 
} 

は、あまりにも多くのジェネリックを入れないように注意してください、またはあなたがなってしまいますジェネリック地獄、またはジェネリック薬を役に立つよりも厄介なものにするたくさんのキャストで。