Home / ぼやきごと / 2017-07-06 / C#:ジェネリック版 IEnumerable 実装クラスなのにLINQが使えない…だと…!?
C#:ジェネリック版 IEnumerable 実装クラスなのにLINQが使えない…だと…!?

[前へ] [次へ] [目次]

C#:ジェネリック版 IEnumerable 実装クラスなのにLINQが使えない…だと…!?

C#といえばLINQ、LINQといえば IEnumerable<T> (以下 IE<T>)なわけですが、先日仕事でちょっとハマった出来事がありました。

※以降のコードは、元のコードの問題点のみわかるように改変したものです。

とあるアセンブリの IE<T> を実装しているはずのクラスオブジェクトに対してLINQを使おうとしたところ…(実際は Visual Studio 2015 でしたが本記事では 2017 で確認)

すべて開くすべて閉じる
  1
  2
  3
  4
  5
  6
  7
  8
  9
 10
 11
 12
 13
 14
 15
-
|
|
!
-
|
|
|
!
-
|
-
|
!
!
// 再現コードの一部抜粋
// FooCollection は IE<Foo> を実装している…はず…。
// もちろん using System.Linq; は書いてあります。
 
/// <summary>
/// FooCollection の各要素の Value プロパティ値を出力する。
/// </summary>
/// <param name="foos">FooCollection 。</param>
static void PrintFooValues(FooCollection foos)
{
    foreach (var v in foos.Select(item => item.Value))
    {
        Console.WriteLine(v);
    }
}
error CS1061: 'FooCollection' に 'Select' の定義が含まれておらず、型 'FooCollection' の最初の引数を受け付ける拡張メソッド 'Select' が見つかりませんでした。using ディレクティブまたはアセンブリ参照が不足していないことを確認してください。

error CS1061: 'FooCollection' に 'Select' の定義が含まれておらず、型 'FooCollection' の最初の引数を受け付ける拡張メソッド 'Select' が見つかりませんでした。using ディレクティブまたはアセンブリ参照が不足していないことを確認してください。

LINQが使えない…だと…!?*1

ビルドエラー内容を見ると、 IE<T> が実装されていないかのような内容です。実際、 int 等に対してLINQを使おうとしてもこれと同じエラーになります。

対象クラスが他人の作ったアセンブリ内で定義されていることもあり、アセンブリのCLRバージョンを調べたり、Google先生に「LINQ 特定クラス 使えない」で聞いてみたりとしてみたもののわからず。

ふと、改めて問題のクラスの定義をよくよく見てみると…

public class FooCollection : BaseCollection, IEnumerable<Foo>

…ん?そういえばこの BaseCollection の定義を見てなかったな?

public class BaseCollection : KeyedCollection<int, Base>

これだー!!

KeyedCollection<int, Base>IE<Base> を実装しています*2。つまり FooCollectionIE<Foo>IE<Base> の2つの IE<T> を実装していたのです。

ならそういうエラーメッセージにしてよ…。

ちなみにLINQを使わず次のようなコードにすると…

すべて開くすべて閉じる
  1
  2
  3
  4
  5
  6
  7
 
-
|
-
|
!
!
static void PrintFooValues(FooCollection foos)
{
    foreach (var foo in foos)
    {
        Console.WriteLine(foo.Value);
    }
}

問題なくビルドが通ります。 fooFoo に型推論されます。継承ツリーの末端側で実装されている IE<T> が優先されるようですね。

当然ながら、末端で複数の IE<T> を実装したクラスの場合は foreach でもビルドエラーになります。…が、そのエラーメッセージは次のように大変わかりやすいものとなっています。

error CS1640: 'IEnumerable<T>' の複数のインスタンスを実装するため、foreach ステートメントは、型 'MyCollection' の変数では操作できません。特定のインターフェイスのインスタンス化にキャストしてください。

foreach パイセンまじカッケーっす!…いや、C#コンパイラがLINQに対してツンツンすぎるのか?

今回問題となったクラスは IE<Foo> を後付けで実装した香りがプンプンしていて、言ってしまえばクラス設計ミス*3です。このアセンブリの開発者はLINQを使っておらず問題に気付かなかったのでしょう。C#erの面汚しめ…(言い過ぎ)。

とはいえ大人の事情でアセンブリを修正してもらうことはできません。ではどうするかというと、要はどちらの IE<T> を使おうとしているのかコンパイラが判断できないのが原因なので、次のようにキャストしてしまえば問題なくLINQが使えるようになります。

すべて開くすべて閉じる
  1
  2
  3
  4
  5
  6
  7
  8
  9
-
!
 
-
|
-
|
!
!
// 再現コード修正版
 
static void PrintFooValues(FooCollection foos)
{
    foreach (var v in ((IEnumerable<Foo>)foos).Select(item => item.Value))
    {
        Console.WriteLine(v);
    }
}

IEnumerable (非ジェネリック版)に対するLINQメソッドはそのまま使えるので、 Cast<T>, OfType<T> 等のメソッドで IE<T> に変換するという手もありますが、やや無駄な処理ですかね。

インタフェースの仕様上は、複数の IE<T> を実装することになんら問題はありません。ですがLINQで使おうとすると不都合があるため、聡明なC#er諸氏はそのような実装を行わないようにしましょう。してくださいお願いします。

Category: [C#][Visual Studio][プログラミング] - 2017-07-06 04:28:13

[前へ] [次へ] [目次]

*1 C#erにとってLINQが使えないことは死刑宣告に等しい。
*2 FooBase を継承している。
*3 BaseCollection をジェネリックにすべき。
添付ファイル: fileVS2017_BuildError_CS1061.png 81件 [詳細]