Zastępowanie lista i praktyka Java

Question

mam trzy klasy (liczba będzie prawdopodobnie rosnąć w przyszłości):

public inteface Base{ } 

public class Select implements Base{ } 

public class Ast implements Base{ } 

public class Gt implements Base{ } 

również potrzebne w klasie List

BaseList extends ArrayList<Base>{ 
    public boolean add(Base b){ 
     throw new UnsupportedOperationException("You should use add%ConcereteBaseType% method instead"); 
    } 

    public boolean add(Select s){ } 

    public boolean add(Ast a){ } 

    public boolean add(Gt g){ } 
} 

Powodem, dlaczego to zrobiłem był to fakt, że nie chcę, aby ktokolwiek dodawał elementy za pomocą wskaźnika do Base. W mojej konkretnej sytuacji byłoby to niebezpieczne.

Jednak wadą jest to, że zostanie wykryte tylko w czasie wykonywania.

Potrzebuję również do iteracji na liście.

Czy to dobra praktyka, aby robić takie rzeczy?

Answer 1

Korzystanie z interface jest w porządku. (Włączanie Base do klasy abstrakcyjnej nie kupi nic tutaj).

Należy sprzyjać kompozycję nad dziedziczenie tutaj choć i wdrożyć Iterable<Base> aby móc używać go w wzmocnione pętle itp

class BaseList implements Iterable<Base> { 
    private List<Base> list = new ArrayList<>(); 

    public boolean add(Select s) { 
     return list.add(s); 
    } 

    public boolean add(Ast a) { 
     return list.add(a); 
    } 

    public boolean add(Gt gt) { 
     return list.add(gt); 
    } 

    @Override 
    public Iterator<Base> iterator() { 
     return Collections.unmodifiableList(list).iterator(); 
    } 
} 

Następnie można powtórzyć listę w następujący sposób:

for (Base b : yourBaseList) { 
    ... 
} 

Answer 2

Można utworzyć klasę Base i abstract, ponieważ nigdy nie można utworzyć instancji, Twoja lista będzie bezpieczna.

Mój sposób: Naprawdę nie rozumiem, dlaczego chciałbyś rzucić wyjątek dla add(Base b), gdy baza jest interface, która sama nigdy nie może być obiektem.

W przeciwnym razie należy użyć schematu typu "przenosić/zawijać", oznacza to: Nie należy rozszerzać ArrayList, ale należy utworzyć klasę , która ma i ArrayList.

class BaseList { 
    private List<Base> list = new ArrayList<>(); 

    public boolean add(Select s) { 
     return list.add(s); 
    } 
    //etc 
}