Cách trả lại một đối tượng có nhiều loại

Question

Hãy lấy một ví dụ để làm cho nó dễ dàng hơn. Tôi xây dựng một danh sách mà nhà xây dựng có một số integer và một số List<Integer>. Danh sách của tôi sẽ chứa tất cả các phần tử của danh sách đã cho nhân với integer. danh sách của tôi không lưu trữ các yếu tố mới nhưng tính họ một cách nhanh chóng:

class MyList extends AbstractList<Integer> implements RandomAccess { 
    private final int multiplier; 
    private final List<Integer> list; 

    public MyList(int multiplier, List<Integer> list) { 
     this.multiplier = multiplier; 
     this.list = list; 
    } 

    @Override 
    public Integer get(int index) { 
     return list.get(index) * multiplier; 
    } 

    @Override 
    public int size() { 
     return list.size(); 
    } 
} 

Sau đó, chúng ta có thể gọi new MyList(3, list) với list = [0, 1, 2, 3] để có được [0, 3, 6, 9].

Tôi muốn giới hạn nhà phát triển cung cấp cho nhà xây dựng MyList danh sách cũng là RandomAccess, để chắc chắn anh ấy sẽ không hủy hoại buổi biểu diễn.

Tôi cố gắng để thay đổi các nhà xây dựng với:

public <E extends List<Integer> & RandomAccess> MyList(int multiplier, E list) 

MyList không phải là vấn đề nhưng bây giờ chúng ta không thể gọi các nhà xây dựng mà không sử dụng một thực hiện của cả hai List<Integer> và RandomAccess như ArrayList<Integer>. Vì vậy, một người có danh sách này: List<Integer> list = new ArrayList<>(); không thể làm new MyList(3, list); (Vì nó được khai báo với List<Integer> thay vì ArrayList<Integer>).

Các giải pháp khác mà tôi có là một trong những điều này:

public MyList(int multiplier, List<Integer> list) { 
     if(!(list instanceof RandomAccess)) { 
      // Do something like log or throw exception 
     } 
     this.multiplier = multiplier; 
     this.list = list; 
    } 

Nhưng bây giờ tôi không thể kiểm tra tại thời gian biên dịch nếu danh sách thực hiện RandomAccess, và tôi cần phải sử dụng instanceof và tôi ghét làm điều này.

Tôi chắc rằng có một cách tốt hơn nhưng nó là gì?

Answer 1

Bạn có thể áp dụng các giải pháp sử dụng bởi Collections.unmodifiableList. Thay vì một hàm tạo công khai, có một phương thức tĩnh trả về một trong hai cách triển khai thực hiện, một phương thức triển khai thực hiện RandomAccess, phương thức kia thì không.

Đây là mã cho Collections.unmodifiableList.

public static <T> List<T> unmodifiableList(List<? extends T> list) { 
    return (list instanceof RandomAccess ? 
      new UnmodifiableRandomAccessList<>(list) : 
      new UnmodifiableList<>(list)); 
} 

Tôi biết bạn đã nói bạn không thích sử dụng instanceof. Tôi cũng vậy, nhưng đôi khi đó là điều tốt nhất để làm.

Lưu ý rằng các giải pháp sử dụng các nhà xây dựng

public <E extends List<Integer> & RandomAccess> MyList(int multiplier, E list) 

không chỉ xấu xí, ở chỗ nó buộc các lập trình viên để cast (ví dụ như để một ArrayList), nhưng nó sẽ không thực sự làm việc. Ví dụ: nếu list là một phiên bản của Collections$UnmodifiableRandomAccessList, thậm chí không thể truyền cho một loại triển khai cả hai List và RandomAccess, bởi vì Collections$UnmodifiableRandomAccessList là riêng tư.

Answer 2

Nếu lớp bạn cần có một danh sách truy cập ngẫu nhiên, sau đó lớp học của bạn nên đối phó với điều đó và không đẩy nhu cầu của lớp học của bạn vào người gọi.Hơn nữa, nó sẽ đơn giản hơn để làm phép nhân trong các nhà xây dựng - bạn phải làm điều đó tại một số điểm, nhưng làm nó sớm có nghĩa là bạn có thể vứt bỏ rất nhiều mã:

class MyList extends ArrayList<Integer> { 

    public MyList(int multiplier, List<Integer> list) { 
     for (Integer i : list) 
      add(i * multiplier); 
    } 
} 

Đó là tất cả bạn cần, và an toàn hơn: với sự tuyệt đối của bạn, cả danh sách và người gọi của bạn đều có tham chiếu đến danh sách. Nếu sau khi gọi hàm dựng, cuộc gọi thay đổi danh sách, mã khác sử dụng danh sách nhân sẽ bất ngờ thấy các giá trị trong danh sách thay đổi.

Answer 3

Tôi khuyên bạn nên sử dụng instanceof. Trên thực tế đây là chính xác những gì các tài liệu RandomAccess gợi ý: danh sách các thuật toán

Generic được khuyến khích để kiểm tra xem danh sách cho trước là một instanceof giao diện này trước khi áp dụng một thuật toán mà sẽ cung cấp hiệu suất kém nếu nó được áp dụng cho một tuần tự danh sách truy cập và thay đổi hành vi của chúng nếu cần thiết để đảm bảo hiệu suất có thể chấp nhận được .

Nhà xây dựng của bạn có thể có khả năng có hai triển khai. Nếu RandomAccess được thực hiện sau đó nó sẽ lưu một tham chiếu đến List nếu không nó sẽ tạo ra một mới ArrayList và bản sao tất cả các yếu tố để nó:

class MyList { 
    private final int multiplier; 
    private final List<Integer> list; 

    public MyList(int multiplier, List<Integer> list) { 
     this.multiplier = multiplier; 
     if (list instanceof RandomAccess) 
      this.list = list; 
     else 
      this.list = new ArrayList<>(list); 
    } 

    public int get(int index) { 
     return multiplier * list.get(index); 
    } 
}