Wie gehe ich mit ungeprüften Besetzungswarnungen um?

Question

Eclipse gibt mir eine Warnung der folgenden Form:

Typ Sicherheit: Ungeprüfter Cast von Object nach HashMap

Dies stammt aus einem Aufruf an eine API, über die ich keine Kontrolle habe und die Object zurückgibt:

HashMap<String, String> getItems(javax.servlet.http.HttpSession session) {
  HashMap<String, String> theHash = (HashMap<String, String>)session.getAttribute("attributeKey");
  return theHash;
}

Ich möchte Eclipse-Warnungen nach Möglichkeit vermeiden, da sie theoretisch zumindest auf ein potenzielles Codeproblem hinweisen. Ich habe allerdings noch keine gute Möglichkeit gefunden, dieses Problem zu beseitigen. Ich kann die einzelne betroffene Zeile in eine eigene Methode auslagern und Folgendes hinzufügen @SuppressWarnings("unchecked") zu dieser Methode, wodurch die Auswirkungen eines Codeblocks, in dem ich Warnungen ignoriere, begrenzt werden. Gibt es bessere Möglichkeiten? Ich möchte diese Warnungen in Eclipse nicht ausschalten.

Bevor ich zu dem Code kam, war es einfacher, aber es gab trotzdem Warnungen:

HashMap getItems(javax.servlet.http.HttpSession session) {
  HashMap theHash = (HashMap)session.getAttribute("attributeKey");
  return theHash;
}

Das Problem war, dass an anderer Stelle, wenn man versuchte, den Hash zu verwenden, Warnungen angezeigt wurden:

HashMap items = getItems(session);
items.put("this", "that");

Type safety: The method put(Object, Object) belongs to the raw type HashMap.  References to generic type HashMap<K,V> should be parameterized.

Answer 1

Überprüfen Sie einfach die Eingabe, bevor Sie sie machen.

Object someObject = session.getAttribute("attributeKey");
if(someObject instanceof HashMap)
HashMap<String, String> theHash = (HashMap<String, String>)someObject;  

Und für alle, die es wissen wollen: Es kommt durchaus vor, dass man Gegenstände erhält, bei denen man sich nicht sicher ist, um welche Art es sich handelt. Viele alte "SOA"-Implementierungen geben verschiedene Objekte weiter, denen man nicht immer trauen sollte. (Der Schrecken!)

EDITAR Ich habe den Beispielcode einmal geändert, um mit den Aktualisierungen des Posters übereinzustimmen, und nach einigen Kommentaren sehe ich, dass instanceof nicht gut mit Generics zusammenspielt. Die Änderung der Prüfung, um das äußere Objekt zu validieren, scheint jedoch gut mit dem Befehlszeilen-Compiler zu funktionieren. Überarbeitetes Beispiel jetzt gepostet.

Answer 2

Dadurch verschwinden die Warnungen...

 static Map<String, String> getItems(HttpSession session) {
        HashMap<?, ?> theHash1 = (HashMap<String,String>)session.getAttribute("attributeKey");
        HashMap<String,String> theHash = (HashMap<String,String>)theHash1;
    return theHash;
}

Answer 3

Lösung: Deaktivieren Sie diese Warnung in Eclipse. Nicht @SuppressWarnings es, nur deaktivieren Sie es vollständig.

Mehrere der oben vorgestellten "Lösungen" sind völlig daneben und machen den Code unlesbar, nur um eine dumme Warnung zu unterdrücken.