Wie verwende ich eine foreach-Schleife in Java, um die Werte in einer HashMap zu durchlaufen?

Question

Ich versuche, den folgenden Code zu kompilieren:

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; 
} 

Ich erhalte eine Fehlermeldung in dem für Zeile, die sagt:

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

Die getData() Methode gibt ein Object (aber in diesem Fall ist die Object zurück hat die HashMap Struktur). MyClass.Key ist eine Enumeration, die ich für die Zwecke meiner Anwendung erstellt habe (in einer anderen Klassendatei - MyClass).

Wenn ich eine foreach-Schleife mit der gleichen Struktur in MyClass.java erstellt habe, trat dieses Problem nicht auf.

Was mache ich falsch?

Answer 1

Eine etwas effizienter Weg, dies zu tun:

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

Wenn möglich, definieren „getData“ also brauchst du die Besetzung nicht.

Answer 2

Wechsel:

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

zu

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

Das Problem ist, dass data.keySet() eine Collection<Object> zurück, wenn Daten nur Map ist. Sobald Sie es generisch machen, gibt keySet() eine Collection<MyClass.Key> zurück. Noch besser ... iterieren über die entrySet(), die eine Collection<MyClass.Key, String> sein wird. Es vermeidet die zusätzlichen Hash-Lookups.

Answer 3

Motlins Antwort ist korrekt.

Ich habe zwei Noten ...

1. nicht toString += ..., verwenden Sie aber StringBuilder stattdessen verwenden und Daten anhängen.

2. Besetzung, die Martin vorgeschlagen hat, wird Ihnen unkontrollierte Warnung geben, die Sie nicht loswerden können, weil es wirklich unsicher ist.

Ein anderer Weg, ohne (und mit String) Warnung:

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

Dies funktioniert, weil toString Methode, die Sie auf key rufen in Objektklasse definiert ist, so dass Sie bei Gießen nicht brauchen alle.

Die Verwendung von entrySet ist noch besser, da es keine weitere Suche in der Karte durchführen muss.

Answer 4

Sie konnten die entrySet greifen stattdessen zu vermeiden, dass die Schlüsselklasse benötigen:

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

Ich fand dieses einfache Beispiel unter java forum.Seine Syntax ist der foreach der Liste sehr ähnlich, die, was ich suchte.

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

[EDIT:] Ich habe es getestet und es funktioniert perfekt.