LINQで最大値ではなく、最大値を持つ要素の取得とforeach。

Question

前提・実現したいこと

LINQの勉強をしています。
LINQで最大値ではなく、最大値を持つ要素を取得したいです。
いくつか方法はあると思いますが、Max()を使うのがおそらく1番シンプルに書けると思うのですが、
その書き方のご教示をお願いします。

また、該当のソースコードのforeach文をLINQで書き換えることは可能ですか？
こちらのご教示もお願いします。

該当のソースコード

ファイルサイズの最大値をもつFileInfoの要素を取得したいです。
ファイルサイズの最大値そのものは取得できました。

          　List<FileInfo> filelist = new List<FileInfo>();
            DirectoryInfo di = new DirectoryInfo(@"C:\Users\username\Desktop\testfolder");

            foreach (FileInfo fi in di.GetFiles())
            {
                filelist.Add(fi);
            }

            //リストの中で最大値のファイルサイズを取得する。
            long maxsize = filelist.Max(s => s.Length);
            Console.WriteLine(maxsize);

            //リストの中で最大値のファイルサイズを持つ要素を取得するにはどうすればいいですか？
            //Maxを使って取得することはできますか？

Answer 1

質問者さんが先のスレッドで使っていた OrderByDescending を使ってできると思います。

以下のような感じ。

FileInfo max = filelist.OrderByDescending(s => s.Length).FirstOrDefault();

もっとスマートな方法はあるかもしれませんが。

Comments

[退会済みユーザー]

ご回答ありがとうございます。
勉強になりました。
Maxを使って簡単にできるかと思っていたのですが、そのような方法はなさそうなんですね。
ありがとうございます。

[SurferOnWww]

Max を使うと、かなり無理やりっぽいですが、以下のような感じでしょうか。これももっとスマートな方法はあるかもしれませんが。

FileInfo max = filelist.Find(s => s.Length == filelist.Max(p => p.Length));

[退会済みユーザー]

ご回答ありがとうございます。
Maxの条件に一致するものをFindで見つけるという書き方ができるのですね。
ご教示いただきありがとうございます。

Answer 2

一番シンプルに書くならAggregate<T,T,T>だと思います。
Listの要素にIComparable<T>を実装すれば引数無しのMaxが使えますが、FileInfoではそのままでは使えませんね。

filelist.Aggregate((result, next) => result.Length > next.Length ? result : next);

追記
無理矢理FileInfoにIComparableを実装するならこんな感じでしょうか。
自作クラスなら自然な形で実装できるのですが・・・

余談ですが、FileInfoに拡張メソッドでCompareToを定義してみましたがやはりダメでした。

filelist.Select(p => new FileInfoWrapper(p)).Max().FileInfo;

class FileInfoWrapper : IComparable<FileInfoWrapper>
{
    public FileInfo FileInfo { get; }
    public FileInfoWrapper(FileInfo fileInfo) => FileInfo = fileInfo;

    public int CompareTo(FileInfoWrapper other)
    {
        if (FileInfo.Length == other.FileInfo.Length)
        {
            return 0;
        }
        else if (FileInfo.Length > other.FileInfo.Length)
        {
            return 1;
        }
        else
        {
            return -1;
        }
    }
}

Comments

[退会済みユーザー]

ご回答ありがとうございます。
Listの要素にIComparable<T>を実装するやり方も気になっていたので、ご教示いただけませんか？

[YamakawaJunichi]

IComparable<T>実装パターンを追記しました。
FileInfoがsealedなためにこのような形になっています。

[退会済みユーザー]

ご回答ありがとうございます。
勉強になりました。
ありがとうございます。