Có một sự thay thế cho thể hiện khi lọc một luồng Java theo lớp không?

Tôi có một tình huống bất ngờ trong một dự án trong đó tất cả các kiểu mở rộng một lớp được đóng gói vào một bộ sưu tập Java; nhưng chỉ một phần mở rộng cụ thể của lớp đó chứa một phương thức bổ sung. Hãy gọi nó là "cũng ()"; và để tôi nói rõ rằng không có phần mở rộng nào khác có nó. Ngay trước khi thực hiện một nhiệm vụ trên mỗi mục trong bộ sưu tập đó, tôi cũng cần gọi () trên mỗi mục thực hiện nó.

Cách dễ nhất về phía trước là đây:

stuff.stream().filter(item -> item instanceof SpecificItem)
            .forEach(item -> ((SpecificItem)item).also()));
stuff.stream().forEach(item -> item.method());

Nó hoạt động tốt, nhưng tôi không thoải mái với "instanceof" trong đó. Đó thường là một dấu hiệu của mùi mã xấu. Rất có khả năng tôi sẽ cấu trúc lại lớp này chỉ để loại bỏ nó. Tuy nhiên, trước khi tôi làm điều gì đó ấn tượng, tôi nghĩ rằng tôi sẽ kiểm tra với cộng đồng và xem liệu ai đó có nhiều kinh nghiệm hơn với Luồng hoặc Bộ sưu tập có giải pháp đơn giản hơn không.

Như một ví dụ (chắc chắn không có kết luận), có thể có được chế độ xem bộ sưu tập lọc các mục theo lớp không?

Tôi khuyên .mapbạn nên thực hiện một cuộc gọi để thực hiện vai diễn cho bạn. Sau đó, mã sau này của bạn có thể sử dụng 'thực tế'.

Trước:

stuff.stream().filter(item -> item instanceof SpecificItem)
            .forEach(item -> ((SpecificItem)item).also()));

Sau:

stuff.stream().filter(item -> item instanceof SpecificItem)
            .map(item -> (SpecificItem)item)
            .forEach(specificItem -> specificItem.also()));

Điều này không hoàn hảo, nhưng dường như làm sạch mọi thứ một chút.

Tôi có một tình huống bất ngờ trong một dự án trong đó tất cả các kiểu mở rộng một lớp được đóng gói vào một bộ sưu tập Java; nhưng chỉ một phần mở rộng của lớp đó thực hiện một phương thức. Hãy gọi nó là "cũng ()". Ngay trước khi thực hiện một nhiệm vụ trên mỗi mục trong bộ sưu tập đó, tôi cũng cần gọi () trên mỗi mục thực hiện nó.

Đó rõ ràng là một thiết kế bị lỗi. Từ những gì bạn đã viết, nó không rõ ràng, có nghĩa là gì, rằng một lớp không thực hiện phương thức. Nếu nó chỉ đơn giản là không làm gì , nó sẽ không thành vấn đề, vì vậy tôi cho rằng có một tác dụng phụ không mong muốn.

Nó hoạt động tốt, nhưng tôi không thoải mái với "instanceof" trong đó.

Ruột của bạn là đúng Thiết kế hướng đối tượng tốt sẽ hoạt động mà không cần biết thêm chính xác đối tượng là gì, cho dù đó là một thể hiện của một loại đặc biệt.

Tuy nhiên, trước khi tôi làm điều gì đó ấn tượng, tôi nghĩ rằng tôi sẽ kiểm tra với cộng đồng và xem liệu ai đó có nhiều kinh nghiệm hơn với Luồng hoặc Bộ sưu tập có giải pháp đơn giản hơn không.

Tái cấu trúc là không kịch tính. Nó cải thiện chất lượng mã.

Với cơ sở mã đã cho của bạn, giải pháp đơn giản nhất là, để đảm bảo, bạn có hai bộ sưu tập riêng biệt, một với cha mẹ và một với lớp con. Nhưng điều đó không hoàn toàn sạch sẽ.

Thật khó để đưa ra khuyến nghị cụ thể mà không biết những gì SpecificItemvà also()thực sự là gì, nhưng:

Xác định also()trên siêu lớp của SpecificItem. Cung cấp cho nó một triển khai mặc định mà không làm gì cả (cung cấp cho nó một thân phương thức trống). Các lớp con riêng lẻ có thể ghi đè lên nó nếu muốn. Tùy thuộc vào những gì alsothực sự là, bạn có thể cần phải đổi tên nó thành một cái gì đó có ý nghĩa cho tất cả các lớp liên quan.

một thay thế:

1. làm cho Cụ thể thực hiện một giao diện (nói "Có thể lọc")
2. tạo một lớp mới mở rộng Stream
3. tạo phương thức 'bộ lọc' mới chấp nhận đối tượng triển khai giao diện của bạn
4. ghi đè phương thức luồng ban đầu và chuyển hướng nó đến triển khai của bạn (bằng cách chuyển tham số vị ngữ sang giao diện của bạn)

theo cách đó, chỉ những đối tượng triển khai giao diện của bạn mới có thể truyền các chủ đề cho phương thức của bạn ... không cần sử dụng thể hiện

public class MyStream extends Stream
{
    //...

    Stream<Filterable> filter(Predicate<? implements Filterable> predicate)
    {
         return super.filter(predicate);
    }
}

Tôi hy vọng tôi không bỏ lỡ bất cứ điều gì rõ ràng ở đây (cũng như được đề xuất bởi nhận xét của @Tristan Burnside ), nhưng tại sao không thể SpecificItem.method()gọi also()trước?

public class SpecificItem extends Item {
    ...
    public void method() {
        also();
        super.method();
    }
}

Như một ví dụ (chắc chắn không có kết luận), có thể có được chế độ xem bộ sưu tập lọc các mục theo lớp không?

Một cách truyền phát mà tôi có thể nghĩ ra, với chi phí có thể là một số tác động hiệu suất (YMMV), là collect() thông qua Collectors.groupingBy()lớp làm khóa, sau đó chọn kết quả bạn muốn từ kết quả Map. Các giá trị được lưu trữ dưới dạng a List, vì vậy nếu bạn muốn thực hiện lọc như vậy trên a Setvà bạn hy vọng sẽ lọc được Setnó, thì bạn cũng sẽ cần một bước bổ sung để tiếp tục đưa các giá trị vào kết quả Set.

Đây là cách tiếp cận của tôi:

stuff.stream().filter(item -> SpecificItem.class.isInstance(item)).map(item  -> SpecificItem.class.cast(item)).forEach(item -> item.also());
stuff.stream().forEach(item -> item.method());

Không có thể hiện , không có phôi rõ ràng (thực ra nó là một biểu mẫu rõ ràng, nhưng bị che bởi một lệnh gọi phương thức). Tôi nghĩ rằng điều này là tốt như nó có thể nhận được.