-1

Необходимо в HashMap сохранить "ключ -> массив значений".

Map<Integer, List<Data>> myMap = new HashMap();

Если записи с таким номером нет, то заполняем myMap:

if (!myMap.containsKey(pos)) {
    List<Data> dataArray = new ArrayList();
    dataArray.add(new Data(a, b, c, d));
    myMap.put(pos, dataArray);
}

Далее идет снова проверка на существование, и если ключ есть, то выводим значение:

if (myMap.containsKey(pos)) 
    myMap.get(pos)

Проблема в том, что если при первой проверке ключа нет, то заполняется, однако второе условие тоже срабатывает. Но если заново вызвать функцию, содержащую эти проверки, то для pos, который, по идее, уже есть в Map, containsKey говорит, что якобы записи с ключом pos еще нет, и начинает заново заполнять Map. Но на самом деле такой ключ в Map уже есть. Как это исправить?

P.S. После добавления в Map containsKey показывает, что запись есть. Map у меня глобальный. И если вызвать containsKey в другом участке кода, то containsKey уже вернет false при том, что запись есть. Почему так, и как это исправить?

13
  • Все исправил смотрите снова
    – Ivan
    10 фев 2017 в 7:12
  • Если у вас многопоточное приложение, то вместо HashMap вам стоит использовать ConcurrentHashMap или другую потокобезопасную реализацию Map
    – Ksenia
    10 фев 2017 в 7:20
  • Однопоточное.Просто нужно если данных нет в Map занести туда а потом оттуда брать.У меня получается постоянно заносятся и все.
    – Ivan
    10 фев 2017 в 7:23
  • 1
    @Ivan я никакие кнопки не жму, ибо в вопросе такого кода нет, а придумывать код в приступе телепатии не собираюсь. Повторюсь: в вопросе должен быть минимальный воспроизводимый пример код. У вас сейчас такого не предоставлено. Так что либо кто-то чудесным образом угадает что у вас за проблема где-то ещё, либо вопрос так и останется неотвеченным.
    – Regent
    10 фев 2017 в 7:58
  • 1
    Вернусь к вопросу pavlofff, Map статический? Вы проверяли, на то что процесс ПРОВАЛИВАЕТСЯ в тело первой ифки или просто на условии точку остановки ставили? 10 фев 2017 в 8:00

1 ответ 1

0

У себя для похожей цели использую такой метод.

/**
 * @param checkMap мапа, куда необходимо добавить инфу
 * @param key      ключ для мапы выше, по которому ищем значения
 * @param value    значение, которое необходимо добавить
 * @param isSet    если true коллекция инициализируется как HashSet, иначе как ArrayList
 * @param <K>      Тип ключа
 * @param <V>      Тип значения
 * @return true если в мапе был данный ключ
 */
public static <K, V> boolean addToGroupMap(Map<K, Collection<V>> checkMap, K key, V value, boolean isSet) {
    assert checkMap != null;
    boolean result = false;

    Collection<V> vList = checkMap.get(key);
    if (vList == null) {
        vList = isSet ? new HashSet<V>() : new ArrayList<V>();
        vList.add(value);
        checkMap.put(key, vList);
    } else {
        vList.add(value);
        result = true;
    }
    return result;
}

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.