Почему IndexOutOfRangeException: Array index is out of range?

Question

[Антон Вертов]

Доброго времени суток. Вылазит ошибка IndexOutOfRangeException: Array index is out of range, но не пойму почему.
Вот код:

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

public class Gameplay : MonoBehaviour {

    string Word;
    public GameObject PrefabCharCell;
    public GameObject[] tempCell;
    public Transform GridCellPos;
    
	void Start ()
    {
        Word = "Meet"; 

        foreach (char ch in Word.ToUpper().ToCharArray())
        {
            for (int i = 0; i < Word.Length; i++)
            {
                tempCell[i] = Instantiate(PrefabCharCell);
                tempCell[i].transform.SetParent(GridCellPos, false);
            }
		}      
    }
}

Я хочу каждый созданный объект сохранить в массив tempCell, чтобы потом работать с ними (объектами), но выдаётся ошибка IndexOutOfRangeException: Array index is out of range, что я выхожу за пределы массива, только почему ?
Подскажите пожалуйста, как решить проблему ? Или альтернативный способ, как работать с созданными объектами (массивами) через скрипт. Заранее огромное спасибо!

Comments

[profesor08]

Где точный текст ошибки, которая тебе вываливается?

[Антон Вертов]

profesor08,
IndexOutOfRangeException: Array index is out of range.
(wrapper stelemref) object:stelemref (object,intptr,object)
Gameplay.Start () (at Assets/Scripts/Gameplay.cs:46)

Answer 1

[EVGENY T.]

Надо выделить память под массив tempCell

tempCell = new GameObject[Word.Length];

Answer 2

[eRKa]

Наверняка потому, что tempCell.Length < Word.Length. На какой то итерации ошибка и вылезет.

Comments

[Антон Вертов]

Подскажите пожалуйста. Всё равно не понимаю. У меня есть массив, я не обозначил, сколько в нем содержится элементов.
Word.Lenght очевидно равен 3 (0,1,2,3), так почему, когда я пытаюсь добавить в tempCell объект, он не соответствует индексам ?

спасибо большое.

[profesor08]

Антон Вертов, очевидно ты считать не умеешь, Word.Length равен 4, потому что в слове "Meet" 4 буквы.
Отсчет начинается с нуля, значит 0 это первый символ, 1 - второй, 2 - третий, 3 - четвертый.

А значит твой tempCell должен быть объявлен как-то так tempCell = new GameObject[4], или так tempCell = new GameObject[Word.Length]

[Антон Вертов]

profesor08, ошибка пропала, но теперь цикл выполняется 16 раз. 0.1.2.3, и так три раза 0.1.2.3, 0.1.2.3, 0.1.2.3. Почему так ?

[profesor08]

Антон Вертов, ну у тебя же там два цикла, и каждый из них работает по 4 раза, а 4 * 4 = 16.

[Антон Вертов]

profesor08, туплю уже дико, голова не работает уже совсем. Спасибо, за помощь Вам большое. Последняя просьба - подскажите пожалуйста, как в одном цикле заполнить и tempCell созданными объектами и разделить одновременно по символам ? (выполнить char ch in Word.ToUpper().ToCharArray())

спасибо!

[profesor08]

Word.charAt(i)

[m0nym]

profesor08, ну ты и терпеливый

[eRKa]

Антон Вертов, Массивы стоит делать когда заранее известна его длина, при добавлении он автоматом не увеличивается, т.е. если объявили его размером 4, то больше он не станет, только создавать новый бОльшей длины и копировать туда предыдущий набор. Поэтому в Вашем случае лучше использовать коллекцию, например

List<GameObject> tempCell = new List<GameObject>();

тогда при добавлении в коллекцию
tempCell.Add(Instantiate(PrefabCharCell))
ее длина будет увеличиваться автоматически по мере необходимости.

Answer 3

[Сурен Джагарян]

for (int i = 0; i < Word.Length - 1; i++)
Попробуй так