Copie superficielle d'une carte en Java

Si je comprends bien, il existe plusieurs façons (peut-être d'autres également) de créer une copie superficielle d'un Mapen Java:

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();

Une voie est-elle préférée à l'autre, et si oui, pourquoi?

Une chose qui mérite d'être mentionnée est que la deuxième méthode donne un avertissement "Cast non coché". Il faut donc en rajouter @SuppressWarnings("unchecked")pour le contourner, ce qui est un peu irritant (voir ci-dessous).

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

Il est toujours préférable de copier en utilisant un constructeur de copie. clone()en Java est cassé (voir SO: Comment remplacer correctement la méthode de clonage? ).

Josh Bloch sur la conception - Constructeur de copie versus clonage

Si vous avez lu l'article sur le clonage dans mon livre, surtout si vous lisez entre les lignes, vous saurez que je pense qu'il cloneest profondément brisé. [...] C'est une honte qui Cloneableest cassée, mais ça arrive.

Bloch (qui d'ailleurs a conçu et implémenté le framework Collection) est même allé plus loin en disant qu'il ne fournit la clone()méthode que «parce que les gens l'attendent». Il ne recommande PAS du tout de l'utiliser.

Je pense que le débat le plus intéressant est de savoir si un constructeur de copie est meilleur qu'une usine de copie, mais c'est une discussion complètement différente.

Ni l'un ni l'autre: le constructeur auquel vous faites référence est défini pour l' implémentation HashMap d'un Map , (ainsi que pour d'autres) mais pas pour l'interface Map elle-même (par exemple, considérez l' implémentation Provider de l'interface Map: vous ne trouvera pas ce constructeur).

En revanche, il n'est pas conseillé d'utiliser la clone()méthode, comme l'explique Josh Bloch.

En ce qui concerne l'interface Map (et votre question, dans laquelle vous demandez comment copier une Map, pas un HashMap), vous devez utiliser Map # putAll () :

Copie tous les mappages de la carte spécifiée vers cette carte (opération facultative). L'effet de cet appel est équivalent à celui d'appeler put (k, v) sur cette mappe une fois pour chaque mappage de la clé k à la valeur v dans la mappe spécifiée.

Exemple:

// 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);

Copiez une carte sans connaître sa mise en œuvre:

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