C# XMLファイルの特定要素の最大値

Question

前提・実現したいこと

XMLファイルをインポートし、teammembers要素の数値が最大の要素を抽出し、コンソール画面に表示させたいのですが、エラーが出てしまい、実行中に例外が発生しました。
XMLファイル内に、ballsport要素が３つあり、そのそれぞれに、teammembers要素が含まれています。
やりたいことは、teammembers要素の数値が最大のballsport要素を抽出し、name要素とteammembers要素をコンソール画面に表示することです。
Maxメソッドがあるので、それを使って上手くクラスやディクショナリークラスを経ずにXDocumentクラスやXElementクラスを使って上記操作を実現したいです。
よろしくお願いします。

発生している問題・エラーメッセージ

System.NullReferenceException: 'Object reference not set to an instance of an object.'

MaxMembers が null でした。

該当のソースコード

        static void Main(string[] args)
        {
            var xdoc = XDocument.Load(@"C:\Users\lifeg\Desktop\ballsports.xml");

            var MaxMembers = xdoc.Root.Elements("teammembers").Max();

            Console.WriteLine("{0} {1}",MaxMembers.Element("name").Value,MaxMembers.Element("teammembers").Value);
        }

<?xml version="1.0" encoding="UTF-8"?>
<ballSports>
 <ballsport>
  <name kanji="籠球">バスケットボール</name>
  <teammembers>5</teammembers>
  <firstplayed>1891</firstplayed>
 </ballsport>
 <ballsport>
  <name kanji="排球">バレーボール</name>
  <teammembers>6</teammembers>
  <firstplayed>1895</firstplayed>
 </ballsport>
 <ballsport>
  <name kanji="野球">ベースボール</name>
  <teammembers>9</teammembers>
  <firstplayed>1846</firstplayed>
 </ballsport>
</ballSports>

試したこと

匿名クラスに一度格納してから、並べ替えることはできましたが、XML自体がすでに構造化されているので、なんとかXDocumentクラスやXElementクラスで表示する方法を模索しています。

補足情報（FW/ツールのバージョンなど）

Microsoft Visual Studio Community 2019
Version 16.4.2

Comments

[TN8001]

最大の要素が複数個あった場合はどうしますか？

[uppi]

追記ありがとうございます。
その場合は、最初に見つかった要素だけとかではなく、すべてコンソールに画面に表示したいです。

Answer 1

var maxVal = xdoc.Descendants("teamnumber").Select(e=>int.Parse(e.Value)).Max().ToString();
xdoc.Descendants("ballsports").Where(e=>e.Element("teamnumber").Value == maxVal).ToList().ForEach(e=>Console.WriteLine(e));

Comments

[papinianus]

ForEachがListだったきがする、を他の回答みて思った。ぱくりました

[TN8001]

あぁなるほど別々に探せばいいんですね^^;

var maxVal = xdoc.Descendants("teammembers").Select(e => int.Parse(e.Value)).Max().ToString();

xdoc.Descendants("ballsport").Where(e => e.Element("teammembers").Value == maxVal).ToList().ForEach(e => Console.WriteLine("{0} {1}", e.Element("name").Value, e.Element("teammembers").Value));

[papinianus]

他にはOrderByDescendingして、出力時に前の要素と比較して最大値である間出力とかですかね。
必ずteamnumberの種類が2種類あることが保証されるならOrderByDeacendingしたものをOrderByDescendingして1つSkipしたものをZipすればスコープ外の変数を参照しなくて済みそうですがトリッキーすぎるかと思いました。

[uppi]

papinianusさん
回答ありがとうございます。
シンプルかつ分かりやすいコードが気に入りました。
ベストアンサーにさせていただきます。
私も早く、このようなコードが、さささっと書けるようになりたいです。

[uppi]

上記回答を参考にして下記の通りコードが完成しました。
自分への備忘録を兼ねてここに掲載します。
var xdoc = XDocument.Load(filepath);

var MaxNumber = xdoc.Descendants("teammembers").Select(e => int.Parse(e.Value)).Max().ToString();
var MaxMembers= xdoc.Root.Elements().Where(e => e.Element("teammembers").Value == MaxNumber);

foreach (var item in MaxMembers)
{
Console.WriteLine("{0} {1}", item.Element("name").Value, item.Element("teammembers").Value);
}

Answer 2

大げさすぎる気がしますが、本体部分はすっきりしていますよね？ｗ
LINQを駆使すれば拡張メソッド部分なしで何とかなるとは思いますが、私は苦手なのでわかりません^^;

using System;
using System.Collections.Generic;
using System.Linq;
using System.Xml.Linq;

namespace Questions233905
{
    class Program
    {
        static void Main()
        {
            var xdoc = XDocument.Load(@"ballsports.xml");
            var MaxMembers = xdoc.Root.Elements().MaxBy(x => int.Parse(x.Element("teammembers").Value));
            MaxMembers.ToList().ForEach(x => Console.WriteLine("{0} {1}", x.Element("name").Value, x.Element("teammembers").Value));
        }
    }

    // https://qiita.com/Zuishin/items/95e171eccc128bdd6429
    static class IEnumerableExtensions
    {
        //public static IEnumerable<T> MinBy<T, U>(this IEnumerable<T> source, Func<T, U> selector) 
        //    => SelectBy(source, selector, (a, b) => Comparer<U>.Default.Compare(a, b) < 0);
        public static IEnumerable<T> MaxBy<T, U>(this IEnumerable<T> source, Func<T, U> selector)
            => SelectBy(source, selector, (a, b) => Comparer<U>.Default.Compare(a, b) > 0);
        private static IEnumerable<T> SelectBy<T, U>(IEnumerable<T> source, Func<T, U> selector, Func<U, U, bool> comparer)
        {
            var list = new LinkedList<T>();
            var prevKey = default(U);
            foreach(var item in source)
            {
                var key = selector(item);
                if(list.Count == 0 || comparer(key, prevKey))
                {
                    list.Clear();
                    list.AddLast(item);
                    prevKey = key;
                }
                else if(Comparer<U>.Default.Compare(key, prevKey) == 0)
                {
                    list.AddLast(item);
                }
            }
            return list;
        }
    }
}

Comments

[uppi]

TN8001さん
回答ありがとうございます。
特にMaxbyメソッドについては知らなかったので参考になりました。
やはり、Linqやラムダ式は便利だとはおもうのですが、難しいですね。