0

リストを定義するクラスと、いくつかのプロパティを持つリスト内のアイテムを検索する2つのループ関数があります。これらの2つのループ関数を組み合わせて、内部のBar.Function（）をパラメーターとして持つもう1つの汎用ループ関数にすることはできますか？

class Foo {
    List<Bar> bar;

    List<int> SomeFunc() {
        List<int> list;
        for (i...) {
           if (bar[i].IsSomething()) {
               list.Add(i);
           }
        }
        return list;
    }


    List<int> SomeOtherFunc() {
        List<int> list;
        for (i...) {
           if (bar[i].IsSomethingElse()) {
               list.Add(i);
           }
        }
        return list;
    }

}
4

3 に答える 3

4

はい、ラムダを使用するとこれは簡単です。

class Foo 
{
        List<Bar> bar;

        List<int> SomeFunc()
        {
            return LoopedTest(b => b.IsSomething());
        }


        List<int> SomeOtherFunc()
        {
            return LoopedTest(b => b.IsSomethingElse());
        }

        private List<int> LoopedTest(Predicate<Bar> test)
        {
            List<int> list = new List<int>();
            for (int i = 0; i < bar.Count; i++)
            {
                if (test(bar[i]))
                {
                    list.Add(i);
                }
            }
            return list;
        }

}

他の人が述べているように、この特定のケースでは、あなたが行っているフィルタリングのためにすでに比較的簡潔な構成があります（phoogの提案で改善されています）：

return IEnumerable.Range(0, bar.Count).Where(i => bar[i].IsSomething()).ToList();

また、これらのソリューションの両方を組み合わせて、非常にクリーンなコードを取得することもできます。

class Foo 
{
        List<Bar> bar;

        List<int> SomeFunc()
        {
            return FindIndexes(b => b.IsSomething());
        }

        List<int> SomeOtherFunc()
        {
            return FindIndexes(b => b.IsSomethingElse());
        }

        private List<int> FindIndexes(Predicate<Bar> test)
        {
            return IEnumerable.Range(0, bar.Count).Where(i => test(bar[i])).ToList();
        }
}
于 2013-01-15T02:25:11.000 に答える
0

単純なLINQ呼び出しはどうですか？

 List<int> res = list.Select(x => x.SomeFunc).Union(list.Select(y => y.SomeFunc2))
于 2013-01-15T02:25:42.790 に答える
-1

アイテム自体のリストを取得する前に、一致のインデックスのリストを探している場合は、Linqを使用できます。

var selection = bar.Where(b => b.IsSomething()).ToList();

これはLinqを使用した単一のステートメントであるため、おそらく独自のメソッドに入れる必要はありませんが、そうすると、次のようになります。

private List<Bar> SomeFunc(Predicate<Bar> test)
{
    return bar.Where(test).ToList();
}

あなたが本当にインデックスを追いかけているのなら、Linqでは次のようになります：

var selection = bar.Select((b, i) => new { b, i })
                .Where(x => x.b.IsSomething())
                .Select(x => x.i)
                .ToList();

そして、メソッドに因数分解すると、次のようになります。

private List<Bar> SomeFunc(Predicate<Bar> test)
{
    return bar.Select((b, i) => new { b, i })
                .Where(x => test(x.b))
                .Select(x => x.i)
                .ToList();
}
于 2013-01-15T02:25:19.360 に答える