NullReferenceException: Object reference not set to an instance of an object почему?

Question

[WeBBeW]

Почему эту ошибку выводит?
Вот код:

Inventory

for(int i = 0; i < inventoryItems.Length;i++)
        {

            if(inventoryItems[i] != null) continue;
            inventoryItems[i] = it;
            print("item add - " + inventoryItems[i]);
            (new InventoryWindow()).Redraw();
        }

InventoryWindow

InventoryWindow

public class InventoryWindow : MonoBehaviour
{
    [SerializeField] public Inventory targetInventory;
    [SerializeField] public Transform itemsPanel;
    void Start()
    {
        Redraw();
    }
    void Update() 
    {
        if(Input.GetKeyDown(KeyCode.F))
            Redraw();
    }
    public void Redraw()
    {
        bool ChildNull = true;
        bool RemoveSprites = false;
        bool finish = false;
        while(finish != true)
        {
            print("z - " + itemsPanel.transform.childCount);
            if(itemsPanel.transform.childCount == 0) {ChildNull = true;}
            else if(itemsPanel.transform.childCount > 0) {ChildNull = false;}

            print("bool - " + ChildNull);
            if(ChildNull == true)//если спрайтов нету 
            {
                for(var i = 0; i < targetInventory.inventoryItems.Length; i++)
                {
                    if(targetInventory.inventoryItems[i] != null)
                    {
                        var item = targetInventory.inventoryItems[i];

                        GameObject icon = new GameObject("Icon");
                        icon.AddComponent<Image>().sprite = item.Icon;
                        icon.transform.SetParent(itemsPanel);
                        icon.transform.localScale = Vector3.one;
                    }
                } 
            }else//если есть 
            {
                if(RemoveSprites == false)
                {
                    for(int i = 0; i < itemsPanel.transform.childCount; i++)   
                    {
                        Destroy(itemsPanel.transform.GetChild(i).gameObject);
                        if(i == (itemsPanel.transform.childCount - 1)) RemoveSprites = true;
                    }
                }else if(RemoveSprites == true)
                {
                    for(var i = 0; i < targetInventory.inventoryItems.Length; i++)
                    {
                        if(targetInventory.inventoryItems[i] != null)
                        {
                            var item = targetInventory.inventoryItems[i];

                            GameObject icon = new GameObject("Icon");
                            icon.AddComponent<Image>().sprite = item.Icon;
                            icon.transform.SetParent(itemsPanel);
                            icon.transform.localScale = Vector3.one;
                        }
                        if(i == (targetInventory.inventoryItems.Length - 1)) finish = true;
                    }    
                }
            }
        }
    }
}

Ошибка на эту строчку :
print("z - " + itemsPanel.transform.childCount);
, но если убрать эту строчку, то на эту

if(itemsPanel.transform.childCount == 0) {ChildNull = true;}

Ошибку показывает если Redraw вызвать из Inventory, а если вызвать из InventoryWindow, то этой ошибки нету

Answer 1

[CHIDWI]

Во - первых использовать While -плохо.
Во - вторых ошибка NullReferenceException, означает что вы пытаетесь получить доступ к обьекту которого нет(null) или пуст, возможно вы его удалили или он еще не создан. Из описанного выплывает что в itemsPanel.transform.childCount - null(пусто). Как вариант можете написать так print("z - " + itemsPanel.transform?.childCount);

Comments

[freeExec]

null там itemsPanel

[CHIDWI]

Так а с чего бы им взяться, если (new InventoryWindow()).Redraw();
Нужно тогда сперва было бы задать параметры для них или обьекты, а потом вызывать метод. Хотя мне кажется что было бы достаточно просто вызвать метод Redraw() без создания экземпляра класса. Это всё вилами по воде конечно, так как надо тестить.

[WeBBeW]

Георгий Норенко, а как можно обратиться к этому Redraw из другого скрипта? Если вот так: InventoryWindow.Redraw();, то выводит ошибку "error CS0120: An object reference is required for the non-static field, method, or property 'InventoryWindow.Redraw()'". А в интернете нашел, что можно обратиться к редраву вот так: (new InventoryWindow()).Redraw();
Так как тогда обратиться к Redraw?

[CHIDWI]

WeBBeW, Сперва нужно получить доступ к этому скрипту и если прочесть что значит ошибка, можно многое понять. Можно пометить обьект статиком как там написано, можно через Find() найти, много как можно получить доступ. Самое простое описано ниже. И не совсем понятно для чего делать public поля еще и сериализируемыми, но это уже ваше дело так сказать.

public InventoryWindow inventoryWindow; //сюда затягиваем в инспекторе обьект на котором висит скрипт InventoryWindow

private void Start()
{
 this.inventoryWindow.Redraw(); //теперь можно это сделать
}

Если честно, то при возникновении подобных вопросов, стоит почитать мануалы/видосики/документацию, дабы понимать основы.

[WeBBeW]

Георгий Норенко, спасибо

Answer 2

[freeExec]

(new InventoryWindow()).Redraw();

А ещё там должно быть написано, что не нужно ньюкать MonoBehaviour классы.

Но если вы так делаете, позаботьтесь, чтобы ваши SerializeField были заданы вами в вашем новом инстансе.

Comments

[WeBBeW]

а как можно по другому сделать? В интернете только такие примеры видел

[freeExec]

WeBBeW, В Unity нужно делать Instantiate для клонирования GameObject

[WeBBeW]

freeExec, что-то я не понимаю. А зачем это делать, если я хочу просто вызвать Redraw(в Inventory.cs) из InventoryWindow.cs?