"Unity3D" 索引超出范围异常：数组索引超出范围（尝试在命中时更改 sprite）

Question

早上好开发者我知道这个问题之前已经被问过但是我没有在这些答案中找到解决我的问题的方法，首先我想告诉你我只是一个统一的初学者： p ，这是我的问题：我正在构建一个打砖块游戏，我想做的是在砖块被球击中时改变它的精灵，为此我正在使用这个脚本 :

 public int maxHits;
 public int timesHit;
 private LevelManager levelManager;
 public Sprite[] hitSprites;

 void Start () {
     timesHit = 0;
     levelManager = GameObject.FindObjectOfType<LevelManager> ();
 }

 void OnCollisionEnter2D(Collision2D collision) {
     print ("collison");
     timesHit++;
 }

     // Update is called once per frame
 void Update () {
     if (timesHit >= maxHits) {
         Destroy (gameObject);
     } else {
         LoadSprite ();
     }
     }

 void LoadSprite(){
         int spriteIndex = timesHit - 1;
         this.GetComponent<SpriteRenderer> ().sprite = hitSprites     [spriteIndex];
     }
 }

但我收到此错误：IndexOutOfRangeException: Array index is out of range. Brick.LoadSprite () (at Assets/Scripts/Brick.cs:34)

而且我每一帧都得到它！，所以它大大降低了游戏场景的速度，我无法再测试我的游戏了。你能告诉我我做错了什么以及如何解决它吗？如果你指导我学习一门课程以更多地了解我的错误并且不再重蹈覆辙，这可能会有所帮助。

Answer 1

发生这种情况是因为你的命中次数从 0 开始，然后当你加载一个精灵时你减去 1 给你 -1。在 LoadSprite 的下一行中，您然后尝试访问位于索引 -1 (hitSprites[-1]) 的 hitSprites 数组，这当然是越界的。

至少，我会添加一些验证来检查索引的范围。

Answer 2

我不知道你在做什么，但你收到了那个错误，因为你正在访问 >= hitSprites 长度的索引。

您应该在使用前检查 spriteIndex 变量是否小于 hitSprites 长度。

您的新 LoadSprite 函数应该是：

void LoadSprite()
{
    int spriteIndex = 0;

    //Don't decrement if timesHit  is 0
    if (timesHit > 0)
    {
        spriteIndex = timesHit - 1;
    }
    else
    {
        spriteIndex = timesHit;
    }

    //Return/Exit function if spriteIndex  is equals or more than hitSprites length
    if (spriteIndex > hitSprites.Length - 1)
    {
        return; 
    }
    this.GetComponent<SpriteRenderer>().sprite = hitSprites[spriteIndex];
}

Answer 3

您需要限制 spriteIndex 不小于 0（第一个数组索引）和超过 hitSprites.Length 减少 1（最后一个数组索引）。

void LoadSprite(){
         int spriteIndex = Mathf.Clamp(timesHit - 1, 0, hitSprites.Length - 1);
         this.GetComponent<SpriteRenderer> ().sprite = hitSprites [spriteIndex];
     }
 }

提示：不要在 Update 中调用太多的 GetComponent.. 在唤醒状态下预取所有组件供以后使用。

private SpriteRenderer _spriteRenderer;

void Awake()
{
_spriteRenderer = GetComponent<SpriteRenderer> ();
}

Answer 4

@程序员@Dan-Cook @user2867426 拳头，我要感谢大家的回答，我从你们那里学到了很多东西，非常感谢！好吧，我有点错误地修正了我的错误哈哈。当我优化我的代码时，我注意到我在 update() 中使用了这个 if 条件，而我可以直接将它用于 OnCollisionEnter2D():

if (timesHit >= maxHits) {
     Destroy (gameObject);
 } else {
     LoadSprite ();
 }

不知怎么解决了我的问题，我不知道如何解决，也许问题是它被称为每一帧，现在不是了！如果你们能向我解释如何从 update() 移动 if 条件来解决问题，我真的很感激 ^^ 再次感谢大家。

"Unity3D" 索引超出范围异常：数组索引超出范围（尝试在命中时更改 sprite）

"Unity3D" Index Out Of Range Exception: Array index is out of range (trying to change sprite when hit)

c#

arrays

unity3d

unityscript

unity5