Matrice di oggetti della superclasse. Come gestirli come sottoclassi?

Question

Avendo queste classi:

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); 

} 

Sto cercando la migliore pratica che mi assicura di manipolare l'oggetto superclasse Mobili come una sottoclasse uno (sedia o un tavolo).

Ho provato con un cast semplice, ma quando chiamo la funzione funziona con l'oggetto mobili, non con il tavolo o la sedia uno.

Quello che ho cercato è qualcosa di simile:

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) 

Non so se il problema è che currentFurniture è dichiarato come

Furniture currentFurniture; 

E così non sarà riconosciuto come presidente o Tabella nonostante il casting o se il design stesso della soluzione è sbagliato.

Answer 1

Il cast viene perso non appena viene riassegnato alla variabile comune. È necessario gestire ogni tipo separatamente:

for (Furniture furniture : furnitures) { 
    if (furniture instanceof Table) { 
     doSomethingInKitchen((Table)furniture); 
    } else if (furniture instanceof Chair) { 
     doSomethingInKitchen((Chair)furniture); 
    } 
} 

Idealmente, però, si dovrebbe evitare di proiettare tutto e implementare la logica diversa della sottoclasse stesso. Per esempio:

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 
    } 
} 

Ora nel tuo Kitchen basta fare

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

Answer 2

dal momento che si eredita Furniture Theres classe senza bisogno di implementare 2 methods per ogni chair e Table

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

è possibile avere un singolo metodo come questo

private void doSomethingInKitchen (Furniture f); 

ed è possibile ottenereRid del casting in forloop e lasciare che il metodo esegua il casting.

private void doSomethingInKitchen (Furniture f){ 

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

}