-1

ここでわずかなc#の問題があります。私は現在、Unityを使用して小さなプラットフォーマーゲームをコーディングしており、衝突などをチェックするレイキャストがいくつかありました。

ここで、これらのレイキャストの結果を整数配列に格納することでコードを少しクリーンアップし始めましたが、IndexOutOfRangeExceptionが発生しています。

コードを何度も読み込もうとしましたが、問題の原因が見つからないようです。誰かが私を助けてくれるなら、私は本当にうれしいです。

これが私のコードです:

using UnityEngine;
using System.Collections;

public class PlayerRayCaster : MonoBehaviour {

    public float playerHeight = 1;
    public enum FeetState {Air, Ground};

    public FeetState playerFeetState = FeetState.Air;
    public int feetHitRays;
    public int behindHitRay;

    //Arrays of rays. value of 1 means that ray hits a target, 0 means that it does not hit.
    public int[] sideRays; // [0-3] = Left side. [4-8] = Right side.
    public int[] depthRays; // [0] = Away from camera. [1] = Towards camera.
    public int[] feetRays;

    public int counter;

    void Start(){
        sideRays = new int[8];
        depthRays = new int[2];
        feetRays = new int[3];

    }
    // Update is called once per frame
    void Update () {
        FeetRays();
        SideRays();
        BehindRay();
    }

    //Rays, which check if the character is bumping into an object, left or right.
    void SideRays(){

        float rayLength = 0.4f;
        counter = 0;

        //Left side rays.
        for(int rayHeight = 0 ; rayHeight>=-4 ; rayHeight-=1 , counter++){
            Debug.Log(sideRays[counter]);
            if(Physics.Raycast(transform.position-new Vector3(0,rayHeight/2,0), new Vector3(-1,0,0),rayLength)){
                sideRays[counter] = 1;
                Debug.DrawRay(transform.position-new Vector3(0,rayHeight/2,0), new Vector3(-rayLength,0,0), Color.green);
            }
            else{
                sideRays[counter] = 0;
                Debug.DrawRay(transform.position-new Vector3(0,rayHeight/2,0), new Vector3(-rayLength,0,0), Color.yellow);
            }
        }

        //Right side rays.
        for(int rayHeight = 0;rayHeight>=-4;rayHeight-=1,counter++){
            if(Physics.Raycast(transform.position-new Vector3(0,rayHeight/2,0), new Vector3(1,0,0),rayLength)){
                sideRays[counter] = 1;
                Debug.DrawRay(transform.position-new Vector3(0,rayHeight/2,0), new Vector3(rayLength,0,0), Color.green);
            }
            else{
                sideRays[counter] = 0;
                Debug.DrawRay(transform.position-new Vector3(0,rayHeight/2,0), new Vector3(rayLength,0,0), Color.yellow);
            }
        }
    }

    //Three rays, which check if the characters feet are on the ground or not.
    void FeetRays(){
        feetHitRays = 0;
        float rayLength = 0.2f;
        //Shoots three rays down from player.
        for(float i=-0.7f;i<=0.7f;i+=0.7f){

            if(Physics.Raycast(transform.position-new Vector3(i/2,0,0), new Vector3(0,-1,0),rayLength)){
                feetHitRays++;
                Debug.DrawRay(transform.position-new Vector3(i/2,0,0), new Vector3(0,-rayLength,0), Color.green);
            }
            else{
                Debug.DrawRay(transform.position-new Vector3(i/2,0,0), new Vector3(0,-rayLength,0), Color.red);
            }
        }

        //Sets the feet state.
        if(feetHitRays==0)
        {
            playerFeetState = PlayerRayCaster.FeetState.Air;
        }
        else{
            playerFeetState = PlayerRayCaster.FeetState.Ground;
        }       
    }
    //Shoots a raycast in z-direction from the character, to check for door access.
    void BehindRay(){

        behindHitRay = 0;
        float rayLength = 2;

        if(Physics.Raycast(transform.position, new Vector3(0,0,1), rayLength)){ //Away from camera
            behindHitRay = 1;
            Debug.DrawRay(transform.position, new Vector3(0,0,rayLength), Color.green);
        }
        if(Physics.Raycast(transform.position, new Vector3(0,0,-1), rayLength)){ // Towards camera.
            behindHitRay = -1;
            Debug.DrawRay(transform.position, new Vector3(0,0,-rayLength), Color.green);
        }
    }
}

これは私に例外を与える行です:

sideRays[counter] = 1;

前もって感謝します。

4

3 に答える 3

5

私はこれが問題だと信じています:

for(int rayHeight = 0 ; rayHeight>=-4 ; rayHeight-=1 , counter++)

基本的にこれを2回実行します。つまり、10回の反復を意味しますが、配列は次のように初期化されます。

sideRays = new int[8];

あなたはよく望むかもしれません:

// Note the > instead of >=
for (int rayHeight = 0; rayHeight > -4; rayHeight--, counter++)

(余談ですが、パブリックフィールドは使用しないことを強くお勧めしますが、それは別の問題です。)

于 2012-02-15T19:29:40.817 に答える
1

境界を確認するfor(int rayHeight = 0;rayHeight>=-4;rayHeight-=1,counter++){ループごとに5回配列にアクセスすることになり、配列に8つの項目があるように定義しました。9と10にアクセスすると、例外がスローされます。

于 2012-02-15T19:34:09.077 に答える
1

サイズが8のsideRayを宣言しますが、ループを一目見ただけで、カウンターが10まで高くなる可能性があることがわかります。

于 2012-02-15T19:30:19.283 に答える