Java 6 Collections.checkedList utilisation appropriée

Question

Je crée plusieurs fonctions dans lesquelles je veux retourner l'interface au lieu de l'implémentation, I.E. Liste au lieu de ArrayList. Ma méthode signature et une brève mise en œuvre suit:

public List<MyAwesomeComposedObject> convert(Collection<MyAwesomeObject> awesomeObjects>) 
{ 
    List<MyAwesomeComposedObject> composedObjects = new ArrayList<MyAwesomeComposedObject>(); 
    for(MyAwesomeObject awesomeObject : awesomeObjects) 
    { 
     MyAwesomeComposedObject composedObject = new MyAwesomeComposedObject(awesomeObject); 
     composedObjects.add(composedObject); 
    } 

    List<MyAwesomeComposedObject> composedObjectList = Collections.checkedList<composedObjects, MyAwesomeComposedObject.class); 
    return composedObjectList; 
} 

Ma question est, est-ce un anti-modèle de quelque sorte? Je veux garantir que l'invocateur de cette méthode obtient l'interface au lieu d'une implémentation. Je ne crois pas non plus qu'il s'agisse d'un cas de sur-ingénierie. Si ce n'est pas la bonne façon de retourner une interface, dans ce scénario, je suis ouvert à l'implémentation correcte.

Attaché est un petit programme qui donne lieu à une exception:

public static void main(String[] args) 
{ 
    Vector v = (Vector) c(); 
} 

static List<Object> c() 
{ 
    List<Object> l = new ArrayList<Object>(); 
    l.add(new Object()); 
    List<Object> o = Collections.checkedList(l, Object.class); 
    return o; 
} 

Le javadoc est ici: checked list

Answer 1

Pour votre question révisée: Il n'y a aucun moyen d'empêcher l'appelant de tenter de jeter à tout ce qu'ils veulent. Si c'est un casting inapproprié, ils auront l'exception. C'est la raison même pour laquelle la conversion d'une interface en une classe concrète est fortement déconseillée.

Si vous êtes vraiment inquiet à ce sujet, pensez à renvoyer un ArrayList au lieu d'un List. Cela devrait décourager le lancer puisqu'ils sont en béton. S'il vous plaît noter que je ne cautionne pas cela, c'est juste une option.

Je veux garantir que l'invocateur de cette méthode devient l'interface au lieu d'une mise en œuvre

Ce n'est pas valide. Vous renvoyez un List où le type déclaré des éléments est une interface, mais chaque élément doit être une instanciation. Tout ce que fait une collection vérifiée empêche l'ajout d'éléments du type incorrect. Rien n'empêche l'utilisateur de revenir au type d'implémentation.

Si vous essayez de faire en sorte que l'utilisateur obtient List au lieu de ArrayList (je suppose ici parce que je ne vois pas d'interface pour vous classe impressionnante), ce nouveau est défectueux parce que l'utilisateur pourrait encore jeter le List à un ArrayList bien que ce serait une mauvaise idée car il risque un ClassCastException.

Answer 2

La liste retournée est un Collections.CheckedList pas un vecteur. Vous ne pouvez pas faire référence à un type que l'objet n'est pas.

Cependant ce que vous pouvez faire est

public static void main(String[] args) { 
    Vector<Object> v = new Vector<Object>(c()); 
} 

---

composedObjects est déjà une liste, vous pouvez le retourner.

public List<MyAwesomeComposedObject> convert(Collection<MyAwesomeObject> awesomeObjects>) { 
    List<MyAwesomeComposedObject> composedObjects = new ArrayList<MyAwesomeComposedObject>(); 
    for(MyAwesomeObject awesomeObject : awesomeObjects)  
     composedObjects.add(new MyAwesomeComposedObject(awesomeObject)); 
    return composedObjects; 
} 

Answer 3

Non, je recommande de garder le code aussi simple que possible. Lisez les Javadoc pour une discussion quand utiliser Collections.checkedList

http://download.oracle.com/javase/7/docs/api/java/util/Collections.html#checkedCollection%28java.util.Collection,%20java.lang.Class%29