Мой персонаж делает неожиданный 3-й прыжок в Unity2D

Question

Привет!

В настоящее время я работаю над 2D-платформерной игрой в движке Unity, где я создал персонажа, который может забить doubleblejump.
В начале все было очень хорошо, но теперь я понял, что есть ошибка в игре, которая позволяет ему выполнить третий прыжок (по какой-то причине). И я не могу ради этого понять, в чем проблема.

PlayerController.cs

using UnityEngine; 
using System.Collections; 

public class PlayerController : MonoBehaviour { 

    /* 
    TODO: Find out why the character ocationally 
    gets 3 jumps instead of 2. 
    I think it's the "isGrounded" that returns 
    a false posetive. 
    */ 

[Header("Ground Recognition")] 
public BoxCollider2D groundCollider; 
public LayerMask groundAbles; 

[Header("Audio")] 
public GameObject jumpSound = null; 

[Header("Visual")] 
public GameObject jumpEffect = null; 


float speed = 5f; 
int maxJumps = 2; 
int currentJumps = 0; 

bool isGrounded = false; 
float boundsLength = 0; 

Vector3 movement; 
// Use this for initialization 
void Start() { 

} 

// Update is called once per frame 
void Update() { 
    CheckGroundCollision(); 
    JumpLogic(); 

    movement = new Vector3(Input.GetAxis("Horizontal") * speed, 0, 0); 
    movement *= Time.deltaTime; 
    transform.position += movement; 

    CheckGroundCollision(); 
} 

void CheckGroundCollision() 
{ 
    isGrounded = groundCollider.IsTouchingLayers(groundAbles); 
} 

void JumpLogic() 
{ 

    if (isGrounded) 
     currentJumps = 0; 

    if (Input.GetButtonDown("Jump") && currentJumps < maxJumps) 
    { 
     GameObject newJumpSound = (GameObject)GameObject.Instantiate(jumpSound, (Vector2)transform.position, transform.rotation); 
     GameObject newJumpEffect = (GameObject)GameObject.Instantiate(jumpEffect, (Vector2)transform.position - new Vector2(0, 0.25f), transform.rotation); 
     GameObject.Destroy(newJumpEffect, 0.2f); 
     GetComponent<Rigidbody2D>().velocity = new Vector2(GetComponent<Rigidbody2D>().velocity.x, 0); 
     GetComponent<Rigidbody2D>().AddForce(new Vector2(0, 10), ForceMode2D.Impulse); 
     currentJumps++; 
     CheckGroundCollision(); 
    } 
} 
} 

Вся помощь ценится!

Answer 1

В void JumpLogic(), вы звоните

currentJumps++; 
CheckGroundCollision(); 

Самый первый раз, когда вы нажмете на прыжок ключ, currentJumps становится 1 (currentJumps ++), но затем isGrounded снова становится истинным (потому что физика не была применена но в текущем фрейме, и персонаж все еще касается наземной прокладки). Таким образом, в следующий раз вызывается JumpLogic(), currentJumps сбрасывается до нуля из-за if (isGrounded) currentJumps = 0; перед запуском кода перехода.

Предлагаю удалить последний звонок на CheckGroundCollision(); внутри void JumpLogic(). Они оба вызваны внутри Update() в любом случае, в порядке, который выглядит так, как будто это сработает.

Я надеюсь, что это поможет!

UPDATE: Я только что заметил, что вы также призывают CheckGroundCollision(); во второй раз в Update() себя. Это также приведет к сбросу переменной перехода.

Попробуйте эту модификацию исходного кода:

void Update() { 
    CheckGroundCollision(); 
    JumpLogic(); 

    movement = new Vector3(Input.GetAxis("Horizontal") * speed, 0, 0); 
    movement *= Time.deltaTime; 
    transform.position += movement; 

    //** NOTE- REMOVED THIS LINE 
} 

... 

void JumpLogic() 
{ 

    if (isGrounded) 
     currentJumps = 0; 

    if (Input.GetButtonDown("Jump") && currentJumps < maxJumps) 
    { 
     GameObject newJumpSound = (GameObject)GameObject.Instantiate(jumpSound, (Vector2)transform.position, transform.rotation); 
     GameObject newJumpEffect = (GameObject)GameObject.Instantiate(jumpEffect, (Vector2)transform.position - new Vector2(0, 0.25f), transform.rotation); 
     GameObject.Destroy(newJumpEffect, 0.2f); 
     GetComponent<Rigidbody2D>().velocity = new Vector2(GetComponent<Rigidbody2D>().velocity.x, 0); 
     GetComponent<Rigidbody2D>().AddForce(new Vector2(0, 10), ForceMode2D.Impulse); 
     currentJumps++; 

     //** NOTE- REMOVED THIS LINE 
    } 
} 

Я надеюсь, что помогает!

Answer 2

Я предлагаю разрешить коллайдеру диктовать игровому контроллеру состояние isGrounded, а не запрашивать коллайдер. Например, продолжая andeart's предложения, ваш код может выглядеть как следующий

void Update() { 
    JumpLogic(); 

    movement = new Vector3(Input.GetAxis("Horizontal") * speed, 0, 0); 
    movement *= Time.deltaTime; 
    transform.position += movement; 
} 

//let the collider indicate to the player controller when 
//a collision occurs, then determine if this collision is relevant 
void OnCollisionEnter2D(Collision2D coll) { 
    if(coll.IsTouchingLayers(groundAbles))  
     isGrounded = true; 

    // or do some other check using layers or tags .... 
} 

void JumpLogic() 
{ 
    if (isGrounded) 
     currentJumps = 0; 

    if (Input.GetButtonDown("Jump") && currentJumps < maxJumps) 
    { 
     GameObject newJumpSound = (GameObject)GameObject.Instantiate(jumpSound, (Vector2)transform.position, transform.rotation); 
     GameObject newJumpEffect = (GameObject)GameObject.Instantiate(jumpEffect, (Vector2)transform.position - new Vector2(0, 0.25f), transform.rotation); 
     GameObject.Destroy(newJumpEffect, 0.2f); 
     GetComponent<Rigidbody2D>().velocity = new Vector2(GetComponent<Rigidbody2D>().velocity.x, 0); 
     GetComponent<Rigidbody2D>().AddForce(new Vector2(0, 10), ForceMode2D.Impulse); 
     currentJumps++; 
    } 
} 

См Collider2D.IsTouchingLayers особенно нижний пункт, который предполагает, что метод IsTouchingLayers не может дать наиболее Acurate результат относительно коллайдера.