Как мне использовать цикл foreach в Java для перебора значений в HashMap?

Question

Я пытаюсь скомпилировать следующий код:

private String dataToString(){
    Map data = (HashMap<MyClass.Key, String>) getData();
    String toString = "";
    for( MyClass.Key key: data.keySet() ){
        toString += key.toString() + ": " + data.get( key );
    return toString;
}

Я получаю сообщение об ошибке в строке for, в котором говорится:

incompatible types
found : java.lang.Object
required: MyClass.Key

Тот Самый getData() метод возвращает Object (но в данном случае Object возвращенный имеет HashMap структура). MyClass.Key это перечисление, которое я создал для целей моего приложения (в другом файле класса - MyClass).

Когда я создал цикл foreach с той же структурой в MyClass.java, Я не сталкивался с этой проблемой.

Что я делаю не так?

Answer 1

Несколько более эффективный способ сделать это:

  Map<MyClass.Key, String> data = (HashMap<MyClass.Key, String>) getData(); 
  StringBuffer sb = new StringBuffer();
  for (Map.Entry<MyClass.Key,String> entry : data.entrySet()) {
       sb.append(entry.getKey());
       sb.append(": ");
       sb.append(entry.getValue());
   }
   return sb.toString();

Если это вообще возможно, определите " getData " так что вам не нужен актерский состав.

Answer 2

Изменение:

Map data = (HashMap<MyClass.Key, String>) getData();

Для

Map<MyClass.Key, String> data = (HashMap<MyClass.Key, String>) getData();

Проблема в том, что data.keySet() возвращает Collection<Object> если данные - это просто Map.Как только вы сделаете его универсальным, keySet() вернет a Collection<MyClass.Key>.Даже лучше...выполните итерацию по entrySet(), который будет представлять собой Collection<MyClass.Key, String>.Это позволяет избежать дополнительных поисков по хэшу.

Answer 3

Я нашел этот простой пример на Java-форуме . Его синтаксис очень похож на foreach в List , который я и искал.

import java.util.Map.Entry;
HashMap nameAndAges = new HashMap<String, Integer>();
for (Entry<String, Integer> entry : nameAndAges.entrySet()) {
        System.out.println("Name : " + entry.getKey() + " age " + entry.getValue());
}

[РЕДАКТИРОВАТЬ:] Я проверил его, и он отлично работает.

Answer 4

Вместо этого вы можете получить entrySet, чтобы не нуждаться в классе ключа:

private String dataToString(){    
    Map data = (HashMap<MyClass.Key, String>) getData();    
    String toString = "";    
    for( Map.Entry entry: data.entrySet() ) {        
        toString += entry.getKey() + ": " + entry.getValue();
    }    
    return toString;
}

Answer 5

Ответ Мотлина правильный.

У меня есть две заметки ...

<Ол>

Не используйте toString += ..., а используйте StringBuilder и добавляйте к нему данные.

В ролях, которые предложил Мартин, вы получите непроверенное предупреждение, от которого вы не сможете избавиться, потому что это действительно небезопасно.

Другой способ, без предупреждения (и с StringBuilder):

private String dataToString(){
    Map<?, ?> data = (Map<?, ?>) getData();
    StringBuilder toString = new StringBuilder();
    for (Object key: data.keySet()) {
        toString.append(key.toString());
        toString.append(": ");
        toString.append(data.get(key));
    }
    return toString.toString();
}

Это работает, потому что метод toString, который вы вызываете key, определен в классе Object, поэтому вам вообще не нужно приводить.

Использование entrySet - еще лучший способ, так как для этого не нужно выполнять другой поиск на карте.