Проблема с генераторами Java

Question

Пытаемся использовать generics-ify некоторый унаследованный код, я застрял. У меня есть ParentObject, который обертывает ChildObject для предоставления операций типа группы в ChildObject. Наиболее полезно, он позволяет итерации по набору дочерних объектов. Когда я пытаюсь добавить некоторые общие элементы в микс, я не могу понять, как заставить его играть дружелюбно с методом итератора без ошибки «именования» или в приведенном ниже примере «Тип возврата несовместим с ошибкой Iterable.iterator(). Любые предложения? (Bonus вопрос - есть лучший способ, чтобы написать метод избежать thegetChildObjectByIndex(), чтобы избежать типа стирания компиляторов предупреждения, кроме подавления предупреждений?) Спасибо большого заранее за любую помощь

public class ParentObject implements Iterable<ChildObject> { 
    protected List<? super ChildObject> theChildObjects; 

    @SuppressWarnings("unchecked") 
    public <T extends ChildObject> T getChildObjectByIndex(int idx) { 
     return (T)theChildObjects.get(idx); 
    } 

    public Iterator<? super ChildObject> iterator() { 
     return java.util.Collections.unmodifiableCollection(this.theChildObjects).iterator(); 
    } 

} 

Answer 1

Если ParentObject содержит только один подтип ChildObject, вы можете параметризуете ParentObject на этом типе:

public class ParentObject<T extends ChildObject> implements Iterable<T> { 
    protected List<T> theChildObjects; 

    public T getChildObjectByIndex(int idx) { 
     return theChildObjects.get(idx); 
    } 

    public Iterator<T> iterator() { 
     return java.util.Collections.unmodifiableCollection(this.theChildObjects).iterator(); 
    } 
} 

Answer 2

Похоже, в вашем примере ParentObject является своим родом контейнер класса, который на самом деле не имеет никакого отношения к ChildObject. Поддерживает ли ChildObject ParentObject? Если это так, это кажется менее идеальным. Также кажется, что вы должны использовать «extends» в генериках вместо супер, если только я просто не понимаю, что вы пытаетесь сделать. Как насчет следующего? Или это слишком просто для того, что вы хотите сделать?

public class WidgetContainer<T> implements Iterable<T> { 
    protected List<T> theWidgets; 

    public T getWidgetByIndex(int idx) { 
     return theWidgets.get(idx); 
    } 

    public Iterator<T> iterator() { 
     return Collections.unmodifiableCollection(theWidgets).iterator(); 
    } 
} 

WidgetContainer<SomeWidget> myWidgetContainer = new WidgetContainer<SomeWidget>(); 

Также я мог бы кэшировать копию неизменяемо коллекции и использовать его каждый раз, когда вам нужно итератора вместо построения одного на лету.