Добавление анимации на префаб из другого скрипта?

Question

[BadCats]

всем привет, имеется такой скрипт:

public class Hero : MonoBehaviour
{
    public CharacterController hero;
    public float speed = 10;
    public float gravity = 5F;
    private Vector3 moveDirection;

 GameObject[] tree =  GameObject.FindGameObjectsWithTag("tree");
    private AnimationClip NewAnimationn;



    // Use this for initialization
    void Start()
    {
      
  
        hero = gameObject.GetComponent<CharacterController>();
         

        



    }

    // Update is called once per frame
    void Update()
    {
        Vector3 moveDirection = Vector3.zero;

        if (hero.isGrounded)
        {
            moveDirection = new Vector3(Input.GetAxis("Horizontal"), 0, Input.GetAxis("Vertical"));
            moveDirection = transform.TransformDirection(moveDirection);
            moveDirection *= speed;

            // раскоментировать для прыжка
            // if (Input.GetButton("Jump"))
            //     moveDirection.y = jumpSpeed;

        }
        moveDirection.y -= gravity * Time.deltaTime;
        hero.Move(moveDirection * Time.deltaTime);
       
        if (Input.GetKey(KeyCode.A))
            {
            for (int i=0;i<tree.Length;i++)
            {
               var p= tree[i].GetComponent<Animation>();
                p.AddClip( NewAnimationn,"TimeLapse");
                p.Play();
                
               
                    }
             }
    }

На сцене имеется префаб- дерево, и не одно, на все префабы я добавил тег "tree" - и на строке

GameObject[] tree =  GameObject.FindGameObjectsWithTag("tree");

- ищу объекты по тегу и запихиваю их в массив, затем при нажатии кнопки D(У меня и персонаж в сторону двигается) и я перебера этот массив - т.е обращаюсь к каждоиу объекту с таким тегом и его компоненте анимация и все это присваиваю в поле с именем p , что бы потом на нем вызвать метод AddClipp() - с помощью которого добовить к каждому префабу дерева нужную мне анимацию, которую я по идее должен передать в качестве аргумента AddClipp()
Как мне в поле NewAnimationn - передать ссылку на

Это вообще анимация? почему спрашиваю? Потому что не знаю! Ведь в проекте также создался файл

И как я понимаю это сам файл анимации, а первый - это так называемый аниматор. Разъясните что к чему.
И последнее - Unity исключение
- при нажатии на клавишу A . В студии же это строка for (int i=0;i<tree.Length;i++) - т.е начало цикла

Answer 1

[Даниил Басманов]

Зачем вы анимацию добавляете в рантайме? Префаб на то и префаб, что его можно настроить заранее в редакторе.
На первой картинке у вас обведён Animator Controller, он вставляется в компонент Animator. Компонент Animation, который вы ищете на деревьях, это старый способ анимирования объектов, он не совместим с аниматором.

Если вы внимательно посмотрите на документацию к GameObject.FindGameObjectsWithTag, то заметите, что этот метод в примере вызывается в старте. Более того, в консоли вам должны были написать:

"FindGameObjectsWithTag is not allowed to be called from a MonoBehaviour constructor (or instance field initializer), call it in Awake or Start instead."

Вы неправильного использовали FindGameObjectsWithTag, поэтому массив деревьев у вас оказался непроинициализирован, что и привело к исключению.

Полезные ссылки:
Официальные туториалы
What is a Null Reference Exception?
Разрешение вопросов, связанных с исключениями: Sys...

Answer 2

[TheTalion]

Зачем добавлять анимацию дерева из героя? Создайте анимацию дерева на дереве.

А вообще анимацию нужно присваивать в самом скрипте. Объявите анимацию как public и перетяните анимацию из вашей папки на нужную строку в скрипте.

Зачем создавать массив GameObject, если можно использовать лист? Зачем перебирать массив с помощью for, а не foreach? Какой вы компонент выбираете на дереве? Он там вообще присвоен?

Очень много очевидных ошибок. Учите язык. Начните с этого: https://professorweb.ru/my/csharp/charp_theory/lev...

Comments

[BadCats]

Теорию теоретически я вроде бы знаю и вот из за того что "вроде бы теоретически" т.е практики ноль - куча очевидных ошибок. Код пишу фактически на практике самостоятельно - первый раз. До этого если посмотрите мои вопросы на Тостере - весь код приводил из уроков по c# Я не хочу никого обидеть, но ведь я думаю у всех первый код - всегда с "очевидными ошибками", которые становятся очевидными только с опытом.

[TheTalion]

BadCats: да, это так, но если вы делаете анимацию объекта через другой объект, то это проблема не кода.

[Даниил Басманов]

TheTalion: В Unity foreach не дотнетовский, а из старой версии моно, где у него корявая имплементация, которая засирает память.

[TheTalion]

Даниил Басманов: а List.ForEach?

[Даниил Басманов]

TheTalion: Он нормальный, но требует Action, который придётся кешировать, если ForEach использовать в каждом кадре.

[TheTalion]

Даниил Басманов: зачем?

[Даниил Басманов]

TheTalion:

using UnityEngine;
using System.Collections.Generic;

public class TestForEach : MonoBehaviour
{
    private List<int> list = new List<int> {1, 2, 3};

    private int count;

    private void Update()
    {
        list.ForEach(i => count += i);
    }
}

Код выше будет создавать новый делегат в каждом кадре, из-за чего будет утекать полкилобайта в секунду на 60 fps.
take.ms/jbNaY

[TheTalion]

Даниил Басманов: это очень странное применение.