クラスでJavaストリームをフィルタリングする場合、instanceofの代わりはありますか？

1つのクラスを拡張するすべての型がJavaコレクションにパックされるプロジェクトで、予期しない状況が発生しました。ただし、そのクラスの特定の拡張のみに追加のメソッドが含まれています。それを「also（）」と呼びましょう。他の拡張機能にはないことをはっきりさせておきます。そのコレクション内のすべてのアイテムで1つのタスクを実行する直前に、それを実装するすべてのアイテムでalso（）を呼び出す必要があります。

最も簡単な方法はこれです：

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

それは正常に動作しますが、「instanceof」がそこにあるのは快適ではありません。これは通常、コードの悪臭のマーカーです。このクラスを取り除くためだけにこのクラスをリファクタリングする可能性は非常に高いです。しかし、劇的なことをする前に、コミュニティに問い合わせて、StreamsまたはCollectionsのどちらかについてより経験のある人がより簡単な解決策を持っているかどうかを確認したいと思いました。

例（確かに非排他的）として、クラスによってエントリをフィルタリングするコレクションのビューを取得することは可能ですか？

私は.mapあなたのためにキャストをするために電話で投げることをお勧めします。その後、後のコードで「本物」を使用できます。

前：

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

後：

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

これは完璧ではありませんが、少し片付けているようです。

1つのクラスを拡張するすべての型がJavaコレクションにパックされるプロジェクトで、予期しない状況が発生しました。ただし、そのクラスの拡張機能のみがメソッドを実装します。「also（）」としましょう。そのコレクション内のすべてのアイテムで1つのタスクを実行する直前に、それを実装するすべてのアイテムでalso（）を呼び出す必要があります。

それは明らかに欠陥のある設計です。あなたが書いたものから、それが何を意味するのかは明らかではありませんが、1つのクラスはメソッドを実装していません。それが何もしない場合は問題ではないので、不要な副作用があると思います。

それは正常に動作しますが、「instanceof」がそこにあるのは快適ではありません。

あなたの根性は正しいです。優れたオブジェクト指向設計は、オブジェクトが正確に何であるかを知ることなく機能し、それが特別な種類のインスタンスであるかどうかに関係します。

しかし、劇的なことをする前に、コミュニティに問い合わせて、StreamsまたはCollectionsのどちらかについてより経験のある人がより簡単な解決策を持っているかどうかを確認したいと思いました。

リファクタリングは劇的ではありません。コードの品質が向上します。

与えられたコードベースで最も簡単な解決策は、2つの別個のコレクションがあることを確認することです。1つは親を持ち、もう1つは子クラスを持ちます。しかし、それはかなりきれいではありません。

実際の内容を知らずに特定の推奨事項を提示することは困難ですが、次のようにSpecificItemなりalso()ます。

also()のスーパークラスで定義しSpecificItemます。何もしないデフォルトの実装を指定します（空のメソッド本体を指定してください）。必要に応じて、個々のサブクラスでオーバーライドできます。also実際の内容によっては、関係するすべてのクラスにとって意味のある名前に変更する必要がある場合があります。

代替：

1. 作るSpecificItemは、インターフェイスを実装（「フィルター処理」と言います）
2. Streamを拡張する新しいクラスを作成する
3. インターフェイスを実装するオブジェクトを受け入れる新しい 'filter'メソッドを作成します
4. 元のストリームメソッドをオーバーライドし、それを実装にリダイレクトします（述語パラメーターをインターフェイスにキャストすることにより）

そうすれば、インターフェイスを実装するオブジェクトだけが、テーマ自体をメソッドに渡すことができます... instanceofを使用する必要はありません

public class MyStream extends Stream
{
    //...

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

ここでも明らかなものを見逃さないようにしたいのですが（@Tristan Burnsideのコメントで示唆されているように）、なぜ最初にSpecificItem.method()呼び出すことができないのalso()ですか？

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

例（確かに非排他的）として、クラスによってエントリをフィルタリングするコレクションのビューを取得することは可能ですか？

おそらくパフォーマンスへの影響（YMMV）を犠牲にして、私が考えることができるストリームyの方法は、キーとしてクラスをcollect() 経由Collectors.groupingBy()して、結果から必要なものを選択することですMap。値はとして保存されるためList、a でそのようなフィルタリングを行うことを期待していて、そのSetフィルターSetから除外したい場合は、結果に値をさらに追加するための追加の手順が必要になりSetます。

これが私のアプローチです：

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

instanceof、明示的なキャストはありません（実際には明示的なキャストですが、メソッド呼び出しによってマスクされます）。これはできる限り良いと思います。