Flache Kopie einer Karte in Java

Soweit ich weiß, gibt es verschiedene Möglichkeiten (möglicherweise auch andere), eine flache Kopie von a Mapin Java zu erstellen :

Map<String, Object> data = new HashMap<String, Object>();
Map<String, Object> shallowCopy;

// first way
shallowCopy = new HashMap<String, Object>(data);

// second way
shallowCopy = (Map<String, Object>) ((HashMap<String, Object>) data).clone();

Wird ein Weg dem anderen vorgezogen, und wenn ja, warum?

Eine erwähnenswerte Sache ist, dass der zweite Weg eine "Unchecked Cast" -Warnung gibt. Sie müssen also etwas hinzufügen @SuppressWarnings("unchecked"), um es zu umgehen , was ein wenig irritierend ist (siehe unten).

@SuppressWarnings("unchecked")
public Map<String, Object> getDataAsMap() {
    // return a shallow copy of the data map
    return (Map<String, Object>) ((HashMap<String, Object>) data).clone();
}

Es ist immer besser, mit einem Kopierkonstruktor zu kopieren. clone()in Java ist kaputt (siehe SO: Wie überschreibe ich die Klonmethode richtig? ).

Josh Bloch über Design - Kopierkonstruktor versus Klonen

Wenn Sie den Artikel über das Klonen in meinem Buch gelesen haben, insbesondere wenn Sie zwischen den Zeilen lesen, werden Sie wissen, dass ich denke, dass er clonezutiefst gebrochen ist. [...] Es ist eine Schande, Cloneabledie kaputt ist, aber es passiert.

Bloch (der übrigens das Collection-Framework entworfen und implementiert hat) ging sogar noch weiter und sagte, dass er die clone()Methode nur "nur weil die Leute es erwarten" bereitstellt . Er empfiehlt es überhaupt nicht.

Ich denke, die interessantere Debatte ist, ob ein Kopierkonstruktor besser ist als eine Kopierfabrik, aber das ist eine ganz andere Diskussion.

Keiner der beiden: Der Konstruktor , auf den Sie sich beziehen, ist für die HashMap- Implementierung einer Map (sowie für andere) definiert, nicht jedoch für die Map-Schnittstelle selbst (betrachten Sie beispielsweise die Provider- Implementierung der Map-Schnittstelle: you wird diesen Konstruktor nicht finden).

Andererseits ist es nicht ratsam, die clone()von Josh Bloch erläuterte Methode zu verwenden .

In Bezug auf die Kartenschnittstelle (und Ihre Frage, in der Sie fragen, wie eine Karte kopiert werden soll, nicht eine HashMap) sollten Sie Map # putAll () verwenden :

Kopiert alle Zuordnungen von der angegebenen Karte in diese Karte (optionaler Vorgang). Der Effekt dieses Aufrufs entspricht dem des Aufrufs von put (k, v) auf dieser Map einmal für jede Zuordnung von Schlüssel k zu Wert v in der angegebenen Map.

Beispiel:

// HashMap here, but it works for every implementation of the Map interface
Map<String, Object> data = new HashMap<String, Object>();
Map<String, Object> shallowCopy = new HashMap<String, Object>();

shallowCopy.putAll(data);

Kopieren Sie eine Karte, ohne deren Implementierung zu kennen:

static final Map shallowCopy(final Map source) throws Exception {
    final Map newMap = source.getClass().newInstance();
    newMap.putAll(source);
    return newMap;
}