C# 配列 配列の境界外とでます

Question

範囲内なのですが例外が発生します 要素数を10に指定しているので範囲内だと思います。
ご教授お願い致します。

    public string[] UserNames = new string[10];
 
    public void UpdateMemberList()//一度だけ呼び出す
    {
        foreach (var p in PhotonNetwork.playerList)
        {
            UserMgr($"User{Count}", p.NickName);//アイコンを表示する
            MyNumber = PhotonNetwork.playerName == p.NickName ? Count : MyNumber;//自分の背番号を取得する
            Debug.Log($"{Count}"+ p.NickName);
            UserNames[8] = p.NickName ?? "UserName";//配列に代入 ←8以上の数字だと例外が発生するので明示的に8と指定しました            
            ++Count;
        }      
    }

IndexOutOfRangeException: Index was outside the bounds of the array.
(wrapper stelemref) System.Object.virt_stelemref_sealed_class(intptr,object)
PhotonMgr.UpdateMemberList () (at Assets/Script/PhotonMgr.cs:96)
PhotonMgr.OnJoinedRoom () (at Assets/Script/PhotonMgr.cs:83)
UnityEngine.GameObject:SendMessage(String, Object, SendMessageOptions)
NetworkingPeer:SendMonoMessage(PhotonNetworkingMessage, Object[]) (at Assets/Photon Unity Networking/Plugins/PhotonNetwork/NetworkingPeer.cs:2856)
NetworkingPeer:OnEvent(EventData) (at Assets/Photon Unity Networking/Plugins/PhotonNetwork/NetworkingPeer.cs:2564)
ExitGames.Client.Photon.PeerBase:DeserializeMessageAndCallback(StreamBuffer)
ExitGames.Client.Photon.EnetPeer:DispatchIncomingCommands()
ExitGames.Client.Photon.PhotonPeer:DispatchIncomingCommands()
PhotonHandler:Update() (at Assets/Photon Unity Networking/Plugins/PhotonNetwork/PhotonHandler.cs:161)


日本語訳:インデックスは配列の境界外でした。

Comments

[y_waiwai]

エラー・メッセージは勝手な省略、翻訳せず、出てきたそのままコピペで提示してください

[退会済みユーザー]

エラーメッセージを追加しました、回答よろしくお願いいたします

[Y.H.]

「8以上の数字だと例外が発生するので明示的に8と指定しました 」
8と指定した状態でエラーが発生するのでしょうか？

[退会済みユーザー]

8と指定した状態で実行するとエラーが発生します

[takabosoft]

代入する前あたりに
Debug.WriteLine(UserNames.Length);
などと入れてみてUserNamesの長さが変わっていないかチェックしてみてはどうでしょう

[退会済みユーザー]

確認してみます

[Y.H.]

> 8と指定した状態で実行するとエラーが発生します
8と指定した状態でキチンとビルドされてますか？

[退会済みユーザー]

Debug.Log($"配列要素数チェック{UserNames.Length}");で実行してみました　出力は8でした
new string[10];としたのですが出力が8でした .Netフレームワーク側のバクでしょうか？

[papinianus]

これunityですよね。詳しくないんですが、UpdateのほかStartなど呼び出しがかかりますよね。そこのコードで何かしてるほうを疑ってください。
C#の配列が10で宣言したら8になるというバグがあったら他の人が気づいてます。010は8進数の8なのでワンチャン

[BluOxy]

確かに原因がC#ではなくUnity側の可能性も考えられるので、Unityタグをつけた方が良いと思います。

[退会済みユーザー]

アクセスレベルをpriveteに落としたら動きました。
恐らくUnityエディター等から値を操作されていたようです　不用意にpublicで宣言してはいけないことを学びました　
回答者の方々ありがとうございました。

Answer 1

UserNames = new string[10];の場合、添え字でアクセスできる範囲は0から9までです。
PhotonNetwork.playerListの個数が11以上になる場合、11回目のループでUserNames[10]という存在しない要素にアクセスすることになるためIndexOutOfRangeExceptionの例外が発生します。

解決策

• CountがUserNames.Length以上のときはUserNames[Count]にアクセスしないよう修正する
• UserNamesを配列ではなくリストで管理し、UserNames.Add(p.NickName ?? "UserName");という形で要素を追加する
• UserNamesを配列ではなくリストで管理し、LINQを使ってUserNamesを更新する(UserNames = PhotonNetwork.playerList.Select(x => x.NickName ?? "UserName");)

Comments

[退会済みユーザー]

UserNames[8] = p.NickName ?? "UserName";　と記述しているので8にしかアクセスしないと思います。
やはりアクセスできる範囲は0から9までですよね、8は範囲なのに代入できないのは理由がわかりません
回答ありがとうございました。

[BluOxy]

UserNamesを参照している箇所は掲示されたコードのみでしょうか。例えばUpdateMemberListが呼ばれる前に他のメソッドでUserNamesをUserNames = new string[5];のように書き換えた場合は例外が発生します。

[BluOxy]

UpdateMemberListのどこかにDebug.Log(UserNames.Length);と書いて10がログに表示されることは確認されたでしょうか。

[退会済みユーザー]

確認してみます

[退会済みユーザー]

Debug.Log($"配列要素数チェック{UserNames.Length}");で実行してみました　出力は8でした

[BluOxy]

VisualStudioで開いているなら、UserNamesという変数をダブルクリックし「すべての参照を検索」をクリックしてください。検索に引っかかったコードでUserNamesを書き換えてないか確認をしてください。

[BluOxy]

また、コンパイルエラーがあれば全て解消し、ビルドが完了した後に動作確認を行ってください。

[退会済みユーザー]

すべての参照を検索から検索しても読み取りしかありませんでした、理由がわからないので配列を使わずに他の方法を使うことにしました。
回答有り難うございました

[BluOxy]

そうですか。他に思いつく怪しい点はUserNamesがpublicなので、Unityのインスペクタ上からUserNamesを設定していないかどうかです。

何か見落としがある気はしますが、念のためスクリプト外から触られないようpublicではなくprivateで宣言した方が良いように感じます。

[退会済みユーザー]

アクセスレベルをpriveteに落としたら動きました。
恐らくUnityエディター等から値を操作されていたようです　不用意にpublicで宣言してはいけないことを学びました　
ありがとうございました。

[BluOxy]

そちらでしたか。解決されて良かったです。