0

Unity выдает сразу 2 ошибки:

NullReferenceException: Object reference not set to an instance of an object
TreeTile.StartUp (Vector3Int position, UnityEngine.Tilemaps.ITilemap tilemap, UnityEngine.GameObject go) (at Assets/Scripts/TileScripts/TreeTile.cs:12)

Error running StartUp for Tile.NullReferenceException: Object reference not set to an instance of an object
TreeTile.StartUp (Vector3Int position, UnityEngine.Tilemaps.ITilemap tilemap, UnityEngine.GameObject go) (at Assets/Scripts/TileScripts/TreeTile.cs:12)

которые указывают на строку

    go.GetComponent<SpriteRenderer>().sorting = -position.y * 2;

Понимаю, что где-то ссылаюсь на null, но где - понять не могу.

using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using UnityEditor;
using UnityEngine.Tilemaps;

[RequireComponent(typeof(SpriteRenderer))]
public class TreeTile : Tile
{

    public override bool StartUp(Vector3Int position, ITilemap tilemap, GameObject go)
    {
        go.GetComponent<SpriteRenderer>().sortingOrder = -position.y * 2;
        return base.StartUp(position, tilemap, go);
    }

#if UNITY_EDITOR
    [MenuItem("Assets/Create/Tiles/Treetile")]
    public static void CreateTreetile()
    {
        string path = EditorUtility.SaveFilePanelInProject("Save Treetile", "New Treetile", "asset", "Save Treetile", "Assets");
        if (path == "")
        {
            return;
        }
        AssetDatabase.CreateAsset(ScriptableObject.CreateInstance<TreeTile>(), path);
    }

#endif

}

Для справки: я пытаюсь добавить в тайловую палитру (Tile Palette) префаб дерева с колайдером и объектом. Это мне, вроде, удалось, но деревья ставились друг на друга. Я использую новый скрипт, чтобы они не наслаивались и сортировались. Проект компилируется, всё запускается и отображается без проблем. Но вот сама ошибка печатается, и это напрягает.

Ниже приведены картинки для наглядности.

введите сюда описание изображениявведите сюда описание изображениявведите сюда описание изображениявведите сюда описание изображения

2
  • Вы добавили не весь код, что за закрывающая фигурная скобка в методе StartUp?
    – user302909
    Commented 20 дек. 2018 в 14:46
  • опечатка, допустил при редактиировании
    – Fadiuty
    Commented 20 дек. 2018 в 22:49

1 ответ 1

1

Очевидно, null вы получаете от метода GetComponent<T>(), т.к. к объекту не прикреплен искомый компонент:

GetComponent Unity API

Returns the component of Type type if the game object has one attached, null if it doesn't.


Скорее всего банальная человеческая ошибка, чтобы "не забывать" вешать необходимые компоненты на префабы - пользуйтесь RequireComponent:

[RequireComponent(typeof(SpriteRenderer))]
public class TreeTile : Tile
...

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


UPDATE

Изучив документацию по теме тайлов в Unity стало ясно, что метод StartUp используется для кастомного кода для кастомных спрайтов. Причем этот метод вызывается, видимо, для инициализации самой TilePallete, а значит не для инстанциированных объектов.

Другими словами Unity за кулисами не передает никакой объект параметром в StartUp, что в этой строке:

go.GetComponent<SpriteRenderer>().sorting = ...;

Равносильно

null.GetComponent<SpriteRenderer>().sorting = ...;

Если посмотреть на пример реализации собственных тайлов в API, видно, что в переопределенном StartUp методе нужно вручную проверять условие go != null:

public override bool StartUp(Vector3Int location, ITilemap tilemap, GameObject go)
{
    if (go != null)
    {
        go.transform.rotation = Random.rotation;
    }
    return true;
}
5
  • Спасибо за ваш ответ! В будущем обязательно поможет, но проблему это, к сожалению, не решило. Проект компилируется, всё запускается и отображается без проблем. Но вот сама ошибка печатается, и это напрягает. Для справки скажу, что я пытаюсь в тайловую карту занести префаб со скриптом. Может где-то там накосячил, но найти ошибку - не удаётся (
    – Fadiuty
    Commented 20 дек. 2018 в 9:06
  • @Fadiuty тогда добавьте полный код и вообще всю информацию в вопрос
    – user302909
    Commented 20 дек. 2018 в 9:08
  • я добавил больше информации, может так лучше будет)
    – Fadiuty
    Commented 20 дек. 2018 в 13:54
  • И если ответ вам помог, поставьте ему галочку.
    – aquaprogit
    Commented 21 дек. 2018 в 17:07
  • Благодарю вас за помощь! Добавление if (go != null) действительно помогло. Впредь буду более тщательно изучать документацию!
    – Fadiuty
    Commented 22 дек. 2018 в 21:06

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.