copia dei riferimenti una mappa in Java

Question

Se ho capito bene, ci sono un paio di modi (forse altri) per creare una copia di un Map in 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(); 

è un modo preferito su l'altro, e se sì, perché?

Una cosa che vale la pena menzionare è che il secondo modo fornisce un avviso "Cast non selezionato". Quindi devi aggiungere @SuppressWarnings("unchecked") per aggirarlo, il che è un po 'irritante (vedi sotto).

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

Answer 1

È sempre meglio copiare utilizzando un costruttore di copie. clone() in Java è rotto (vedere SO: How to properly override clone method?).

Josh Bloch on Design - Copy Constructor versus Cloning

Se avete letto l'articolo sulla clonazione nel mio libro, soprattutto se si legge tra le righe, si sa che penso clone è profondamente rotto. [...] È un peccato che Cloneable non funzioni, ma succede.

Bloch (che tra l'altro, ha progettato e realizzato la struttura di raccolta) anche andato oltre nel dire che egli fornisce solo il metodo clone() solo "perché la gente si aspetta". In realtà NON consiglia di usarlo affatto.

---

penso che il dibattito più interessante è se un costruttore di copia è meglio di una fabbrica di copia, ma questa è una discussione del tutto diversa.

Answer 2

Nessuno dei due: il constructor che si fa riferimento è definito per la HashMap implementazione di un Map, (così come per gli altri), ma non per la mappa stessa interfaccia (ad esempio, si consideri la Provider attuazione del Interfaccia della mappa: non troverai questo costruttore).

D'altro canto non è consigliabile utilizzare il metodo clone(), come spiegato da Josh Bloch.

Per quanto riguarda l'interfaccia Map (e della tua domanda, in cui si chiede come copiare una mappa, non un HashMap), si dovrebbe usare Map#putAll():

Copia tutti i mapping dal specificato mappa a questa mappa (operazione opzionale). L'effetto di questa chiamata è equivalente a quello di chiamando put (k, v) su questa mappa una volta per ogni mappatura dalla chiave k al valore v nella mappa specificata.

Esempio:

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

Answer 3

Copia una mappa senza conoscere la sua attuazione:

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