Datentypen Generics

[Neumi5694]

Da ich des öfteren Listen von Objekten (verschiedene Typen) mit einem Schlüssel in einer Map hinterlege und immer wieder sehr ähnlichen Code benötige, um eine Liste zu erhalten, hab ich mir folgende Hilfsmethode geschrieben:

    public static <T> ArrayList<T> getOrAddToMap(Map<Object, ArrayList<T>> map, Object id) {
        ArrayList<T> result = map.get(id);
        if (result == null) {
            result = new ArrayList<>();
            map.put(id, result);
        }
        return result;
    }

Ziel ist es, aus der Map eine Liste anhand des Schlüssels zu erhalten und gegebenenfalls neu anzulegen, sollte dem Schlüssel noch keine Liste zugeordnet sein.

Wenn ich das Ganze dann aber so aufrufe:

    HashMap<Long, ArrayList<MyClass>> listCache = new HashMap<>();
    Long id = 123456;
    ArrayList<MyClass> aList = getOrAddToMap(listCache, id);

..dann mag das der Compiler nicht.
Wie müsste ich den Kopf der Methode umschreiben, damit das auch für andere Klassen als Long und MyClass aufgerufen werden kann?

 

[Saheeda]

Dir fehlt ne Klammer:
HashMap<Long, ArrayList<MyClass>> listCache = new HashMap<>();

Ansonsten sag mal bitte (mit entsprechenden Codezeilen), was der Compiler kritisiert.

 

[Neumi5694]

Oh, danke. Das war nur ein Tippfehler im Beispiel.

Hier die Meldung:

error: method getOrAddToMap in class MyTools cannot be applied to given types;
MyTools.getOrAddToMap(listCache, id);
required: Map<Object,ArrayList<T>>,Object
found: Map<Long,ArrayList<MyClass>>,Long
reason: cannot infer type-variable(s) T
(argument mismatch; Map<Long,ArrayList<MyClass>> cannot be converted to Map<Object,ArrayList<T>>)
where T is a type-variable:
T extends Object declared in method <T>getOrAddToMap(Map<Object,ArrayList<T>>,Object)




Mir kam grad der Gedanke, dass ich es ja nicht auf Listen limitieren muss.
Vielleicht kann ich ja anstatt ArrayList<T> einfach nur <T> verwenden, solange die Klasse einen leeren Konstruktor hat. Ohne List-Cast könnte es besser klappen.

 

[Harry Kane]

Ändere die getOrAddToMap Methode wie folgt ab:

public static <T, I> ArrayList<T> getOrAddToMap(Map<I, ArrayList<T>> map, I id) {
    ArrayList<T> result = map.get(id);
    if (result == null) {
        result = new ArrayList<T>();
        map.put(id, result);
    }
    return result;
}

 

[Neumi5694]

Perfekt, danke.

 

[Saheeda]

Vielleicht noch als kleine Ergänzung zu Harry Kane. Du kannst auch einschränken, welche Typen als Argument zugelassen werden:

public static <T, I extends Number> ArrayList<T> getOrAddToMap(Map<I, ArrayList<T>> map, I id) {
    ArrayList<T> result = map.get(id);
    if (result == null) {
        result = new ArrayList<T>();
        map.put(id, result);
    }
    return result;
}

Hier würdest du z.B. nur von "Number" abgeleitete Klassen als Typ zulassen, also z.B. Long, Integer, Float.
https://docs.oracle.com/javase/7/docs/api/java/lang/Number.html