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を使って簡単にできるかと思っていたのですが、そのような方法はなさそうなんですね。
ありがとうございます。

[退会済みユーザー]

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なためにこのような形になっています。

[退会済みユーザー]

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

Answer 3

LINQで最大値ではなく、最大値を持つ要素を取得したいです。

.NET6以降MaxByが追加されたので、非常に簡単になりました^^
Enumerable.MaxBy メソッド (System.Linq) | Microsoft Learn

また、該当のソースコードのforeach文をLINQで書き換えることは可能ですか？

わざわざリストに入れる必要はないですね。
EnumerateFilesから、そのままLinqに繋げればいいでしょう。
DirectoryInfo.EnumerateFiles メソッド (System.IO) | Microsoft Learn

---

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

こちらも.NET6以降MaxにIComparer<T>を渡すオーバーロードが追加されています。
Enumerable.Max メソッド (System.Linq) | Microsoft Learn

//var path = Path.Combine(Environment.GetFolderPath(Environment.SpecialFolder.DesktopDirectory), "testfolder");
var path = Environment.GetFolderPath(Environment.SpecialFolder.DesktopDirectory);

var maxBy = new DirectoryInfo(path)
    .EnumerateFiles()
    .MaxBy(s => s.Length);

Console.WriteLine(maxBy?.Name ?? "empty files");


var max = new DirectoryInfo(path)
    .EnumerateFiles()
    .Max(new FileInfoComparer());

Console.WriteLine(max?.Name ?? "empty files");

class FileInfoComparer : Comparer<FileInfo>
{
    public override int Compare(FileInfo? x, FileInfo? y)
        => Comparer<long?>.Default.Compare(x?.Length, y?.Length);
}

---

最大値の要素が複数あった場合、MaxByは要素ひとつしか返しません。
すべて列挙したいならMaxByは使えません。

一度Maxを求めてWhereするとか、GroupByしてMaxByするとかひと工夫必要です。
面倒だったらMoreLINQのMaximaを使うのも手です。

using static MoreLinq.Extensions.MaximaExtension;

int[] array = [-5, 3, -1, 5];
Console.WriteLine(array.MaxBy(Math.Abs)); // -5

var absMax = array.Max(Math.Abs);
Console.WriteLine(string.Join(", ", array.Where(x => Math.Abs(x) == absMax))); // -5, 5
Console.WriteLine(string.Join(", ", array.GroupBy(Math.Abs).MaxBy(g => g.Key)!)); // -5, 5
Console.WriteLine(string.Join(", ", array.Maxima(Math.Abs))); // -5, 5

LINQ で最小値・最大値を柔軟に求める
c# - Linq MaxBy with all elements? - Stack Overflow
c# - MaxBy() is there a way to get multiple max values? - Stack Overflow

Comments

[dodox86]

ご回答の意図から外れたコメントかもしれませんが、
> var maxBy = new DirectoryInfo(path)
> .EnumerateFiles()
> .MaxBy(s => s.Length);

だけだと、ファイルサイズMaxのファイルが複数存在しても1つしか取得しないようです。当初の質問投稿時の意図から推察すると、恐らくそれでは困るかと。

外したコメントであったらすみません。[一部修正済み]

[TN8001]

> だけだと、ファイルサイズMaxのファイルが複数存在しても1つしか取得しないようです。当初の質問投稿時の意図から推察すると、恐らくそれでは困るかと。

LINQのMaxに言及している時点で「最大の１個でいい」と思い込んでいましたが、要素（FileInfo）が欲しいなら全部出たほうがいいに決まってますよね^^;
タイトル的にもそこは回収しておくべきでしたね（でもあんまキレイな方法ないんよなw

補足ありがとうございます^^