it-roy-ru.com

Мелкая копия карты в Java

Насколько я понимаю, есть несколько способов (возможно, и других) для создания поверхностной копии Map в 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();

Является ли один путь предпочтительнее другого, и если да, то почему?

Стоит отметить, что второй способ выдает предупреждение "Unchecked Cast". Таким образом, вы должны добавить @SuppressWarnings("unchecked"), чтобы обойти это, что немного раздражает (см. Ниже).

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

Всегда лучше копировать, используя конструктор копирования. clone() в Java не работает (см. SO: Как правильно переопределить метод клонирования? ).

Джош Блох на Дизайн - Копировать конструктор против клонирования

Если вы прочитали статью о клонировании в моей книге, особенно если вы читаете между строк, вы поймете, что я думаю, что clone глубоко нарушен. [...] Обидно, что Cloneable не работает, но это случается.

Блох (который, кстати, разработал и реализовал платформу Collection) даже сказал, что он предоставляет метод clone() только "потому что люди этого ожидают". Он на самом деле не рекомендует использовать его вообще.


Я думаю, что более интересная дискуссия о том, лучше ли конструктор копирования, чем фабрика копий, но это совсем другое обсуждение.

106
polygenelubricants

Ни то, ни другое: конструктор , на который вы ссылаетесь, не определено для реализации HashMapMap , (как и для других) но не для самого интерфейса Map (например, рассмотрим реализацию Provider интерфейса Map: вы не найдете этот конструктор).

С другой стороны, не рекомендуется использовать метод clone(), как объяснил Джош Блох.

Что касается интерфейса Map (и вашего вопроса, в котором вы спрашиваете, как скопировать Map, а не HashMap), вы должны использовать Map # putAll () :

Копирует все сопоставления с указанной карты на эту карту (необязательная операция). Эффект этого вызова эквивалентен эффекту вызова put (k, v) на этой карте один раз для каждого отображения от ключа k до значения v в указанной карте.

Пример:

// 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);
57
Luca Fagioli

Скопируйте карту, не зная ее реализации:

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