c# - (アップキャスト) インスタンスが特定のインターフェースを実装していないかどうかを調べる最良の方法

Question

これを行う必要があるのは「デザインの匂い」かもしれませんが、別の質問について考えてみると、これの逆を実装する最もクリーンな方法は何だろうと思っていました。

foreach(ISomethingable somethingableClass in collectionOfRelatedObjects)
{
  somethingableClass.DoSomething();
}

つまり、特定のインターフェイスを実装していないすべてのオブジェクトを取得/反復する方法は?

おそらく、最高レベルにアップキャストすることから始める必要があります。

foreach(ParentType parentType in collectionOfRelatedObjects)
{
  // TODO: iterate through everything which *doesn't* implement ISomethingable 
}

TODO を解決することで答えます: 最もクリーン/シンプルかつ/または最も効率的な方法で

score 6 · Accepted Answer

このようなもの？

foreach (ParentType parentType in collectionOfRelatedObjects) {
    if (!(parentType is ISomethingable)) {
    }
}

score 3 · Accepted Answer

これでうまくいくはずです：

collectionOfRelatedObjects.Where(o => !(o is ISomethingable))

score 3 · Accepted Answer

おそらく、最後まで行って変数名を改善するのが最善です。

foreach (object obj in collectionOfRelatedObjects)
{
    if (obj is ISomethingable) continue;

    //do something to/with the not-ISomethingable
}

score 0 · Accepted Answer

LINQ拡張メソッドOfType<>（）の助けを借りて、次のように書くことができます。

using System.Linq;

...

foreach(ISomethingable s in collection.OfType<ISomethingable>())
{
  s.DoSomething();
}

score 0 · Accepted Answer

JD OConal はこれを行うための最良の方法ですが、補足として、 as キーワードを使用してオブジェクトをキャストできます。その型でない場合は null を返します。

次のようなものです：

foreach (ParentType parentType in collectionOfRelatedObjects) {
    var obj = (parentType as ISomethingable);
    if (obj == null)  {
    }
}

5 に答える 5