Płytka kopia mapy w języku Java

Jak rozumiem, istnieje kilka sposobów (być może także innych) na utworzenie płytkiej kopii pliku Mapw Javie:

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

Czy jeden sposób jest preferowany od drugiego, a jeśli tak, to dlaczego?

Warto wspomnieć o tym, że druga metoda daje ostrzeżenie o „Unchecked Cast”. Musisz więc dodać, @SuppressWarnings("unchecked")aby to obejść, co jest trochę irytujące (patrz poniżej).

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

Zawsze lepiej jest kopiować za pomocą konstruktora kopiującego. clone()w Javie jest uszkodzony (patrz SO: Jak poprawnie zastąpić metodę clone? ).

Josh Bloch on Design - Copy Constructor versus Cloning

Jeśli przeczytałeś artykuł o klonowaniu w mojej książce, zwłaszcza jeśli czytasz między wierszami, będziesz wiedział, że moim zdaniem clonejest głęboko zepsuty. […] Szkoda, że Cloneablejest zepsuty, ale się zdarza.

Bloch (który, nawiasem mówiąc, zaprojektował i wdrożył framework Collection) poszedł nawet dalej, mówiąc, że dostarcza clone()metody tylko „dlatego, że ludzie tego oczekują”. W rzeczywistości wcale NIE zaleca jego używania.

Myślę, że bardziej interesującą debatą jest to, czy konstruktor kopii jest lepszy niż fabryka kopii, ale to zupełnie inna dyskusja.

Żaden z tych dwóch: konstruktor , do którego się odnosisz, nie jest zdefiniowany dla implementacji HashMap w Map (jak również dla innych), ale nie dla samego interfejsu Map (na przykład rozważ implementację dostawcy interfejsu Map: ty nie znajdzie tego konstruktora).

Z drugiej strony nie zaleca się stosowania tej clone()metody, jak wyjaśnia Josh Bloch.

W odniesieniu do interfejsu mapy (i twojego pytania, w którym pytasz, jak skopiować mapę, a nie HashMap), powinieneś użyć Map # putAll () :

Kopiuje wszystkie mapowania z określonej mapy do tej mapy (operacja opcjonalna). Efekt tego wywołania jest równoważny wywołaniu metody put (k, v) na tej mapie raz dla każdego odwzorowania klucza k na wartość v w określonej mapie.

Przykład:

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

Skopiuj mapę bez znajomości jej implementacji:

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