C#における、クラス内での複数のメンバ変数を比較したソートについて

Question

C#でのクラス配列のソートで悩んでいます。

using System;
using System.Collections.Generic;

namespace ProgramCS
{
	class MainClass
	{
		public class PlayerData
		{
			public int mAttack;
			public int mDefence;
			public int mMagic;
		}

		public static void Main(string[] args)
		{
			int playerTotal = int.Parse(Console.ReadLine());

			List<PlayerData> player = new List<PlayerData>(playerTotal);

			for (int i = 0; i < playerTotal; i++)
			{
				player.Add(new PlayerData());
			}

			//各プレイヤーの能力値の割り当て
			for (int i = 0; i < playerTotal; i++)
			{
				var n = Console.ReadLine();
				string[] str = n.Split(' ');

				player[i].mAttack = int.Parse(str[0]);
				player[i].mDefence = int.Parse(str[1]);
				player[i].mMagic = int.Parse(str[2]);
				n = string.Empty;
			}

			player.Sort((x, y) =>
			{
				return y.mAttack.CompareTo(x.mAttack);
			});

			foreach (PlayerData a in player)
			{
				Console.WriteLine("{0} {1} {2}", a.mAttack, a.mDefence, a.mMagic);
			}
		}
	}
}

上記コードにて、各プレイヤーのAttackの数値が最も大きい者から順に出力出来る様にしました。ですが、最終的には、
・Attackが同値だったら次はDefenceで比較
・Defenceも同値だったら最後はMagicで比較
という風に、Attack > Defence > Magicの優先順位で比較したい場合、Sort関数等を使用してコンパクトにコードをまとめる事が出来るのでしょうか？
それとも、原始的なやり方(for文で比較し値を置き換える)でしか不可能でしょうか？
もしご存知でしたらアドバイス頂ければと思います。

Answer 1

LINQ を使うと簡単です。

player.Sort((x, y) =>
{
    return y.mAttack.CompareTo(x.mAttack);
});

foreach (PlayerData a in player)
{
    Console.WriteLine("{0} {1} {2}", a.mAttack, a.mDefence, a.mMagic);
}

ここを↓に

var sortedPlayers = player
   .OrderByDescending(p => p.mAttack)
   .ThenByDescending(p => p.mDefence)
   .ThenByDescending(p => p.mMagic);
            
foreach (var a in sortedPlayers)
{
    Console.WriteLine("{0} {1} {2}", a.mAttack, a.mDefence, a.mMagic);
}

Comments

[finalfight]

Linqの使用は初めてでしたが、仰られた通りに修正し実行した結果、こちらも望む結果が得られました。Linqを使うとコードがかなり分かり易くなり、非常に便利だと実感します。
大変勉強になりました。どうもありがとうございます。
一番最初に投稿して頂きましたので、ベストアンサーとさせて頂きます。

[hihijiji]

>一番最初に投稿して頂きましたので、ベストアンサーとさせて頂きます。
一番最初は、tkturboさんですよ。
その理由なら、BAはtkturboさんへ付け替えてくだされたし

[tkturbo]

私的には、こちらの方がスマートな書き方なのでこちらをBAにした方がよい、に一票。

Answer 2

player.Sort((x, y) =>
{
  int compare = y.mAttack.CompareTo(x.mAttack);
  if (compare != 0 )
  {
    return compare;
  }
  compare = y.mDefence.CompareTo(x.mDefence);
  if (compare != 0 )
  {
    return compare;
  }
  return y.mMagic.CompareTo(x.mMagic);
});

こんな感じですかね。

Comments

[finalfight]

Sort関数をそのまま使う事が出来て安心しました。変数を用意するアイディアが現時点では思い浮かべなかったので、大変助かります。どうもありがとうございます。

Answer 3

比較に用いるlambda式の部分を次にようにすればよいと思います。

(x, y) => {
  var c = y.mAttack.CompareTo(x.mAttack);
  if (c != 0) return c;
  c = y.mDefence.CompareTo(x.mDefence);
  if (c != 0) return c;
  return y.mMagic.CompareTo(x.mMagic);
}

もしこの比較がPlayerDataを比較する主たる比較方法になるのでしたら以下のようにIComparable<T>として定義してやるとスッキリすると思います。

public class PlayerData : IComparable<PlayerData> {
  public int mAttack;
  public int mDefence;
  public int mMagic;

  public int CompareTo(PlayerData that) {
    if (that == null) return 1;
    var c = that.mAttack.CompareTo(mAttack);
    if (c != 0) return c;
    ...
  }
}
...
List<PlayerData> players = ...;
players.Sort();

Comments

[finalfight]

修正して実行した結果、望む結果が得られました。
compare変数を用意して、compare != 0 比較変数同士が等しくなければreturnするというやり方を知れて、大変嬉しいです。また、続けてアドバイス頂いたコードについても導入し、勉強してみます。

一番最初にご回答して頂いたので、ベストアンサーとさせて頂きます。どうもありがとうございました。

[KSwordOfHaste]

えーと、tkturboさんの回答の方が早いのですが・・・ＢＡは付け替えていただいて結構ですよ！
hihijijiさん回答もスッキリしてますね・・・

[finalfight]

KSwordOfHasteさんの回答が一番先頭に来ていたので、勘違いしておりました。
...BAの変更について非常に悩んだ結果、KSwordOfHasteさん自ら仰って頂いたので、大変恐れ入りますが、ご変更する事に致します。本当に申し訳無いです。
改めて、コードについてのアドバイスを頂きありがとうございました。

[KSwordOfHaste]

> 本当に申し訳
no problemです～