変数の説明がわかりません

Question

私のボタンの設定を変更するために使用する私のループの1つでは、インスペクタではなくAddListener関数も使用します。私は0から4までの "i"の範囲を与える5つの項目を持っていますが、 "i"のトラフを呼び出す必要があります、それは常に5を記録します。決して5に達しません。

p.s.私はCustomEditorを使用して、インスペクタで2つのボタン "プレビューレイアウト"と "プレビューの削除"を表示します。

コード：

using UnityEngine; 
using System.Collections; 
using UnityEditor; 
using UnityEngine.UI; 

public class RateMeManager : MonoBehaviour { 

    public GameObject rateMeCanvas; 
    public Sprite emptyStar, fullStar, button; 
    public float spriteWidth, spriteHeight, spritePadding; 

    [HideInInspector] 
    public GameObject currentCanvas, tempButton; 

    void Start() { 
     RemovePreview(); 
     GenerateStars(); 
    } 

    // Update is called once per frame 
    public void GenerateStars() { 
     RectTransform myRectTransform; 
     if (currentCanvas != null) 
     { 
      GameObject temp; 
      temp = currentCanvas; 
      DestroyImmediate(temp); 
     } 
     currentCanvas = Instantiate(rateMeCanvas, Vector3.zero, Quaternion.identity) as GameObject; 
     GameObject subCanvas = currentCanvas.transform.FindChild("subCanvas").gameObject; 
     myRectTransform = subCanvas.GetComponent<RectTransform>(); 
     myRectTransform.sizeDelta = new Vector2((5*spriteWidth) + (4*spritePadding), spriteHeight); 
     myRectTransform.anchoredPosition = Vector2.zero; 
     Button[] buttons = subCanvas.GetComponentsInChildren<Button>(); 
     float[] positions = new float[] {((2*spriteWidth)+(2*spritePadding))*-1, ((1 * spriteWidth) + (1 * spritePadding)) * -1 , 0, ((1 * spriteWidth) + (1 * spritePadding)), ((2 * spriteWidth) + (2 * spritePadding))}; 
     for (int i = 0; i < buttons.Length; i++) 
     { 
      Debug.Log(i); 
      tempButton = buttons[i].gameObject; 
      tempButton.GetComponent<Button>().image.sprite = emptyStar; 
      myRectTransform = buttons[i].GetComponent<RectTransform>(); 
      myRectTransform.sizeDelta = new Vector2(spriteWidth, spriteHeight); 
      myRectTransform.anchoredPosition = new Vector2(positions[i], 0); 
      tempButton.GetComponent<Button>().onClick.AddListener(() => OnGivenRate(i)); 
     } 
    } 

    public void RemovePreview() 
    { 
     DestroyImmediate(currentCanvas); 
    } 

    private void OnGivenRate(int stars) 
    { 
     Debug.Log("pressed star: " + stars); 
    } 

    public class RateMeEditor 
    { 
     [CustomEditor(typeof(RateMeManager))] 
     public class button : Editor 
     { 
      public override void OnInspectorGUI() 
      { 
       base.OnInspectorGUI(); 

       RateMeManager myScript = (RateMeManager)target; 
       if (GUILayout.Button("Preview Layout")) 
       { 
        myScript.GenerateStars(); 
       } 
       if (GUILayout.Button("Delete Preview")) 
       { 
        myScript.RemovePreview(); 
       } 
      } 
     } 
    } 
} 

Answer 1

あなたのエラーはここにある：

tempButton.GetComponent<Button>().onClick.AddListener(() => OnGivenRate(i)); 

あなたはOnGivenRateまたは閉鎖を使用するためにそれを渡す前に、変数にiを保存する必要があります。そのためループiの終わりに はそれだ5に等しく、なぜあなたはあなたのボタンをクリックしiディスプレイ5.

ときに実行します。

と

var rate = i; 
tempButton.GetComponent<Button>().onClick.AddListener(() => OnGivenRate(rate)); 

または

Action<int> OnGivenRateClosure(int rate) 
{ 
    return() => OnGivenRate(rate); 
} 

tempButton.GetComponent<Button>().onClick.AddListener(OnGivenRateClosure(i)); 

Answer 2

あなたが閉鎖にアクセスしているので、ループは別のサイクルにあるとiが既に変更されたときforループ内呼び出された関数は、唯一しばらくi値を取得します。

別のこと。あなたが書く場合：

for(int i = 0; i < 5; i++) 

コードはi0、1、2、3、4、の値が、iの最後の値があるとforループ内のステートメントを実行します。実際には、のサイクルが終了すると、iがインクリメントされ、チェック「i < 5」が作成され、結果が偽になり、ループが終了します。

Answer 3

あなた for -loop上 iから

あなたの参照：。

tempButton.GetComponent（）onClick.AddListener（（）=> OnGivenRate（I））;

そして匿名メソッドを使用してOnGivenRate(i)を呼び出します。このコードはfor -loopの範囲外にありますが、変数iにアクセスできます。変数は、() => OnGivenRate(i)匿名メソッドによって参照されるとき、i=5（for -loopが終了したとき）を持つ可能性が最も高いでしょう。

Answer 4

コードは冗長です。あなたは既にforループの前に配列としてbuttonを持っていて、次にそれをgameObject（tempButton）に変換してボタンにもう一度....これは参照問題のようです。 forループを以下のコードに置き換えてください。

for (int i = 0; i < buttons.Length; i++) 
{ 
    buttons[i].image.sprite = emptyStar; 
    myRectTransform = buttons[i].GetComponent<RectTransform>(); 
    myRectTransform.sizeDelta = new Vector2(spriteWidth, spriteHeight); 
    myRectTransform.anchoredPosition = new Vector2(positions[i], 0); 
    buttons[i].onClick.AddListener(() => OnGivenRate(i+1)); 
} 

注意差：ボタン[I] .onClick.AddListenerを（（1〜5を表示する

for (int i = 0; i < buttons.Length; i++) 
{ 
    buttons[i].image.sprite = emptyStar; 
    myRectTransform = buttons[i].GetComponent<RectTransform>(); 
    myRectTransform.sizeDelta = new Vector2(spriteWidth, spriteHeight); 
    myRectTransform.anchoredPosition = new Vector2(positions[i], 0); 
    buttons[i].onClick.AddListener(() => OnGivenRate(i)); 
} 

：0~4を表示する

）=> OnGivenRate（i + 1））;