Unityでタグからランダムにアクティブ化したい

Question

前提・実現したいこと

Unityで間違い探しを作ろうとしています。
20箇所の決まった場所をタグで取得してその中から間違いを毎回ランダムに10箇所アクティブにし、その数をテキストに表示しようとしています。
しかし、アクティブになる数が6~9箇所程度になってしまいます。

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

10箇所アクティブにしたいのですが、アクティブになる数が6~9箇所程度になってしまいます。

該当のソースコード

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

public class random : MonoBehaviour
{
    GameObject[] Offs;
    
    void Awake()
    { 
      Offs = GameObject.FindGameObjectsWithTag("off");
      foreach (GameObject OffObj in Offs)
      OffObj.SetActive(false);
        {
            int a = 0;
            while (a < 10)
            {
                {
                    int number = Random.Range(0, Offs.Length);
                    Offs[number].SetActive(true);
                    Offs[number].tag = "on";
                    ++a;
                }
            }
        }
    }
}

Answer 1

次に下記のコードでは純粋なランダム値でオブジェクトの操作をしているので、重複が考慮されていません。仮に乱数で 1 -> 12 -> 1 -> 5 -> 1 というように同じ数字が出た場合に既にActiveになったオブジェクトを再度Activeにしようとしています（つまりActiveになって欲しい数よりも少なくなってしまう）

int number = Random.Range(0, Offs.Length);
Offs[number].SetActive(true);
Offs[number].tag = "on";
++a;

この解決方法は色々ありますが、かんたんなのは重複しない乱数の生成を行うことです。
リストとかに数字を入れてシャッフルしてからn個取り出すみたいなやり方だったり、色々あるので検索してみてください。（teratailでも頻出する質問なのですぐに見つかります）

Comments

[aki2924]

なるほど！ありがとうございます。
いろいろ勉強して試してみます。

Answer 2

下記の様に書き直したら出来ました。

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

public class random : MonoBehaviour
{
    GameObject[] Offs;
    GameObject[] Ons;

    void Awake()
    {
        Offs = GameObject.FindGameObjectsWithTag("off");
        foreach (GameObject OffObj in Offs)
            OffObj.SetActive(false);
        {
           
            while (true)
            {
                {
                    int number = Random.Range(0, Offs.Length);
                    Offs[number].SetActive(true);
                    Offs[number].tag = "on";

                    Ons = GameObject.FindGameObjectsWithTag("on");
                    foreach (GameObject OnObj in Ons) ;
                        int Count = Ons.Length;

                        if (Ons.Length == 10)
                    {
                        break;
                    }
                   
                }
            }
        }
    }
}