【Unity】2つのListの要素が一致しているか調べる方法

Question

前提

Unityで、ホテル用ベッドメイキングの
チェックリストアプリを作っています。
部屋ごとにベッドが複数あり、チェックを押したら、
どの部屋のどの番号のベッドがチェックされたのかを
Listに保管しています。
このListはクラスのListで、int型の変数を２つ持っています。
（部屋番号とベッド番号）

実現したいこと

あるボタン（Closeボタン）を押したときに、
すべての部屋のベッドがチェックされたかどうかを確認したいです。

すべてのベッドのリストと、チェックされたベッドのリストを比較し、
同じ要素があればそれをさらに別のリストに入れ、
その総数がベッドの総数と一致しているかを調べたいです。
（もし一致していなかったら、お知らせが出ます）

発生している問題

FindAllというメソッドを使って、
共通の値を持ったクラスを見つけたいと思ったのですが、
どのように記述すればうまくいくのかわかりません。

該当のソースコード

public class BedPanelCon : MonoBehaviour
{

   //チェックされたベッド
    public List<BedData> bedDataList;
   
   //ホテルのすべてのベッド
    public List<OriginalBedData> originalBeddataList;

   //
    public List<CloseBedCheck> closeBedCheckList;


 public void Start()
 {  
　　bedDataList = new List<BedData>();
      originalBeddataList = new List<OriginalBedData>();
      closeBedCheckList = new List<CloseBedCheck>();

//…中略

public void PushBedButtonMakinglist(int roomNO, int BedNo)
    {
       BedData b = new BedData();
       b.RoomNo = roomNO;
       b.BedNo = BedNo;
       bedDataList.Add(b);
    }

public void MakingOriginalBedDatalist(int roomNO, int BedNo)
    {
        OriginalBedData d = new OriginalBedData();
        d.RoomNo = roomNO;
        d.BedNo = BedNo;
        originalBeddataList.Add(d);
    }
   
[System.Serializable]
public class BedData
{
    public int RoomNo;
    public int BedNo;
}

[System.Serializable]
public class OriginalBedData
{
    public int RoomNo;
    public int BedNo;
}

//以下のボタンを押したとき、未チェックのベッドがあるか確認する処理をしたいです。
public void PushClose()
    {
　　　//以下の書き方だと、エラーが出ます。
        closeBedCheckList = bedDataList.FindAll(originalBeddataList.Contains);

//…中略
        
    }

エラー文を読んでもまだ理解できる力がありません。
（エラー文：メソッドグループからPredicate<BedData>へ変換することはできません）
また、ひとつの変数ではなく、Listに格納されたクラスの中の変数を調べるには
どのようにしたらよいでしょうか。

おわかりの方いましたら、ご教授頂ければ幸いです。

Comments

[ozwk]

BedDataとOriginalBedDataでクラスが分かれている意味はなんですか?

[Zuishin]

Equals をオーバーライドして他のオブジェクトと同じ値かどうかを自己言及させるとロジックが簡単になります。
あるいは「部屋番号-ベッド番号」という文字列に変換すると HashSet が使えるようになります。

> メソッドグループからPredicate<BedData>へ変換することはできません

これは関数呼び出しのカッコを忘れた時に出るメッセージです。

[Matsuri_]

ozwkさん
BedDataとOriginalBedDataでクラスが分かれている意味はなんですか?
への回答になります。

まず、個々のリストに何を記録しているかですが、
BedDataはチェックされたベッド（ベッドメイクが終わったベッドのリスト）で、
OriginalBedDataはホテルにあるすべてのベッドのリスト（チェックされたかどうかは関係ないです）
になります。

クラスが分かれている意味ですが、
以下の処理を実行するために、それぞれ分けなければいけないと思っていました。

１.ベッドメイクが終わったら、ベッドのボタンを押してチェックを入れる。
２.BedDataにそのベッドのルームナンバーとベッドナンバーが記録される。
３.Closeボタン（終了ボタン）を押したとき、OriginalBedDataに記録されているすべてのホテルのベッドと、BedDataに記録されたホテルのベッドが一致しているかどうかを確認する。

そもそも、上記の方法（ルームナンバーとベッドナンバーの変数を持ったクラスを作っていく）で正しい処理なのかどうかわかりません。
もっといい方法があれば、教えて頂けると幸いです。
よろしくお願い致します。

[Zuishin]

正しいか正しくないかで言えば、意図通りの仕事ができるソフトが正しいのでこれはダメ。
それとは別にソースがダサいかダサくないかで言えば、むちゃくちゃダサくて保守性も生産性も低いので、書き直せるなら全部書き直しするのがいい。
でも人材がいないのにそんな無理を言っても仕方がないので、意図通りの動きを目指せばそれでいい。

[Matsuri_]

Zuishinさん

攻撃的な回答は求めていませんので、
次回からこちらへの書き込みを控えてくださるよう、よろしくお願いいたします。

[ozwk]

ソフトウェアの正しさとはまず第一に意図通りの仕事ができるということなので、そういった意味では正しくないです。

それとは別に保守性や生産性の観点で言えば、どちらも低そうという点で筋が悪いので、状況が許すならば全部書き直しをおすすめします。
とはいえ人材がいないのにそんな無理を言っても仕方がないので、意図通りの動きを目指せばそれでいいと思います。

[Zuishin]

求められる本当のことを言っただけで攻撃などしていない。そのような攻撃的なコメントは求めていないので、よそでどうぞ。

[Zuishin]

あなたの言う「正しい」の意味がわからなかったため、それを「ダサくない」の意味かと解釈しただけなので、ちゃんと伝わる言葉を書くようにすればそのような評価は読まずに済んだはず。

[Matsuri_]

ZuishinさんのオーバーライドやHashsetなども知らなかったので、調べてみますね。
ありがとうございました。

Answer 1

他のコミュニティにおいて、クラスのリストは作らず、変数を書き換える形で解決する方法を教えて頂き、
解決致しました。