UnityでのC#を使ったボタン判定

前提・実現したいこと

ボタンを押したら画像が変わりトランプの柄が表示されるようにしたい

    for (var i = 0; i < 13; i++)
    {

        D_[i].onClick.AddListener(delegate
    {

        D[i].sprite = Deck[i];
    });

    }

を追加することでＤ０からＤ１２（ダイヤの１からダイヤの１３）までのトランプのボタンを作れると思ったのですが
うまくいきませんでした

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

Array index is out of range

該当のソースコード

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


public class Sinken : MonoBehaviour
{


    public Sprite TBack;

    public Sprite[] TArray = new Sprite[52];


    Image Back;



    UnityEngine.Sprite[] Deck = new UnityEngine.Sprite[52];

    UnityEngine.UI.Image[] D = new UnityEngine.UI.Image[13];
    UnityEngine.UI.Button[] D_ = new UnityEngine.UI.Button[13];

    UnityEngine.UI.Image[] K = new UnityEngine.UI.Image[13];
    UnityEngine.UI.Button[] K_ = new UnityEngine.UI.Button[13];

    UnityEngine.UI.Image[] H = new UnityEngine.UI.Image[13];
    UnityEngine.UI.Button[] H_ = new UnityEngine.UI.Button[13];

    UnityEngine.UI.Image[] S = new UnityEngine.UI.Image[13];
    UnityEngine.UI.Button[] S_ = new UnityEngine.UI.Button[13];


    // Use this for initialization
    void Start()
    {


        for (var i = 0; i < 13; i++)

        
        {
            D[i] = GameObject.Find("D"+(i+1)).GetComponent<Image>();
            D_[i] = GameObject.Find("D" + (i+1)).GetComponent<Button>();

            K[i] = GameObject.Find("K" + (i+1)).GetComponent<Image>();
            K_[i] = GameObject.Find("K" + (i+1)).GetComponent<Button>();

            H[i] = GameObject.Find("H" + (i+1)).GetComponent<Image>();
            H_[i] = GameObject.Find("H" + (i+1)).GetComponent<Button>();

            S[i] = GameObject.Find("S" + (i+1)).GetComponent<Image>();
            S_[i] = GameObject.Find("S" + (i+1)).GetComponent<Button>();
        }


        for (var i = 0; i < TArray.Length; i++)
        {
            Deck[i] = TArray[i];
        }


        for (var i = 0; i < 13; i++)
        {

            D_[i].onClick.AddListener(delegate
        {

            D[i].sprite = Deck[i];
        });

        }



    }

試したこと

forを消して

        D_[０].onClick.AddListener(delegate
    {

        D[０].sprite = Deck[０];
    });

のように０～１２まで試したときはきちんと動いてくれました。

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

行動規範の内容に同意します

回答1件

ベストアンサー

こんにちは。

質問のコードでは、匿名メソッド中で変数iのキャプチャが発生しているため、匿名メソッド内のiの値がfor文により書き換えられてしまっており、onClickの実行時にD[13]にアクセスしているためIndexOutOfRangeExceptionが発生します。
キャプチャはそう簡単に理解できる概念ではないため、説明するのが難しいですが、とりあえずは以下のように書き換えることでエラーを回避できるはずです。

csharp
1        for (var i = 0; i < 13; i++)
2        {
3            var x = i; // for文のスコープ内で変数を宣言
4            D_[x].onClick.AddListener(delegate
5            {
6                D[x].sprite = Deck[x]; // xを利用する
7            });
8        }