C# Unity 敵の撃破リストの検索が上手くいくようにしたい

Question

C#　複合型のリストを検索したい
・Unity にてゲームで使う敵の撃破リストを
・リストの中身「 Int・String・Int 」で構成
・リストは外部から読み取り可能・関数を使う場合のみ色々とリストの変更が可能
→ 他の質問の方から流用・アレンジさせてもらってます

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

// リスト管理 クラス
public class EnemyKillList : MonoBehaviour
{
    // 外部から読み取り専用のリスト
    // リスト内データの「更新・追加・削除」は専用の関数でなければアクセス出来ない
    [SerializeField]
    public static IReadOnlyList<IReadOnlyRecordData> Readlist => Recodlist;

    // privateリスト <static>が付いているのは上の読み取り専用を<static>にする為
    [SerializeField]
    private static List<RecordData> Recodlist = new List<RecordData>();

    // リストデータ追加 関数
    // 引数からリスト型 変換
    // その後リスト 追加
    public static void AddData(int OrderNumber, string EnemyName, int KillCount)
    {
        var rd = new RecordData();
        rd.OrderNumber = OrderNumber;
        rd.EnemyName = EnemyName;
        rd.KillCount = KillCount;
        Recodlist.Add(rd);
    }

　　///　問題箇所A　///
    // リスト内データ<撃破数>更新 関数
    public static void UpdateData(int Number, string Name, int Count)
    {
        var Findlist = new RecordData { OrderNumber = Number, EnemyName = Name };
        var Updatelist = new RecordData { OrderNumber = Number, EnemyName = Name, KillCount = Count };

        // リスト内のデータ 検索
        if (Recodlist.Contains(Findlist))　//　← ここのリスト内の検索方法が上手くいきません
        {
            // データがリストの何番目かを検索　変数に代入
            int IndexNum = Recodlist.IndexOf(Findlist);

            // データの撃破数を置き換える
            Recodlist[IndexNum].KillCount = Updatelist.KillCount;
        }
    }

　　///　問題箇所B　///
    // リスト内データ削除 関数
    public static void RemoveData(int Number, string Name, int Count)
    {
        var Removelist = new RecordData { OrderNumber = Number, EnemyName = Name, KillCount = Count };

        // リスト内に引数と同じデータがあれば削除
        if (Recodlist.Contains())
        {
            Recodlist.Remove(Removelist);
        }

        // 無い場合は処理終了
        else
        {
            return;
        }
    }
}

// 読み取り専用インターフェイス クラス
public interface IReadOnlyRecordData
{
    int OrderNumber { get; }
    string EnemyName { get; }
    int KillCount { get; }
}

// リスト内部・読み取り専用 クラス
public class RecordData : IReadOnlyRecordData
{
    public int OrderNumber { get; set; }
    public string EnemyName { get; set; }
    public int KillCount { get; set; }
}

問題点
・問題箇所のリスト内検索が上手くいかない
・問題箇所Aの部分では「OrderNumber・EnemyName」で検索をかけ該当するものがある際には「KillCount(撃破数)」を加算・上書きする
・問題箇所Bの部分では「OrderNumber・EnemyName・KillCount」で検索をかけ該当するものがある際にそのリストを削除する
・検索する型が間違っていると考えていますが未だ解決に至っていません

未だ未熟なのでコードに不備などあるかもしれませんかその際は適宜指摘してもらえるとありがたいです

Comments

[ayousanz]

ちょっと記憶が確かではないのですが，classの入ったListを検索するときはいい感じにしないと一致系のメゾットがうまく動作しない気がします．一度for文で回してOrderNumberとEnemyNameがどちらとも一致している場合という風に書き換えて試してもらってもいいでしょうか？

仮にfor文ですべてif判定を行ってできるのであれば，classのFindあたりを修正したらAはうまくいくと思います．(Bも同様)

[YAmaGNZ]

1570pさんの回答に対して「参考にしながら実装できました。ありがとうございます！」と返しておきながら「まだ回答を求めています」というのはどういうことなのでしょうか？

[Koma2000]

すみません朝寝ぼけていたみたいです　押した記憶はなかったのですが...
ご指摘ありがとうございます。

[YAmaGNZ]

解決されたのであれば、ベストアンサーを決めてあげてはどうでしょうか。

Answer 1

クラスインスタンス同士の等値比較は暗黙的に RefarenceEqual（参照に対する比較）として扱われるため、質問のRecodlist.Contains(Findlist) は常にfalseを返します。

等値比較をクラス内変数の等値性によるものにしたい場合は、RecordDataにIEquatable<T>を実装します。詳しくは以下のリンクをご参照ください。

IEquatableを完全に理解する
https://qiita.com/cactuaroid/items/f277a097ecf51eb247c0

---

追記 (2021/04/27)

参考コード書きました。リンク先のページ上部にあるRunを押すと実際に試せます。
https://dotnetfiddle.net/g0rAUT

Comments

[Koma2000]

ご回答ありがとうございます
回答頂いたものを実践したいと思います

[Koma2000]

せっかく回答を頂いておいて申し訳ないですが
リンク先の記事や自分でも調べてみても実装が出来ませんでした
すみません

Answer 2

検索がうまくいかないというFindListの部分はnewされて新しく作られたRecoadDataのインスタンスですので、当然RecodList内に存在しません。全く同じ値を持っているRecoadDataだとしてもそれは異なる存在です。

List内からNumberとNameが同じものを探したいというのなら、
手っ取り早くLinqを使用(using System.Linq;を追加)、

// RecodListの中からNumberとNameが同じ値のRecoadDataの最初に見つけた参照を得る、見つけられなければnull、同一の値があるようなListの場合、最初に見つけた物となるためこれもバグとなりえる
var data = RecodList.FirstOrDefault(_ => _.OrderNumber == Number && _.EnemyName == Name);

if (data != null) data.killCount = Count;
else // リスト内に存在しなかった場合の処理

みたいな感じか？

RecodList.Contains(～
として使いたい場合、
public static void UpdateData(RecoadData data)
として、RecoadDataそのものを引数とする？
RecodList.Contains(data)

他、Bの問題個所も上記と同じ理由でnewで生成したものと比較してもList内には存在しないため、常にfalseが返ってくる。

Comments

[Koma2000]

明確な問題点の解説ありがとうございます勉強になりました
コメントを参考に試してみます

[Koma2000]

参考にしながら実装できました
ありがとうございます！