Làm cách nào để sử dụng vòng lặp foreach trong Java để lặp qua các giá trị trong HashMap?

Question

Tôi cố gắng để biên dịch đoạn mã sau:

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

tôi nhận được một lỗi trong cho dòng nói rằng:

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

Phương pháp getData() trả về một Object (nhưng trong trường hợp này Object trở có cấu trúc HashMap). MyClass.Key là một enum mà tôi đã tạo ra cho các mục đích của ứng dụng của tôi (trong một tập tin lớp học - MyClass).

Khi tôi tạo vòng lặp foreach có cùng cấu trúc trong MyClass.java, tôi không gặp sự cố này.

Tôi đang làm gì sai?

Answer 1

Một cách nhẹ hiệu quả hơn để làm điều này:

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

Nếu có thể, xác định "getData" vì vậy bạn không cần dàn diễn viên.

Answer 2

Thay đổi:

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

để

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

Vấn đề là data.keySet() trả về một Collection<Object> nếu dữ liệu chỉ là một Map. Khi bạn làm cho nó chung chung, keySet() sẽ trả về một Collection<MyClass.Key>. Thậm chí tốt hơn ... lặp lại trên entrySet(), sẽ là Collection<MyClass.Key, String>. Nó tránh tìm kiếm băm bổ sung.

Answer 3

Câu trả lời của Motlin là chính xác.

Tôi có hai ghi chú ...

1. Không sử dụng toString += ..., nhưng sử dụng StringBuilder thay vào đó và nối thêm dữ liệu vào nó.

2. Diễn viên mà Martin đề xuất sẽ cung cấp cho bạn cảnh báo không được kiểm soát, mà bạn sẽ không thể thoát khỏi, vì nó thực sự không an toàn.

Một cách khác mà không cần cảnh báo (và với 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(); 
} 

này hoạt động, bởi vì phương thức toString mà bạn gọi vào key được định nghĩa trong lớp Object, vì vậy bạn không cần phải casting tại tất cả các.

Sử dụng entrySet thậm chí còn tốt hơn, vì không cần phải thực hiện tìm kiếm khác trong bản đồ.

Answer 4

Bạn có thể lấy các entrySet thay vào đó, để tránh cần lớp chính:

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

Tôi đã tìm thấy ví dụ đơn giản này tại java forum.Cú pháp của nó rất giống với cú pháp của Danh sách foreach, đó là những gì tôi đang tìm kiếm.

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:] Tôi đã thử nghiệm và hoạt động hoàn hảo.