web-dev-qa-db-ja.com

C#:リスト内のすべてのアイテムの任意のプロパティの最大値と最小値を取得する

タイプIThingのアイテムを保持する特殊なリストがあります。

_public class ThingList : IList<IThing>
{...}

public interface IThing
{
    Decimal Weight { get; set; }
    Decimal Velocity { get; set; }
    Decimal Distance { get; set; }
    Decimal Age { get; set; }
    Decimal AnotherValue { get; set; }

    [...even more properties and methods...]
}
_

リスト内のすべてのものの特定のプロパティの最大値または最小値を知る必要がある場合があります。 「聞かないでください」という理由で、リストにそれを理解させます。

_public class ThingList : IList<IThing>
{
    public Decimal GetMaximumWeight()
    {
        Decimal result = 0;
        foreach (IThing thing in this) {
            result = Math.Max(result, thing.Weight);
        }
        return result;
    }
}
_

それはとてもいいです。ただし、最小の重量が必要な場合もあれば、最大の速度が必要な場合もあります。すべてのプロパティにGetMaximum*()/GetMinimum*()ペアは必要ありません。

1つの解決策はリフレクションです。次のようなもの(鼻を押さえて、コードの臭いが強い!):

_Decimal GetMaximum(String propertyName);
Decimal GetMinimum(String propertyName);
_

これを達成するためのより良い、臭いの少ない方法はありますか?

ありがとう、エリック

編集:@マット:.Net 2.0

結論:.Net 2.0(Visual Studio 2005を使用)にはこれ以上の方法はありません。たぶん、近いうちに.Net3.5とVisualStudio2008に移行する必要があります。みんなありがとう。

結論:リフレクションよりもはるかに優れたさまざまな方法があります。ランタイムとC#のバージョンによって異なります。違いについては、ジョンスキートの回答をご覧ください。すべての答えは非常に役に立ちます。

Sklivvzの提案(匿名メソッド)に行きます。 Sklivvzのアイデアを実装する他の人々(Konrad Rudolph、Matt Hamilton、Coincoin)からのコードスニペットがいくつかあります。残念ながら、私は1つの答えしか「受け入れる」ことができません。

どうもありがとうございました。 Sklivvzだけがクレジットを取得しますが、すべて「受け入れられた」と感じることができます;-)

18
EricSchaefer

はい、デリゲートメソッドと匿名メソッドを使用する必要があります。

例については、 ここ を参照してください。

基本的には、 リストの検索メソッド に似たものを実装する必要があります。

これがサンプル実装です

public class Thing
{
    public int theInt;
    public char theChar;
    public DateTime theDateTime;

    public Thing(int theInt, char theChar, DateTime theDateTime)
    {
        this.theInt = theInt;
        this.theChar = theChar;
        this.theDateTime = theDateTime;
    }

    public string Dump()
    {
        return string.Format("I: {0}, S: {1}, D: {2}", 
            theInt, theChar, theDateTime);
    }
}

public class ThingCollection: List<Thing>
{
    public delegate Thing AggregateFunction(Thing Best, 
                        Thing Candidate);

    public Thing Aggregate(Thing Seed, AggregateFunction Func)
    {
        Thing res = Seed;
        foreach (Thing t in this) 
        {
            res = Func(res, t);
        }
        return res;
    }
}

class MainClass
{
    public static void Main(string[] args)
    {
        Thing a = new Thing(1,'z',DateTime.Now);
        Thing b = new Thing(2,'y',DateTime.Now.AddDays(1));
        Thing c = new Thing(3,'x',DateTime.Now.AddDays(-1));
        Thing d = new Thing(4,'w',DateTime.Now.AddDays(2));
        Thing e = new Thing(5,'v',DateTime.Now.AddDays(-2));

        ThingCollection tc = new ThingCollection();

        tc.AddRange(new Thing[]{a,b,c,d,e});

        Thing result;

        //Max by date
        result = tc.Aggregate(tc[0], 
            delegate (Thing Best, Thing Candidate) 
            { 
                return (Candidate.theDateTime.CompareTo(
                    Best.theDateTime) > 0) ? 
                    Candidate : 
                    Best;  
            }
        );
        Console.WriteLine("Max by date: {0}", result.Dump());

        //Min by char
        result = tc.Aggregate(tc[0], 
            delegate (Thing Best, Thing Candidate) 
            { 
                return (Candidate.theChar < Best.theChar) ? 
                    Candidate : 
                    Best; 
            }
        );
        Console.WriteLine("Min by char: {0}", result.Dump());               
    }
}

結果:

Max by date: I: 4, S: w, D: 10/3/2008 12:44:07 AM
Min by char: I: 5, S: v, D: 9/29/2008 12:44:07 AM

10
Sklivvz

(.NET 2.0の回答、およびVS2005のLINQBridgeを反映するように編集されました...)

ここには3つの状況があります-OPには.NET2.0しかありませんが、同じ問題に直面している他の人々はそうではないかもしれません...

1).NET 3.5およびC#3.0の使用:次のようなオブジェクトにLINQを使用します。

decimal maxWeight = list.Max(thing => thing.Weight);
decimal minWeight = list.Min(thing => thing.Weight);

2).NET 2.0およびC#3.0の使用: LINQBridge と同じコードを使用します

3).NET 2.0およびC#2.0の使用: LINQBridge および匿名メソッドを使用します。

decimal maxWeight = Enumerable.Max(list, delegate(IThing thing) 
    { return thing.Weight; }
);
decimal minWeight = Enumerable.Min(list, delegate(IThing thing)
    { return thing.Weight; }
);

(上記をテストするためのC#2.0コンパイラがありません。変換があいまいな場合は、デリゲートをFunc <IThing、decimal>にキャストしてください。)

LINQBridgeはVS2005で動作しますが、拡張メソッド、ラムダ式、クエリ式などは取得できません。明らかにC#3に移行する方が良いオプションですが、同じ機能を自分で実装するよりもLINQBridgeを使用することをお勧めします。

これらの提案はすべて、最大値と最小値の両方を取得する必要がある場合に、リストを2回歩くことを含みます。ディスクからの読み込みが遅いなどの状況で、一度に複数の集計を計算したい場合は、私の "Push LINQ" コードを確認することをお勧めします。 MiscUtil で。 (これは.NET 2.0でも機能します。)

32
Jon Skeet

.NET 3.5とLINQを使用していた場合:

Decimal result = myThingList.Max(i => i.Weight);

これにより、最小値と最大値の計算は非常に簡単になります。

19
Matt Hamilton

.NET 3.5を使用している場合は、ラムダを使用しないのはなぜですか?

public Decimal GetMaximum(Func<IThing, Decimal> prop) {
    Decimal result = Decimal.MinValue;
    foreach (IThing thing in this)
        result = Math.Max(result, prop(thing));

    return result;
}

使用法:

Decimal result = list.GetMaximum(x => x.Weight);

これは強く型付けされており、効率的です。すでにこれを正確に実行している拡張メソッドもあります。

8
Konrad Rudolph

C#2.0および.Net 2.0の場合、Maxに対して次の操作を実行できます。

public delegate Decimal GetProperty<TElement>(TElement element);

public static Decimal Max<TElement>(IEnumerable<TElement> enumeration, 
                                    GetProperty<TElement> getProperty)
{
    Decimal max = Decimal.MinValue;

    foreach (TElement element in enumeration)
    {
        Decimal propertyValue = getProperty(element);
        max = Math.Max(max, propertyValue);
    }

    return max;
}

そして、これがあなたがそれをどのように使うかです:

string[] array = new string[] {"s","sss","ddsddd","333","44432333"};

Max(array, delegate(string e) { return e.Length;});

上記の関数を使用せずに、C#3.0、.Net 3.5、およびLinqでこれを行う方法は次のとおりです。

string[] array = new string[] {"s","sss","ddsddd","333","44432333"};
array.Max( e => e.Length);
3
Coincoin

これは、SkilwzのアイデアでC#2.0を使用した試みです。

public delegate T GetPropertyValueDelegate<T>(IThing t);

public T GetMaximum<T>(GetPropertyValueDelegate<T> getter)
    where T : IComparable
{
    if (this.Count == 0) return default(T);

    T max = getter(this[0]);
    for (int i = 1; i < this.Count; i++)
    {
        T ti = getter(this[i]);
        if (max.CompareTo(ti) < 0) max = ti;
    }
    return max;
}

あなたはそれをこのように使うでしょう:

ThingList list;
Decimal maxWeight = list.GetMaximum(delegate(IThing t) { return t.Weight; });
3
Matt Hamilton

一般化された.Net2ソリューションはどうですか?

public delegate A AggregateAction<A, B>( A prevResult, B currentElement );

public static Tagg Aggregate<Tcoll, Tagg>( 
    IEnumerable<Tcoll> source, Tagg seed, AggregateAction<Tagg, Tcoll> func )
{
    Tagg result = seed;

    foreach ( Tcoll element in source ) 
        result = func( result, element );

    return result;
}

//this makes max easy
public static int Max( IEnumerable<int> source )
{
    return Aggregate<int,int>( source, 0, 
        delegate( int prev, int curr ) { return curr > prev ? curr : prev; } );
}

//but you could also do sum
public static int Sum( IEnumerable<int> source )
{
    return Aggregate<int,int>( source, 0, 
        delegate( int prev, int curr ) { return curr + prev; } );
}
2
Keith

結論:.Net 2.0(Visual Studio 2005を使用)にはこれ以上の方法はありません。

あなたは答え(特にジョンの答え)を誤解しているようです。あなたは彼の答えからオプション3を使うことができます。 LinqBridgeを使用したくない場合でも、デリゲートを使用して、私が投稿したメソッドと同様に、Maxメソッドを自分で実装できます。

delegate Decimal PropertyValue(IThing thing);

public class ThingList : IList<IThing> {
    public Decimal Max(PropertyValue prop) {
        Decimal result = Decimal.MinValue;
        foreach (IThing thing in this) {
            result = Math.Max(result, prop(thing));
        }
        return result;
    }
}

使用法:

ThingList lst;
lst.Max(delegate(IThing thing) { return thing.Age; });
2
Konrad Rudolph