sécurité Type: décoché jeté

Question

Dans mon fichier contexte d'application de printemps, j'ai quelque chose comme:

<util:map id="someMap" map-class="java.util.HashMap" key-type="java.lang.String" value-type="java.lang.String"> 
    <entry key="some_key" value="some value" /> 
    <entry key="some_key_2" value="some value" /> 
</util:map> 

En classe java, la mise en œuvre ressemble à:

private Map<String, String> someMap = new HashMap<String, String>(); 
someMap = (HashMap<String, String>)getApplicationContext().getBean("someMap"); 

Dans Eclipse, je vois un avertissement qui dit:

sécurité type: non contrôlé de l'objet moulé à hashmap

Qu'est-ce que j'ai fait de mal? Comment puis-je résoudre le problème?

Answer 1

Eh bien, tout d'abord, vous gaspillez de la mémoire avec le nouvel appel de création HashMap. Votre deuxième ligne ignore complètement la référence à cette hashmap créée, la rendant alors disponible au garbage collector. Alors, ne faites pas cela, utilisez:

private Map<String, String> someMap = (HashMap<String, String>)getApplicationContext().getBean("someMap"); 

En second lieu, le compilateur se plaint que vous lancez l'objet à un HashMap sans vérifier si elle est un HashMap. Mais, même si vous deviez faire:

if(getApplicationContext().getBean("someMap") instanceof HashMap) { 
    private Map<String, String> someMap = (HashMap<String, String>)getApplicationContext().getBean("someMap"); 
} 

Vous obtiendrez probablement toujours cet avertissement. Le problème est, getBean renvoie Object, donc on ne sait pas quel est le type. Le convertir en HashMap directement ne causerait pas le problème avec le second cas (et peut-être qu'il n'y aurait pas d'avertissement dans le premier cas, je ne suis pas sûr de la pédantité du compilateur Java avec les avertissements pour Java 5). Cependant, vous le convertissez en HashMap<String, String>. Les HashMaps sont en réalité des cartes qui prennent un objet comme une clé et ont un objet comme valeur, HashMap<Object, Object> si vous voulez. Ainsi, il n'y a aucune garantie que lorsque vous obtenez votre bean, il peut être représenté comme HashMap<String, String> parce que vous pourriez avoir HashMap<Date, Calendar> parce que la représentation non générique qui est retournée peut avoir des objets.

Si le code compile et que vous pouvez exécuter String value = map.get("thisString"); sans aucune erreur, ne vous inquiétez pas de cet avertissement. Mais si la carte n'est pas complètement de chaînes de caractères à des valeurs de chaîne, vous obtiendrez un ClassCastException lors de l'exécution, car les génériques ne peuvent pas bloquer cela dans ce cas.

Answer 2

Un avertissement est juste cela. Un avertissement. Parfois, les avertissements ne sont pas pertinents, parfois ils ne le sont pas. Ils sont utilisés pour attirer votre attention sur quelque chose que le compilateur pense être un problème, mais peut-être pas.

Dans le cas des moulages, il y aura toujours un avertissement dans ce cas. Si vous êtes absolument certain qu'un casting particulier sera en sécurité, alors vous devriez envisager d'ajouter une annotation comme celui-ci (je ne suis pas sûr de la syntaxe) juste avant la ligne:

@SuppressWarnings (value="unchecked") 

Answer 3

Vous obtenez ce message parce que getBean renvoie une référence d'objet et vous le transtypez dans le bon type. Java 1.5 vous donne un avertissement. C'est la nature d'utiliser Java 1.5 ou mieux avec du code qui fonctionne comme ça. Le ressort a la version de typesafe

someMap=getApplicationContext().getBean<HashMap<String, String>>("someMap"); 

sur sa liste de tâches.

Answer 4

Le problème est qu'un casting est un contrôle d'exécution - mais en raison de l'effacement de type, lors de l'exécution il n'y a en fait pas de différence entre un HashMap<String,String> et HashMap<Foo,Bar> pour toute autre Foo et Bar.

Utilisez @SuppressWarnings("unchecked") et tenez votre nez. Oh, la liste et la campagne pour les génériques réifiés en Java :)

Answer 5

Comme les messages ci-dessus indiquent ne peut être différenciée entre un List<Object> et un List<String> ou List<Integer>.

Je l'ai résolu ce message d'erreur pour un problème similaire:

List<String> strList = (List<String>) someFunction(); 
String s = strList.get(0); 

ce qui suit:

List<?> strList = (List<?>) someFunction(); 
String s = (String) strList.get(0); 

Explication: La première conversion de type vérifie que l'objet est une liste sans se soucier de les types détenus à l'intérieur (puisque nous ne pouvons pas vérifier les types internes au niveau de la liste). La deuxième conversion est maintenant requise car le compilateur sait seulement que la liste contient des sortes d'objets. Cela vérifie le type de chaque objet dans la liste lors de son accès.

Answer 6

Si vous voulez vraiment vous débarrasser des avertissements, une chose que vous pouvez faire est de créer une classe qui s'étend de la classe générique.

Par exemple, si vous essayez d'utiliser

private Map<String, String> someMap = new HashMap<String, String>(); 

Vous pouvez créer une nouvelle classe comme telle

public class StringMap extends HashMap<String, String>() 
{ 
    // Override constructors 
} 

Ensuite, lorsque vous utilisez

someMap = (StringMap) getApplicationContext().getBean("someMap"); 

Le compilateur T-ELLE savoir quels sont les types (non plus génériques), et il n'y aura pas d'avertissement. Cela peut ne pas toujours être la solution parfaite, certains pourraient argumenter que ce genre de défait le but des classes génériques, mais vous réutilisez toujours le même code de la classe générique, vous déclarez simplement au moment de la compilation quel type vous voulez utiliser.

Answer 7

Une autre solution, si vous trouvez souvent le même objet et que vous ne voulez pas vider votre code avec @SupressWarnings("unchecked"), serait de créer une méthode avec l'annotation. De cette façon, vous centralisez la distribution et, espérons-le, réduisez la possibilité d'erreur.

@SuppressWarnings("unchecked") 
public static List<String> getFooStrings(Map<String, List<String>> ctx) { 
    return (List<String>) ctx.get("foos"); 
} 

Answer 8

Ci-dessous le code provoque la sécurité Avertissement de type

Map<String, Object> myInput = (Map<String, Object>) myRequest.get();

Solution

Créer une nouvelle carte objet sans mentionner les paramètres parce que le type d'objet tenu dans la liste est n ot vérifié.

Étape 1: Créer une nouvelle carte temporaire

Map<?, ?> tempMap = (Map<?, ?>) myRequest.get();

Étape 2: Instantiate la carte principale

Map<String, Object> myInput=new HashMap<>(myInputObj.size()); 

Étape 3: Itérer la carte temporaire et définir les valeurs dans la carte principale

for(Map.Entry<?, ?> entry :myInputObj.entrySet()){ 
     myInput.put((String)entry.getKey(),entry.getValue()); 
    }