[Unity][C#][Linq]ゲーム中にプレイヤー達のそれぞれのスコア(キル数)をランキング形式で表示したい

Question

内容

Unityにて、ゲーム中にプレイヤーたちの「名前」「キル数」を降順でランキング形式で表示したいです。
※イメージ↓

コード↓(こちらを参考にさせていただきました)

using System.Collections;
using System.Collections.Generic;
using System;
using UnityEngine;
using UnityEngine.UI;
using System.Linq;

public class Scores{
    public string Name { get; set;}
    public int Kill { get; set;}
}

public class Scoreboard : MonoBehaviour
{
    public List<string> namesList = new List<string>(); //後でほかのオブジェクトからリストに追加されていく
    public Text playernames_text;   //名前テキスト
    public Text playerkills_text;   //キル数テキスト
    public Text playerRank_text;    //ランクテキスト

    void Update(){
        var scores = new Scores[]{
        };
        foreach(string name in namesList){
            new Scores()
            {
                Name = name,
                Kill = 0,
            };
        }
        //↑ダメ元でやってみたけどやっぱり無理だった。(そもそも表示されない)
        var ranking = from s1 in scores
                          let higher = from s2 in scores  // 現在の生徒より点数が良いプレイヤーを取得
                                       where s2.Kill > s1.Kill
                                       select s2
                          select new
                          {
                              s1.Name,
                              s1.Kill,
                              Rank = higher.Count() + 1, // 現在の生徒より点数が良い生徒の人数 + 1 が順位
                          } into s3
                          orderby s3.Rank  // 順位で並び替え
                          select s3;

            foreach(var scoreboard in ranking)
            {
                playerRank_text.text = scoreboard.Rank +"\n"; //改行
                playernames_text.text = scoreboard.Name +"\n"; //改行
                playerkills_text.text = scoreboard.Kill +"\n"; //改行
            }
        }
}

using System;
using System.Collections;
using System.Collections.Generic;
using UnityEngine;

public class Player : MonoBehaviour
{
    public string name;
    public int kill = 0;                                        //キル数

    public Scoreboard scoreboard_script;

    void Start ()
    {
        //スコア
            name = "a";       //とりあえず"a"とする
            scoreboard_script.namesList.Add(name); //Scoreboard.csのnamesListに追加する
    }
}

今日「Linq」というものを知ったため、そもそもの書き方が間違っているかもしれません。
参考にしたサイトのように書くと以下↓のコードになりますが、これだと後でプレイヤーが増減したときに変更できないし...
※ちなみにUnityには表示されました(おかしいけど)

        var scores = new Scores[]{
            new Scores(){
                Name = "a",
                Kill = 0,
            },
            new Scores()
            {
                Name = "b",
                Kill = 1,
            },
        };

どうすればうまく動作するのか、ご教授頂きたいです。

もし、ほかに実装できる方法があれば、それも合わせてご教授頂きたいです。

※抜けている箇所などがあれば修正または追記しますので、教えていただけるとありがたいです。

追記１

Player.csを追加しました。

追記２

"「長さが0の配列」になっている"との指摘を頂いたため、コードを書き換えたのですが、エラーが発生してしまいました。

        var scores = new Scores[namesList.Count];  //配列の長さを名前リストと同じ長さにする
        scores = namesList.ToArray();                    //リストから配列に代入
        //error CS0029: Cannot implicitly convert type 'string[]' to 'Scores[]'
        foreach(var name in namesList){
            new Scores()
            {
                Name = name,
                Kill = 0,
            };
        }

エラー文

error CS0029: Cannot implicitly convert type 'string[]' to 'Scores[]'

"stringのリストは変換できません"みたいなことが書かれていますが、ではどうしたらいいのかさっぱりわかりません...
こちらのエラーについても、解決いただきたいです。

追記３

fiveHundredさんに教えてもらったようにFor文を使って追加するように書き換えましたが、For文の中でそれぞれのプレイヤーの名前は取得できているものの、うまくScoresに代入できていないようでして...
今一度解説をお願いしたいです。たくさんの追記申し訳ないです...

        var scores = new Scores[namesList.Count];
        for (int i = 0; i < namesList.Count; i++){
            new Scores(){
                Name = namesList[i],
                Kill = 0,
            };
        };

実行したときのエラー文

NullReferenceException: Object reference not set to an instance of an object
Scoreboard+<>c.<FixedUpdate>b__6_1 (<>f__AnonymousType0`2[<s1>j__TPar,<higher>j__TPar] <>h__TransparentIdentifier0) (at Scoreboard.cs:69)

System.Linq.Utilities+<>c__DisplayClass2_0`3[TSource,TMiddle,TResult].<CombineSelectors>b__0 (TSource x) (at <351e49e2a5bf4fd6beabb458ce2255f3>:0)
System.Linq.Enumerable+SelectArrayIterator`2[TSource,TResult].ToArray () (at <351e49e2a5bf4fd6beabb458ce2255f3>:0)
System.Linq.Buffer`1[TElement]..ctor (System.Collections.Generic.IEnumerable`1[T] source) (at <351e49e2a5bf4fd6beabb458ce2255f3>:0)
System.Linq.OrderedEnumerable`1+<GetEnumerator>d__3[TElement].MoveNext () (at <351e49e2a5bf4fd6beabb458ce2255f3>:0)
Scoreboard.Update () (at Scoreboard.cs:78)

//以下エラーの場所
        var ranking = from s1 in scores
                          let higher = from s2 in scores  // 現在のプレイヤーより点数が良いプレイヤーを取得
                                       where s2.Kill > s1.Kill
                                       select s2
                          select new  //NullReferenceException:～
                          {
                              s1.Name,
                              s1.Kill,
                              Rank = higher.Count() + 1, // 現在のプレイヤーより点数が良いプレイヤーの人数 + 1 が順位
                          } into s3
                          orderby s3.Rank  // 順位で並び替え
                          select s3;

追記４(解決しました！！！)

fiveHundredさんに指摘された通り、代入がされていませんでしたが、追記のコード通りに書いたところ、見事に上手く動作しました！

↓早速作ったBotを入れて試してみました！

↓プレイヤーがBotをキルすると...

キル数が多い順にソートされた！！！(ちょっと不格好ですが...)

最後に

今回の不具合の原因は完全に私が名前を覚えたばっかりの「Linq」を全く学ばずにUnityに実装しようとしたことが原因です。
次回から新しいものに手を出す場合は、少しは基礎知識をつけたうえでコードを書いていこうと思います。

回答してくれた方々、どうもありがとうございました！

Comments

[退会済みユーザー]

> 参考にしたサイトのように書くと以下↓のコードになりますが、これだと後でプレイヤーが増減したときに変更できないし...

ハードコーディングしないで済む方法（何らかのデータから動的に作成する方法）はあると思いますので　それを考えれば良いのでは？　「//後でほかのオブジェクトからリストに追加されていく」とか書いてありますが、それ以上のことは第三者には分からないので、どう実装するか質問者さんが考える他なさそうです。

[omaetoomae]

ご指摘ありがとうございます。

[退会済みユーザー]

その返事ではどうするのか分かりませんが？

「//後でほかのオブジェクトからリストに追加されていく」と書かれても第三者には分からないので、元データ (from s1 in scores の scores のこと) の具体的サンプルを示してもらわないと話が進まないのですよ。

Answer 1

質問のコメントに書きましたが、「//後でほかのオブジェクトからリストに追加されていく」と書かれても第三者には分からないので、元データ (from s1 in scores の scores のこと) の具体的サンプルを示してもらわないと話が進まないのですよ。

「追記２」で Searchboard.cs を追記してそこに配列 Scores[] を作成するコードを書いたようですが、元になる nameList の中身が不明だし、Kill = 0 に固定したのでは並べ替えのサンプルとして不適です。

例えば、質問の画像のように Name を "a" ～ "j" としてそれから nameList を作り、それをベースに from s1 in scores の scores を生成し、生成する際各 Name の Kill には Name 毎に異なる値を入れておくというようなサンプルを作らないと話が進みません。

それから、scores を Kill で降順に並べ替えて連番を振るという Linq 式ですが、参考にしてるサイトの記事のやり方は間違ってはいないかもしれませんが感心しません。

IEnumerable<T> インターフェイスの Select 拡張メソッドのオーバーロードの一つに Select<TSource,TResult>(IEnumerable<TSource>, Func<TSource,Int32,TResult>) があって、それを使えば 0 番から始まる連番の index を取得することができます。具体例は以下の記事を見てください。

Entity Framework で ROW_NUMBER
http://surferonwww.info/BlogEngine/post/2021/09/20/row-number-in-entity-framework.aspx

上記を考慮に入れたサンプルコードを載せておきます。

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

namespace ConsoleAppLinq
{
    public class Scores
    {
        public string Name { get; set; }
        public int Kill { get; set; }
    }

    internal class Program
    {
        static void Main(string[] args)
        {
            // 「後でほかのオブジェクトからリストに追加されていく」そうだがそれをここでやる
            List<string> namesList = new List<string>
            {
                "a", "b", "c", "d", "e", "f", "g", "h", "i", "j"
            };

            // namesList から Scores クラスのリストを作る。配列にせず List<Scores> とした。
            // Kill のデータはどこから持ってくるのか不明だが、値を入れないと話ができない
            // のでサンプルとしてランダムな値を入れておく
            var scores = new List<Scores>();
            var rand = new Random();
            foreach (string name in namesList)
            {
                var score = new Scores
                {
                    Name = name,
                    Kill = rand.Next(1, 100)
                };
                scores.Add(score);
            }

            // scores を Kill で降順に並べ替えて連番を振る。結果は匿名クラスのリストに格納 
            var sortedList = scores
               .OrderByDescending(s => s.Kill)
               .Select((a, index) => new 
               {
                   Rank = index + 1,
                   Name = a.Name,
                   Kill = a.Kill
               });

            // 結果の表示
            foreach (var item in sortedList)
            {
                Console.WriteLine($"Rank: {item.Rank}, Name: {item.Name}, Kill: {item.Kill}");
            }
            
        }
    }
}

結果は以下の通り。Kill = rand.Next(1, 100) で値を入れているので結果はアプリの実行のたび違います。

Comments

[omaetoomae]

回答ありがとうございます。
無事解決しました。

次回からはもっと質問内容を分かりやすくまとめて質問しようと思います。
何度もご指摘していただきありがとうございました。

Answer 2

        var scores = new Scores[]{
        };
        foreach(string name in namesList){
            new Scores()
            {
                Name = name,
                Kill = 0,
            };
        }

上記でScoresを生成しておきながら、どこにも代入されていません。
よって、配列が空のままなので、動かないのは当然です。

また、scoresもこのままだと「長さが0の配列」になっております。
配列は長さを途中で変更することはできません。
初期化時に長さを指定するか、Listを使ってください。

---

追記：

        scores = namesList.ToArray();                    //リストから配列に代入

上記ではscoresはScoresの配列なのに対し、namesList.ToArray()はstringの配列です。
したがって、scoresに代入できないのは当然です。

そして、

        foreach(var name in namesList){
            new Scores()
            {
                Name = name,
                Kill = 0,
            };
        }

上記、「どこにも代入されていません」って言っているのに全く改善されていません。

配列でやる場合には、for文でループさせ、scores[～]に（各要素ごとに）Scoresを代入させます。
リストであれば、～.Add(～);でリストに追加させます。

---

追記２：

まだ出来ていません。
new Scores()自体がどこにも代入されていないです。

以下のようにする必要があります。

        var scores = new Scores[namesList.Count];
        for (int i = 0; i < namesList.Count; i++){
            scores[i] = new Scores(){ // ←scoresに代入
                Name = namesList[i],
                Kill = 0,
            };
        };

Comments

[omaetoomae]

回答ありがとうございます。
追記したように、回答者さんの指摘されたようにコードを変更してみましたが、別の不具合が発生してしまいました。一度目を通していただけるとありがたいです。

[fiveHundred]

回答に追記しましたので、確認をお願いします。

[omaetoomae]

回答者さんに教えてもらったようにFor文にてScoresクラスに代入を試みましたが、For文の中でそれぞれのプレイヤーの名前は取得できているものの、うまくScoresクラスに代入ができていないようでして...
内容は追記しましたが、今一度説明をしていただきたいです。分からず屋で申し訳ありません...

[fiveHundred]

再度追記しましたので、確認をお願いします。

[omaetoomae]

何度も回答ありがとうございます。
無事解決しました！

私がまだ「Linq」を全く知らずにUnityに実装したがために回答者さんに何度も追記をしていただくことになってしまい誠に申し訳ありませんでした。
本当に、お世話になりました！