スーパークラスオブジェクトの配列。それらをサブクラスとして管理する方法は？

Question

は、これらのクラスを持つ：

public abstract class Furniture 

public class Chair : Furniture 

public class Table : Furniture 

public class Kitchen 
{ 
ArrayList <Furniture> furnitures; 
//other code 
public void function() 
{ 
    Furniture furniture = furnitures.get(); 
    doSomethingInKitchen(furniture); 
} 


private void doSomethingInKitchen (Chair c); 
private void doSomethingInKitchen (Table t); 

} 

私は私がサブクラス1（椅子やテーブル）としてスーパークラスの家具のオブジェクトを操作するには保証のベストプラクティスを探しています。

私は単純なキャストで試してみましたが、私が関数を呼び出すと、テーブルやチェアのものではなく、家具オブジェクトで動作します。

私が試したことのようなものです：問題はcurrentFurnitureが

Furniture currentFurniture; 

として宣言され、それが議長として認識されないことがあるかどうかは知りません

for each Furniture in Array List 
if(furniture.get() istance of Table) 
{ 
currentFurniture = (Table) furniture.get(); 
} 

else if (furniture.get() istanceof Chair) 
{ 
currentFurniture = (Chair) furniture.get(); 
} 
doSomethingInKitchen(currentFurniture) 

キャスティングにもかかわらず、またはソリューションのデザイン自体が間違っている場合は、テーブル。

Answer 1

キャストは、共通変数に再割り当てするとすぐに失われます。

理想的には、キャストは避けて、サブクラス自体に異なるロジックを実装します。たとえば：

abstract class Furniture { 
    abstract void doSomethingInKitchen(); 
} 

class Table extends Furniture { 
    @Override 
    void doSomethingInKitchen() { 
     // Table-specific logic 
    } 
} 

class Chair extends Furniture { 
    @Override 
    void doSomethingInKitchen() { 
     // Chair-specific logic 
    } 
} 

は今、あなたはFurnitureクラスを継承しているので、あなただけの

for (Furniture furniture : furnitures) { 
    furniture.doSomethingInKitchen(); 
} 

Answer 2

を行い、あなたのKitchenにすることができますそれぞれchairとTable

private void doSomethingInKitchen (Chair c); 
private void doSomethingInKitchen (Table t); 

ため2 methodsを実装する必要がtheresのこのような単一の方法を持つ

であり、鋳造のRaidをforloopに取得して、このメソッドにキャストを実行させることができます。

private void doSomethingInKitchen (Furniture f){ 

    if(f instanceof Table){ 
    //code for the table 
    }else{ 
    //code for the chair 
    } 

}