Problème de génériques Java

Question

Tentative de génériques-si un code hérité, je suis coincé. J'ai un parentObject qui encapsule un objet ChildObject pour fournir des opérations de type groupe sur ChildObject. Le plus utilement, il permet l'itération sur la collection d'objets enfants. Lorsque j'essaie d'ajouter des génériques au mixage, je n'arrive pas à le rendre convivial avec la méthode itérateur sans une erreur de "naming clash", ou dans l'exemple ci-dessous, un "Le type de retour est incompatible avec l'erreur Iterable.iterator() ". Des suggestions? (Question bonus - est-il une meilleure façon d'écrire la méthode thegetChildObjectByIndex d'éviter() pour éviter d'avertissement autre que complier d'effacement de type supprimer les avertissements?) Merci beaucoup à l'avance pour toute aide

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

Si ParentObjet ne contient un sous-type de ChildObject, vous pouvez paramètrer ParentObjet sur ce type:

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

on dirait que dans votre exemple ParentObjet est une sorte de classe de conteneur qui n'a vraiment aucun rapport avec ChildObject. Est-ce que ChildObject étend ParentObject? Si oui, cela semble moins qu'idéale. Il semble aussi que vous devriez utiliser "extend" dans les génériques au lieu de super, à moins que je ne comprenne pas ce que vous essayez de faire. Qu'en est-il de ce qui suit? Ou est-ce trop simple pour ce que vous voulez faire?

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

Aussi je pourrais mettre en cache une copie de la collection et non modifiable utiliser chaque fois que vous avez besoin d'un iterator au lieu de construire une à la volée.