【UniRx】ボタンにOnClickAsObservable().Subscribe() で処理を登録するとき、for文でまとめて登録しようとするとうまくいかない

Question

発生する現象

ボタンにOnClickAsObservable().Subscribe()で処理を登録するとき、for文でまとめて登録しようとすると、うまくいかない現象が発生しています。
これに関して解決方法などありますでしょうか。

ソースコード

ボタンがあるUI用クラス

using UnityEngine;
using UnityEngine.UI;
using UniRx;
using System;

public class ButtonUI : MonoBehaviour
{
    [SerializeField] private Button[] ChoiceButtons;

    /// <summary>
    /// 指定したインデックスのボタンをクリックしたときの動作を登録
    /// </summary>
    /// <param name="index"></param>
    /// <param name="buttonAction"></param>
    public void SetButtonClickAction(int index, Action buttonAction)
    {
        ChoiceButtons[index].OnClickAsObservable().Subscribe(_ => buttonAction());
    }
}

ボタンを押したときの処理を登録するクラス

using UnityEngine;
using System;

public class ChoiceEvent
{
    private ButtonUI ui;

    /// <summary>
    /// 選択肢イベントでボタンを押下したときの処理を追加
    /// </summary>
    private void SetButtonAction()
    {
        ui = GameObject.FindObjectOfType<ButtonUI>();
        // なぜかfor文を使用するとボタンのイベント登録がミスる
        // 現状ボタンの数は4つなので、こう書けば一応動作はする模様
        ui.SetButtonClickAction(0, () => 
        {
            Debug.Log($"選択肢{0}を押下");
        });
        
        ui.SetButtonClickAction(1, () => 
        {
            Debug.Log($"選択肢{1}を押下");
        });

        ui.SetButtonClickAction(2, () => 
        {
            Debug.Log($"選択肢{2}を押下");
        });

        ui.SetButtonClickAction(3, () => 
        {
            Debug.Log($"選択肢{3}を押下");
        });
    }
}

Unity バージョン

• 2021.2.11

Comments

[ozwk]

「うまくいかない」とはどうなるのでしょう?

Answer 1

ui.SetButtonClickAction(i, () => 
        {
            var x = i;
            Debug.Log($"選択肢{x}を押下");
        });

Answer 2

/// <summary>
/// 選択肢イベントでボタンを押下したときの処理を追加
/// </summary>
private void SetButtonAction()
{
    ui = GameObject.FindObjectOfType<ButtonUI>();
    for(int i = 0; i < 4; i++)
    {
        // iを一旦別の変数に格納すると、正常に動作する
        var count = i;
        ui.SetButtonClickAction(count, () => 
        {
            Debug.Log($"選択肢{count}を押下");
        });
    }
}

このように、indexを一旦別の変数に格納すると正常に動作する模様です。
参考：https://qiita.com/D4ik1/items/21db399d075e7269b0da

C# だけでなく、よく発生する事象とのことで、そういうものとして割り切るしかないようです。

Comments

[bboydaisuke]

それ、付いている回答と同じことを言ってますよ。なので「自己解決」じゃないのでは？

> よく発生する事象

単純に文法を間違えてしまったということですよね。ラムダ式の仮引数名に、既に存在している変数名を割り当ててしまっていた、つまり変数宣言が重複していた、ということですよね。

[honjoriki]

すいません、調べて記入しているうちに他の方も回答してくださっていたようです。

[ozwk]

> 単純に文法を間違えてしまったということですよね。ラムダ式の仮引数名に、既に存在している変数名を割り当ててしまっていた、つまり変数宣言が重複していた、ということですよね。

違いますよ。4つのラムダ式がすべて同じ変数 i をキャプチャしていたため
4つすべてが同じ出力になってしまっていたのです。