Loại mã của tôi có an toàn không?

Question

Tôi khá chắc chắn đây là loại an toàn, nhưng chỉ muốn kiểm tra như Eclipse đang yêu cầu tôi đặt một chú thích @SuppressWarnings("unchecked").

Map<String, IFace> faces; 

public <T extends IFace> T getFace(String key) 
{ 
    return (T) faces.get(key); 
} 

Answer 1

Nó không an toàn. Bạn đang upcasting ở đây vì vậy nếu bạn cast vào một lớp học không tương thích có nguồn gốc, bạn sẽ đi qua một lỗi tại một số điểm.

Ví dụ: nếu A_Face và B_Face cả hai đều mở rộng IFace. Bạn có thể tại một số thời điểm được đúc B_Face dưới dạng A_Face không an toàn.

Answer 2

Nhìn vào trường hợp cực đoan. Cho phép nói rằng IFace là acutally Object, mã của bạn sau đó trông như thế này:

static Map<String, Object> myMap = new HashMap<>(); 

public static void main(String[] args) throws Exception { 
    myMap.put("ONE", 1); 
    myMap.put("TWO", "TWO"); 
    myMap.put("THREE", new Date()); 

    final Calendar calendar1 = getThing("ONE"); 
    final Calendar calendar2 = getThing("TWO"); 
    final Calendar calendar3 = getThing("THREE"); 
} 

public static <T> T getThing(String key) { 
    return (T) myMap.get(key); 
} 

Vì vậy, bạn đang đặt tại class hơn extends Object vào Map của bạn (vì vậy bất kỳ class).

Tuy nhiên, khi bạn gọi getThing bạn đang thực hiện truyền ngầm đến loại mong muốn của mình. Rõ ràng là tôi cũng có thể gọi số getThing với bất kỳ số class nào và nó sẽ cố gắng truyền đến nó một cách mù quáng.

Trong ví dụ trên, tôi đang đặt một số thứ vào số Map và sau đó cố gắng truy xuất chúng tất cả dưới dạng Calendar s.

Answer 3

Một cách cổ điển để xử lý này là với một "container không đồng nhất typesafe":

Map<Class<?>, IFace> faces; 

public <T extends IFace> T getFace(Class<T> key) { 
    return t.cast(faces.get(key)); 
} 

Bạn sử dụng lớp của giao diện như một chìa khóa, chứ không phải là một chuỗi, và sau đó bạn có thể sử dụng các lớp đều đạt như một chìa khóa để đúc một cách an toàn giá trị trả lại cho đúng loại.