Comment puis-je corriger "l'expression de La Liste type de besoins unchecked conversion...'?

Dans le fragment de code Java:

SyndFeedInput fr = new SyndFeedInput();
SyndFeed sf = fr.build(new XmlReader(myInputStream));
List<SyndEntry> entries = sf.getEntries();

la dernière ligne génère l'avertissement

", L'expression " de type List besoins unchecked conversion pour se conformer à List<SyndEntry>"

Ce qui est un moyen approprié pour résoudre ce problème?

InformationsquelleAutor user46277 | 2008-12-15
  1. 94

    Depuis getEntries retourne un raw List, il pourrait contenir quoi que ce soit.

    L'avertissement-gratuit approche est de créer un nouveau List<SyndEntry>, alors jetez chaque élément de la sf.getEntries() résultat à SyndEntry avant de l'ajouter à votre nouvelle liste. Collections.checkedList ne pas faire cette vérification pour vous—même s'il aurait été possible de le mettre en œuvre pour le faire.

    En faisant votre propre cast avant, vous êtes "en respectant les termes de la garantie" de Java génériques: si un ClassCastException est soulevé, il va être associé avec un cast dans le code source, pas un invisible cast inséré par le compilateur.

    • Merci, c'est un éclairage intéressant sur la "garantie" et de l'invisible cast fait par le compilateur par rapport à une fonte de le faire de manière explicite dans mon propre code.
    • Oui, la valeur de l'onu-réifiée des génériques est quelque peu limitée, mais c'est une chose qu'il ne fournissez. Juste pour préciser, cela nécessite que votre code compile sans type avertissements de sécurité.
    • Salut erickson, je suis d'accord que cela est vraiment la meilleure solution. Vérifier ma réponse stackoverflow.com/questions/367626/... pour une version générique de cette solution.
    InformationsquelleAutor erickson
  2. 111

    C'est un problème courant lorsque vous traitez avec des pré-Java 5 Api. Pour automatiser la solution de erickson, vous pouvez créer le générique de la méthode:

    public static <T> List<T> castList(Class<? extends T> clazz, Collection<?> c) {
    List<T> r = new ArrayList<T>(c.size());
    for(Object o: c)
      r.add(clazz.cast(o));
    return r;
}

    Cela vous permet de faire:

    List<SyndEntry> entries = castList(SyndEntry.class, sf.getEntries());

    Parce que cette solution vérifie que les éléments ont, en effet, le bon type de l'élément par le biais d'un casting, c'est sûr, et ne nécessite pas de SuppressWarnings.

    • Concernant la méthode proposée par Bruno, Wouldnt ce mal performances de l'application lorsque avoir des Listes avec de nombreux éléments?. Java serait d'avoir à jeter chacune et chacun d'entre eux.
    • Si vous voulez des garanties, c'est le coût. Est-il une autre option moins coûteuse? Évidemment, si vous avez le contrôle sur le invoquée raw collection retour de la méthode, ou même de l'invocation de la méthode ou de l'accès à la collection à l'aide d'un paresseux-approche par la demande. Tout ce qui tient compte de l'ensemble de la collection, après l'invocation de la méthode?
    InformationsquelleAutor Bruno De Fraine
  3. 25

    Il ressemble SyndFeed n'est pas l'utilisation de génériques.

    Vous pourriez avoir soit un dangereux en fonte et un avertissement de suppression:

    @SuppressWarnings("unchecked")
List<SyndEntry> entries = (List<SyndEntry>) sf.getEntries();

    ou appelez Les Collections.checkedList - même si vous aurez toujours besoin de supprimer l'avertissement:

    @SuppressWarnings("unchecked")
List<SyndEntry> entries = Collections.checkedList(sf.getEntries(), SyndEntry.class);
    • Depuis qu'ils ont à la fois supprimer l'avertissement, aucun avantage à l'un ou l'autre, ou une préférence? Merci! Aussi: est-ce que le plâtre nécessaire si l'décoché la suppression est en place?
    • Eh bien, Collections.checkedList permettra d'éviter l'ajout de toute non-SyndEntry éléments plus tard. Personnellement, je n'utilisez pas checkedList beaucoup, mais alors je n'ai pas souvent dans cette case décochée cast situation de toute façon...
    InformationsquelleAutor Jon Skeet
  4. 8

    Avez-vous écrit le SyndFeed?

    Ne sf.getEntries retour Liste ou List<SyndEntry>? Ma conjecture est qu'il renvoie List et la modifier pour revenir List<SyndEntry> va résoudre le problème.

    Si SyndFeed est partie d'une bibliothèque, je ne pense pas que vous pouvez supprimer l'avertissement sans ajouter le @SuppressWarning("unchecked") annotation à votre méthode.

    InformationsquelleAutor Alex B
  5. 2

    Si vous utilisez de Goyave et de tout ce que vous voulez faire est de parcourir vos valeurs: 

    for(SyndEntry entry: Iterables.filter(sf.getEntries(), SyndEntry.class){
  ...
}

    Si vous avez besoin d'une Liste vous pouvez utiliser 

    List<SyndEntry> list = Lists.newArrayList(
    Iterables.filter(sf.getEntries(), SyndEntry.class));

    ou

    List<SyndEntry> list = ImmutableList.copyOf(
    Iterables.filter(sf.getEntries(), SyndEntry.class));
    InformationsquelleAutor Joseph K. Strauss
  6. 1
    SyndFeedInput fr = new SyndFeedInput();
SyndFeed sf = fr.build(new XmlReader(myInputStream));
List<?> entries = sf.getEntries();
    • Même si le code fourni ici résout le problème, je vous encourage d'ailleurs à expliquer brièvement pourquoi il le fait. Veuillez expliquer pourquoi la posté réponse résout le problème.
    InformationsquelleAutor Honglonglong
  7. 1

    Si vous regardez la javadoc de la classe SyndFeed (je suppose que vous faites référence à la classe com.sun.syndication.feed.synd.SyndFeed), la méthode getEntries() ne retourne pas java.util.List<SyndEntry>, mais renvoie simplement java.util.List.

    Si vous avez besoin d'un cast explicite pour cette.

    InformationsquelleAutor Shyam
  8. 0

    Si vous ne voulez pas mettre de @SuppressWarning("unchecked") sur chaque sf.getEntries() l'appel, vous pouvez toujours faire un wrapper qui retourne la Liste.

    Voir cette autre question

    InformationsquelleAutor Boune
  9. 0

    Encore plus facile

    return new ArrayList<?>(getResultOfHibernateCallback(...))

    • Ensuite, vous traiter avec un bon casting (re-casting?) au moment de l'utilisation de chaque élément dans ArrayList<?>.
    InformationsquelleAutor DennisTemper