Este problema sucede por la siguiente razón.
Al usar métodos anónimos en C#, se crea lo que se conoce como Closure
.
Un closure es, básicamente, un bloque de código que puede ser ejecutado en el futuro,pero que mantiene el ambiente en el que fue creado, es decir, puede utilizar todavía las variables locales del método que lo creó, incluso aunque ese método haya finalizado su ejecución.1
Para conseguir esto, lo que se hace es lo que se conoce como Variables Capturadas
. Sin entrar en mucha profundidad, lo que se hace es guardar una referencia a la variable que se quiere capturar.
Entrando en el caso que nos ocupa. Cuando tu creas el método anónimo (eventData) => { showBuildTypeName(i); }
, lo que .net hace internamente es almacenar una referencia a la variable i
. El problema es que esa variable, tras el bucle for
, acaba obviamente con un resultado de 5
. Con lo que cuando realmente el método anónimo es ejecutado, el valor de i
siempre sera 5
.
La solución es simplemente crear una referencia interna para cada valor, y de esa manera el closure
será capaz de capturar la variable con su valor correcto. En tu caso, capturamos el valor de i
en una nueva variable ID
, y pasamos esa referencia en el método anónimo:
for (int i = 0; i < 5; i++)
{
buildTypeButtons[i] = Instantiate(BuildTypeButton, new Vector2(0, posY), Quaternion.identity) as GameObject;
buildTypeButtons[i].transform.SetParent(panelTypesGO.transform, false);
EventTrigger.Entry entry = new EventTrigger.Entry();
entry.eventID = EventTriggerType.PointerEnter;
int ID = i;
entry.callback.AddListener((eventData) => { showBuildTypeName(ID); });
buildTypeButtons[i].GetComponent<EventTrigger>().triggers.Add(entry);
posY -= 45;
}
1 https://stackoverflow.com/a/428624/579895
(Si alguien quiere profundizar un poco mas en el tema, aqui hay una explicación de @EricLipper que era parte del equipo de diseño del lenguaje C#, en inglés)
int ID = i;
despues deentry.eventID = EventTriggerType.PointerEnter;
y cambia elAddListener
asi:entry.callback.AddListener((eventData) => { showBuildTypeName(ID); });