Вычисление двухмерного массива

[artas-7]

Добрый день. Вот уперся в двухмерный массив. В итоге должно получиться следующее: отображение нескольких кнопок, каждой из них присвоена своя текстура, при нажатии на кнопку меняется текстура на объекте.
Массив написал, как линейный он работает, пытаюсь его переделать для двухмерного, чтобы кнопки отображались по двум осям, а не только по одной.
Пытался добавить еще одну переменную, в коде это k... но в итоге ничего не отображается.
Подскажите пожалуйста где может быть ошибка?

Синтаксис:

Синтаксис: [ Показать ]

Используется csharp

using UnityEngine;
using System.Collections;

public class example: MonoBehaviour {
static int Count = 4;
int k = 0;     
int Counti = 2;
int Countj = 2;
public Texture[] vec = new Texture[Count];
bool[] vb = new bool [Count];
public GameObject down01;
public Texture SPF6901;
public Texture SPF6902;
public Texture SPF6903;
public Texture SPF6904;
public Texture SPF6905;

void OnGUI()
{
vec[0] = SPF6901;
vec[1] = SPF6902;
vec[2] = SPF6903;
vec[3] = SPF6904;
vec[4] = SPF6905;

k=0;   
        for(int i=0; i < Counti; i++){
        for(int j=0; j < Countj; j++){

if (GUI.Button(new Rect(10+15*j, 10 + 10*i, 30, 30), vec[k]))
vb[k]=!vb[k];
k++;
}
}
k=0;
        for(int i=0; i < Counti; i++){
        for(int j=0; j < Countj; j++){
if (vb[k]) {
down01.GetComponent<MeshRenderer>().material.mainTexture = vec[k];
vb[k]=!vb[k];
k++;
}
}
}
}
}

 

[Paul Siberdt]

А почему бы не пользовать массив из Vector2, раз нужно два значения? Зачем пользовать двумерный?

[waruiyume]

А в консоль западло смотреть?

Скрытый текст: показать

IndexOutOfRangeException: Array index is out of range.
(wrapper stelemref) object:stelemref (object,intptr,object)
example.OnGUI () (at Assets/example.cs:29)

Синтаксис:

Синтаксис: [ Показать ]

Используется csharp

using UnityEngine;
using System.Collections;

public class example: MonoBehaviour {
static int Count = 5;/////////было 4
int k = 0;    
int Counti = 2;
int Countj = 2;
//public Texture[] vec = new Texture[Count];
        Texture[] vec = new Texture[Count];
bool[] vb = new bool [Count];
public GameObject down01;
public Texture SPF6901;
public Texture SPF6902;
public Texture SPF6903;
public Texture SPF6904;
public Texture SPF6905;
void OnGUI()
{
        vec[0] = SPF6901;
        vec[1] = SPF6902;
        vec[2] = SPF6903;
        vec[3] = SPF6904;
        vec[4] = SPF6905;

        k=0;  
        for(int i=0; i < Counti; i++)
        {
                for(int j=0; j < Countj; j++)
                {
                        if (GUI.Button(new Rect(10+15*j, 10 + 10*i, 30, 30), vec[k]))
                        vb[k]=!vb[k];
                        k++;
                }
        }
        k=0;
        for(int i=0; i < Counti; i++)
        {
                for(int j=0; j < Countj; j++)
                {
                        if (vb[k])
                        {
                                down01.GetComponent<MeshRenderer>().material.mainTexture = vec[k];
                                vb[k]=!vb[k];
                                k++;
                        }
                }
        }
}
}
 

[Golandez]

Не понятно как он будет двухмерным,если вы его таковым не обьявили и зачем инициализация в OnGUI ?
P.S. waruiyume +1.Не заметил сразу.

[Paul Siberdt]

А... я про вектора, а тут ж массив текстурный.
Все равно ничто не мешает пользовать одномерный массив, увеличив число ячеек вдвое, или выдумать свой класс:

Код: Выделить всё

class cButton{
var tex1 : Texture;
var tex2 : Texture;
}
var buttons : cButton[];

Многомерные массивы - это большая какашка, на которой поскальзываешься в самый неподходящий момент и, как правило, падаешь в нее исключительно физиономией.

[artas-7]

да действительно
static int Count = 5;
не заметил...
теперь все работает.
но вот какая другая проблема.
он выводит кнопки, но не каждую отдельную, а просто их дублирует по осям. создает их копии всего лишь, а мне хочется чтобы создавал новую колонку с кнопками...

[Golandez]

Вы продублировали одно событие на все ячейки.Чего же вы тогда хотите? Сделайте для каждой ячейки свое событие.

[artas-7]

в том то и проблема.
до этого было много много кнопок...
но их примерно 1000, возникла проблема с их размещением по координатам, сегодня стал упрощать, сделал вот такой массив,
массив работает по одной оси, либо по горизонтали, либо по вертикали, что в принципе упростило работу в разы, теперь хочу сделать еще проще, чтобы этот массив размещал кнопки не по одной оси, а по двум...
вот последнее догнать никак не могу...

[Zaicheg]

В будущем за такое форматирование кода (как в первом сообщении темы) будут выдаваться предупреждения и баны.

Надо так (на мелких разрешениях в браузере может выглядеть не весело, но при переносе в VS или Notepad получаем нормальный код, в некоторых случаях даже с табуляциями). Форматировал при помощи Visual Studio, с галочкой "Keep tabs" и функцией автоформата (Ctrl + E → D):

Синтаксис:

Синтаксис: [ Показать ]

Используется csharp

using UnityEngine;
using System.Collections;

public class example : MonoBehaviour
{
        static int Count = 4;
        int k = 0;
        int Counti = 2;
        int Countj = 2;
        public Texture[] vec = new Texture[Count];
        bool[] vb = new bool[Count];
        public GameObject down01;
        public Texture SPF6901;
        public Texture SPF6902;
        public Texture SPF6903;
        public Texture SPF6904;
        public Texture SPF6905;

        void OnGUI()
        {
                vec[0] = SPF6901;
                vec[1] = SPF6902;
                vec[2] = SPF6903;
                vec[3] = SPF6904;
                vec[4] = SPF6905;

                k = 0;
                for (int i = 0; i < Counti; i++)
                {
                        for (int j = 0; j < Countj; j++)
                        {

                                if (GUI.Button(new Rect(10 + 15 * j, 10 + 10 * i, 30, 30), vec[k]))
                                        vb[k] = !vb[k];
                                k++;
                        }
                }
                k = 0;
                for (int i = 0; i < Counti; i++)
                {
                        for (int j = 0; j < Countj; j++)
                        {
                                if (vb[k])
                                {
                                        down01.GetComponent<MeshRenderer>().material.mainTexture = vec[k];
                                        vb[k] = !vb[k];
                                        k++;
                                }
                        }
                }
        }
}

[Order]

А чего бы не сделать класс ячейки в котором будет содержание, текстуры, координаты на экране и прочее и затолкать его в одномерный массив.
При выведении пересчитывать координаты ячейки с учетом положения сумки и разрешений экрана.