Как я могу возродить пикапы в массиве и через сопрограмму?

Question

У меня возникают трудности с возвращением моих пикапов из золота после того, как они были уничтожены после смерти.Идея заключается в том, что если игрок не может собрать 5 золотых слитков, активирует контрольную точку и умирает, текущее золото уничтожается и сбрасывается после исчезновения черного экрана.

У меня сейчас есть сопрограммав моем диспетчере здоровья, который работает правильно, если игрок умирает и сбрасывает их.У меня есть скрипт Gold Pickup, который уничтожает золото, если его не забрали.Я просто не могу заставить их воссоздать.Я попытался добавить экземпляр кода в сопрограмму Health Manager и в сценарии Gold Pickup.Ничто не похоже на работу.Если я не получаю сообщения о том, что «индекс массива выходит за пределы диапазона», это «ссылка на объект не установлена ​​на экземпляр объекта» и т. Д.

public class GoldPickup : MonoBehaviour{
    public int value;
    public GameObject pickupEffect;
    public GameObject[] goldBarArray;
    public HealthManager healthManager;
    public Checkpoint checkpoint;

    private Vector3 goldRespawnPoint;
    private Quaternion goldStartPosition;

    void Start()
    {
        //To destroy multiple objects at once, use FindGameObjectsWithTag.
        //GetComponent is considered more efficient than FindObjectOfType, but the latter avoids any errors saying an object reference hasn't been set.
        goldBarArray = GameObject.FindGameObjectsWithTag("Gold");
        healthManager = FindObjectOfType<HealthManager>();
        //FindObjectOfType<Checkpoint>();
        checkpoint = FindObjectOfType<Checkpoint>();
        goldRespawnPoint = transform.position;
        goldStartPosition = transform.rotation;
    }

    public void OnTriggerEnter(Collider other)
    {
        if (other.gameObject.CompareTag("Player"))
        {
            FindObjectOfType<GameManager>().AddGold(value);
            Instantiate(pickupEffect, transform.position, transform.rotation);
            Destroy(gameObject);
        }
    }

    public void DestroyGold()
    {
        //For Statics, an object reference isn't necessary. Use the FindObjectOfType to find the appropriate script and reference the Type, such as HealthManager.
        if (checkpoint.checkpoint1On == false)
        {
            foreach (GameObject Gold in goldBarArray)
            {
                Destroy(Gold);
                Instantiate(goldBarArray[5], goldRespawnPoint, goldStartPosition);
                goldRespawnPoint = transform.position;
                goldStartPosition = transform.rotation;
                //healthManager.RespawnCo();
            }
        }

    }
    /*public void GoldReset()
    {
        if (healthManager.isRespawning == true)
        {
            if (checkpoint.checkpoint1On == false)
            {
                StartCoroutine("GoldRespawnCo");
            }
        }

        else if (_respawnCoroutine != null)
        {
            StopCoroutine(_respawnCoroutine);
            _respawnCoroutine = StartCoroutine("GoldRespawnCo");
        }*/

    /*public IEnumerator GoldRespawnCo()
    {
        if (checkpoint.checkpoint1On == false)
        {
            Instantiate(goldPrefab, goldRespawnPoint, goldStartPosition);
            transform.position = goldRespawnPoint;
            transform.rotation = goldStartPosition;
        }
        else
        {
            yield return null;
        }
    }*/

    /*if (thePlayer.gameObject.activeInHierarchy == false)
    {
        Destroy(gameObject);
        Instantiate(goldBar, transform.position, transform.rotation);
    }
    else
    {
        if (thePlayer.gameObject.activeInHierarchy == true)
        {
            transform.position = respawnPoint;
            transform.rotation = startPosition;
        }
    }*/
}

    public class HealthManager : MonoBehaviour
    //The counters will count down and will keep counting down based on the length variables
    public int maxHealth;
    public int currentHealth;
    public PlayerController thePlayer;
    //public GoldPickup goldPickup;
    //public GoldPickup[] goldPickup;
    public float invincibilityLength;
    public Renderer playerRenderer;
    public float flashLength;
    public float respawnLength;
    public GameObject deathEffect;
    public Image blackScreen;
    public float fadeSpeed;
    public float waitForFade;
    public bool isRespawning;
    //public GameObject goldBar;
    //To reference another script's function, such as in the DeathTrigger script, make a public DeathTrigger, give it a reference name, and put it into the Start function. Use the reference name and assign it using GetComponent. Call another script's method by using the reference name, followed by a dot and the name of the method. Eg: deathTrigger.DestroyGold().

    private Quaternion startPosition;
    //private Quaternion goldPosition;
    private float flashCounter;
    private float invincibilityCounter;
    private Vector3 respawnPoint;
    //private Vector3 goldRespawnPoint;
    private bool isFadetoBlack;
    private bool isFadefromBlack;
    //private Coroutine _respawnCoroutine;
    //private Vector3 goldRespawnPoint;
    //private Quaternion goldStartPosition;

    void Start()
    {
        currentHealth = maxHealth;
        respawnPoint = thePlayer.transform.position;
        startPosition = thePlayer.transform.rotation;
        //goldPickup = GetComponent<GoldPickup>();
        //goldRespawnPoint = goldBar.transform.position;
        //goldStartPosition = goldBar.transform.rotation;
        //goldRespawnPoint = transform.position;
        //goldStartPosition = transform.rotation;
        //goldPickup = FindObjectOfType<GoldPickup>();
        //goldRespawnPoint = goldBar.transform.position;
        //goldPosition = goldBar.transform.rotation;
    }

    void Update()
    {
        //These functions are checked every frame until the player takes damage
        if (invincibilityCounter > 0)
        {
            invincibilityCounter -= Time.deltaTime;
            flashCounter -= Time.deltaTime;
            if (flashCounter <= 0)
            //The Flash Counter is currently set at 0.1 and will be within the 0 region as it counts down. During this period, the playerRenderer will alternate between on and off
            {
                playerRenderer.enabled = !playerRenderer.enabled;
                //The Flash Counter will keep counting down and reloop depending on the Flash Length time
                flashCounter = flashLength;
            }
            //This makes sure after the flashing and invincibility has worn off that the player renderer is always turned back on so you can see the player
            if (invincibilityCounter <= 0)
            {
                playerRenderer.enabled = true;
            }
        }

        if (isFadetoBlack)
        {
            blackScreen.color = new Color(blackScreen.color.r, blackScreen.color.g, blackScreen.color.b, Mathf.MoveTowards(blackScreen.color.a, 1f, fadeSpeed * Time.deltaTime));
            if (blackScreen.color.a == 1f)
            {
                isFadetoBlack = false;
            }
        }
        if (isFadefromBlack)
        {
            blackScreen.color = new Color(blackScreen.color.r, blackScreen.color.g, blackScreen.color.b, Mathf.MoveTowards(blackScreen.color.a, 0f, fadeSpeed * Time.deltaTime));
            if (blackScreen.color.a == 0f)
            {
                isFadefromBlack = false;
            }
        }
    }

    public void HurtPlayer(int damage, Vector3 direction)
    {
        //If the invincibility countdown reaches zero it stops, making you no longer invincible and prone to taking damage again
        if (invincibilityCounter <= 0)
        {
            currentHealth -= damage;
            if (currentHealth <= 0)
            {
                Respawn();
            }

            else
            {
                thePlayer.Knockback(direction);
                invincibilityCounter = invincibilityLength;
                playerRenderer.enabled = false;
                flashCounter = flashLength;
            }
        }
    }

    public void Respawn()
    {
        //A StartCoroutine must be set up before the IEnumerator can begin
        if (!isRespawning)
        {
            StartCoroutine("RespawnCo");
        }
    }

    //IEnumerators or Coroutines will execute the code separately at specified times while the rest of the code in a codeblock will carry on executing as normal.
    //To prevent an error appearing below the name of the Coroutine, be sure to place a yield return somewhere within the code block. Either yield return null or a new WaitForSeconds.
    public IEnumerator RespawnCo()
    {
        if (GameManager.currentGold < 5)
        {
            isRespawning = true;
            thePlayer.gameObject.SetActive(false);
            Instantiate(deathEffect, respawnPoint, startPosition);
            yield return new WaitForSeconds(respawnLength);
            isFadetoBlack = true;
            yield return new WaitForSeconds(waitForFade);
            //To reference another script's function quickly and just the once, use the FindObjectOfType function. This is considered to be slow however.
            FindObjectOfType<GoldPickup>().DestroyGold();
            //GetComponent<GoldPickup>().DestroyGold();
            //Instantiate(goldBar, goldRespawnPoint, Quaternion.identity);
            isFadefromBlack = true;
            //goldRespawnPoint = goldBar.transform.position;
            //goldStartPosition = goldBar.transform.rotation;
            isRespawning = false;
            thePlayer.gameObject.SetActive(true);
            thePlayer.transform.position = respawnPoint;
            thePlayer.transform.rotation = startPosition;
            currentHealth = maxHealth;
            invincibilityCounter = invincibilityLength;
            playerRenderer.enabled = false;
            flashCounter = flashLength;
            GameManager.currentGold = 0;
            GetComponent<GameManager>().SetCountText();
            StopCoroutine("RespawnCo");

            /*isRespawning = true;
            thePlayer.gameObject.SetActive(false);
            yield return new WaitForSeconds(respawnLength);
            isFadetoBlack = true;
            yield return new WaitForSeconds(waitForFade);
            isFadefromBlack = true;
            invincibilityCounter = invincibilityLength;
            playerRenderer.enabled = false;
            flashCounter = flashLength;
            SceneManager.LoadScene("Level 1");
            GameManager.currentGold = 0;*/
        }

        else if(GameManager.currentGold >= 5)
        {
            isRespawning = true;
            thePlayer.gameObject.SetActive(false);
            Instantiate(deathEffect, respawnPoint, startPosition);
            yield return new WaitForSeconds(respawnLength);
            isFadetoBlack = true;
            yield return new WaitForSeconds(waitForFade);
            isFadefromBlack = true;
            isRespawning = false;
            thePlayer.gameObject.SetActive(true);
            thePlayer.transform.position = respawnPoint;
            thePlayer.transform.rotation = startPosition;
            currentHealth = maxHealth;
            invincibilityCounter = invincibilityLength;
            playerRenderer.enabled = false;
            flashCounter = flashLength;
        }
    }

    /*public void HealPlayer(int healAmount)
    {
        currentHealth += healAmount;
        if(currentHealth > maxHealth)
        {
            currentHealth = maxHealth;
        }
    }*/

    public void SetSpawnPoint(Vector3 newPosition)
    {
        respawnPoint = newPosition;
    }


public class Checkpoint : MonoBehaviour

    public HealthManager theHealthManager;
    public Renderer cpRenderer;
    public Renderer postRenderer;
    public SpriteRenderer pcRenderer;
    public Material cpOff;
    public Material cpOn;
    public Material postOff;
    public Material postOn;
    public GameObject[] infoPanels;
    public bool checkpoint1On;

    //Make sure to assign a value to a bool with '=' and in an 'if' statement somewhere in the code to prevent warnings.
    //private bool checkpoint1IsActivated;
    private bool infoPanel1Activated;

    void Start()
    {
        theHealthManager = FindObjectOfType<HealthManager>();
    }

    void Update()
    //Key presses are better handled in the Update function and will recognise keys being pressed once every frame.
    {
        if (checkpoint1On == true)
        {
            if (infoPanel1Activated == false)
            {
                if (Input.GetKeyDown(KeyCode.Space))
                {
                    infoPanels[0].SetActive(true);
                    infoPanel1Activated = true;
                }
            }
            else
            {
                if (infoPanel1Activated == true)
                {
                    if (Input.GetKeyDown(KeyCode.Space))
                    {
                        infoPanels[0].SetActive(false);
                        infoPanel1Activated = false;
                    }
                }
            }
        }
    }

    public void Checkpoint1On()
    {
        cpRenderer.material = cpOn;
        postRenderer.material = postOn;
        pcRenderer.color = new Color(1f, 1f, 1f, 1f);
        checkpoint1On = true;
    }

    //[] makes a variable an Array (a list). The 'foreach' loop will check through all the Checkpoint objects

    //Checkpoint[] checkpoints = FindObjectsOfType<Checkpoint>();

    //For each Checkpoint Array called 'checkpoints', look for 'cp' and turn the others in the list off

    /*foreach (Checkpoint cp in checkpoints)
    {
        cp.CheckpointOff();
    }
    theRenderer.material = cpOn;*/

    public void Checkpoint1Off()
    {
        cpRenderer.material = cpOff;
        postRenderer.material = postOff;
        pcRenderer.color = new Color(1f, 1f, 1f, 5f);
        checkpoint1On = false;
    }

    public void OnTriggerStay(Collider other)
    {
        if (other.gameObject.CompareTag("Player"))
        {
            if (GameManager.currentGold >= 5)
            {
                if (Input.GetKeyDown(KeyCode.Return))
                {
                    theHealthManager.SetSpawnPoint(transform.position);
                    Checkpoint1On();
                    checkpoint1On = true;
                }
            }
            else if (GameManager.currentGold <= 5)
            {
                checkpoint1On = false;
            }
        }
    }

Answer 1

В вашей функции DestroyGold() вы создаете экземпляр золота следующим образом:

foreach (GameObject Gold in goldBarArray)
{
        Destroy(Gold);
        Instantiate(goldBarArray[5], goldRespawnPoint, goldStartPosition);
        goldRespawnPoint = transform.position;
        goldStartPosition = transform.rotation;
        //healthManager.RespawnCo();
}

Но transform.position и transform.rotation получают только положение и поворот текущего объекта (т.е. независимо от того, какой сценарий используется)прикреплен к).Таким образом, вы не только порождаете все золото в одном месте, но и порождаете золото в месте объекта, содержащего ваш сценарий, а не там, где вы на самом деле хотите, чтобы он пошел!

Не зная об объектах многов вашей сцене вот что я могу вам сказать: попробуйте создать Transform[] для хранения мест, где вы хотите возродить золото.Кроме того, убедитесь, что вы назначили goldRespawnPoint и goldStartPosition ДО того, как вы вызовете Instantiate() в цикле foreach.Наконец, просто общий совет: вы никогда не должны использовать variable == true или variable == false в выражении if.Вы можете просто использовать if(variable) или if(!variable), соответственно.Он будет работать точно так же, будучи более читабельным и уменьшая объем кода, который вам нужно написать.

РЕДАКТИРОВАТЬ 1: В ответ на комментарии я добавил конкретные примеры кода для реализацииэти предложения.

Для начала, вы, вероятно, получаете ошибку вне диапазона из-за goldBarArray[5].Поскольку массивы начинаются с индекса 0, вы можете получить доступ только к элементу n-1 в массиве n.Подробнее о том, как это исправить, на следующем шаге.

Теперь для массива Transform.В области, где вы объявляете свои публичные переменные (вверху скрипта), добавьте строку

public Transform[] spawnPoints;

Затем, снова в Unity вы сможете назначить эти точки появления в Инспекторе.

EDIT 2: Кроме того, в цикле foreach вы пытаетесь создать один из золотых слитков со сцены, но они удаляются с помощью оператора Destroy(Gold);.Вместо этого вы должны создать экземпляр из сборного, который не будет уничтожен.Для этого добавьте

public GameObject goldPrefab;

вместе с остальными открытыми переменными.Затем в редакторе создайте сборный блок, перетащив один из золотых слитков из иерархии в папку «Активы».Наконец, задайте для этого префаба значение goldPrefab в Инспекторе.

Теперь вы можете немного очистить цикл foreach.Вы можете избавиться от строк goldRespawnPoint и goldStartPosition, потому что места респауна будут содержаться в массиве Transform, который мы только что создали.Опять же, не зная, как устроена ваша сцена, мне нужно было просто сделать обоснованное предположение о том, что будет работать.Попробуйте этот цикл:

int spawnPointCounter = 0;
foreach(GameObject Gold in goldBarArray){
    Destroy(Gold);
    Transform currentSP = spawnPoints[spawnPointCounter];
    Instantiate(goldPrefab, currentSP.position, currentSP.rotation);
    spawnPointCounter++;
}