Помогите оптимизировать скрипт

[e-x]

Есть массив, мои кнопки

public GameObject[] buttons

на кнопках висит скрипт Button, мне нужна обращаться к элементу Image скрипту Button далее у меня везде так

Синтаксис:

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

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

public void ButtonClickDown()
        {
            for (int i = 0; i < buttons.Length; i++)
            {
                if (buttons[i].GetComponent<Button>().id == currentId)
                {
                    buttons[i].GetComponent<Image>().color = buttons[i].GetComponent<Button>().selectedColor;
                    switch (currentId)
                    {
                        case 1:
                            SlotCraftEnable();
                            break;
                    }
                    switch (currentId)
                    {
                        case 2:
                            SlotHandsEnable();
                            break;
                    }
                    foreach (GameObject item in slotItem)
                    {
                        item.GetComponent<Image>().color = buttons[i].GetComponent<Button>().selectedColor;
                    }
                }
                if (buttons[i].GetComponent<Button>().id != currentId)
                {
                    buttons[i].GetComponent<Image>().color = disableColor;
                    buttons[i].GetComponent<Button>().enabled = false;
                }
            }
        }

 

везде вообщем GetComponent мне кажется как то это неправильно, может можно как-то указать в функции Старт GetComponent ?

[jr900]

Все GetComponent по возможности нужно кэшировать в отдельные переменные в Start:

Button _button = GetComponent<Button>();
Image _image = GetComponent<Image>();

А в Update уже юзать полученные переменные.

[e-x]

Я чет не совсем понял, в моем случае у меня массив

public GameObject[] buttons

я работаю с buttons[i].....как в моем случае должно быть?

Спасибо!

[Tolking]

не public GameObject[] buttons, а public Button[] buttons
А у Button есть image, color или spriteState на конец...

[e-x]

Ну ОК одну проблему так решим, но к имеджу если надо то все равно будет потом так

buttons[i].GetComponent<Image>().color = highliterColor;

[Tolking]

buttons[i].color
buttons[i].image.color
buttons[i].spriteState.highlightedSprite.color
buttons[i].spriteState.pressedSprite.color
buttons[i].spriteState.disabledSprite.color

[e-x]

Так в том то и дело что не находит у меня то что Вы пишите, я не использую Button из UI...у меня Button это image

[Tolking]

Ты что-то придумал, и что то городишь свое... А от нас чего хочешь? Либо дальше велик изобретай, либо пользуйся, тем, что есть и спрашивай о том, что другие могут знать... Мы откуда можем знать, что ты там придумал и для чего?

В своем бутоне сделай поле image, как в Button из UI... Только если тебе нужна кнопка, то точно нужно велик изобретать?

[e-x]

Кнопка из UI не интересно, хочу сам все с нуля.
А GetComponent кешировать в Старте..это как вообще правильно?Или все же строить так чтоб не прибегать к этому?

[Tolking]

Ну можно вообще не прибегать к GetComponent и руками установить нужный объект в поле до рантайма...

[Alkos26Rus]

Кнопка из UI не интересно, хочу сам все с нуля.
А GetComponent кешировать в Старте..это как вообще правильно?Или все же строить так чтоб не прибегать к этому?

Это твой собственный класс Button? Во-первых так не называй никогда, у юнити уже есть такой. Если это твой, то ты туда можешь любые поля прописать. А массив уже делай не из GameObject а из твоего класса Button и обращайся напрямую buttons[i].id и т.д

[e-x]

Так и сделал, налепил в Button имеджов, руками вставил, гет компонент не нужно.

А как понять что в Unity уже есть какойто клас, название? Где это написано?

Да Button это мой клас, Юнити не ругался, может то что пространство имен у меня другое.

[1max1]

Класс юнити https://docs.unity3d.com/ru/2018.4/Scri ... utton.html