Unity Spain

using System.Collections;
using System.Collections.Generic;
using UnityEngine;
public class ControladorPersonaje : MonoBehaviour
{
    Rigidbody2D rigi;
    public float fuerzaSalto=100f;
    private bool enSuelo=true;
    public Transform comprobadorSuelo;
    float comprobadorRadio=0.07f;
    public LayerMask mascaraSuelo;
    private bool dobleSalto=false;
    private Animator animator;
    private bool corriendo=false;
    public float velocidad=1f;
    private AudioSource sonido;
 
        // Start is called before the first frame update
    void Start()
    {
    }
    void Awake()
    {
        rigi=GetComponent<Rigidbody2D>();
        animator=GetComponent<Animator>();
        sonido = GetComponent<AudioSource>();
    }
    void FixedUpdate()
    {
        //Si está parado
        if (corriendo){
            //Aplicar velocidad en el eje X
            rigi.velocity= new Vector2(velocidad,rigi.velocity.y);
            animator.SetFloat("velX",rigi.velocity.x);
        }
        //Comprobamos si comprobadorSuelo+Radio tocan con los objetos 
        //que pertenezcan a mascaraSuelo
        enSuelo=Physics2D.OverlapCircle(comprobadorSuelo.position,comprobadorRadio,mascaraSuelo);
        
        animator.SetBool("isGrounded",enSuelo);
        if (enSuelo){
            dobleSalto=false;
        }
    }
    // Update is called once per frame
    void Update()
    {
        if (Input.GetMouseButtonDown(0)){
            if(corriendo){
                if ((enSuelo || !dobleSalto)){
                    sonido.Play();
                    //Empujón hacia arriba
                    //Opción 1. Metodo up de Vector2
                    rigi.AddForce(Vector2.up*fuerzaSalto);
                    //Otra opción
                    //Aplicamos una fuerza de 0 en X y de fuerzasalto en Y
                    //rigi.AddForce(new Vector2(0,fuerzaSalto));
                    if(!dobleSalto && !enSuelo){
                        dobleSalto=true;
                    }
                }
            }
            else{
                corriendo=true;
                NotificationCenter.DefaultCenter().PostNotification(this, "PersonajeEmpiezaACorrer");
            }
        }
        if (Input.GetMouseButtonDown(1)){
            
            if (corriendo){
             
            }
            else{
                rigi.velocity = new Vector2(velocidad * 1.25f, rigi.velocity.y);
            }
            
        }
        
    }

Cita de: Rootet date=1566753372Buenas a todos, estoy con un juego de plataforma en 2D, recién empezando. Necesito añadir algo en el script de "Controlador del personaje" para que al pulsar el botón derecho del ratón, aumente la velocidad del personaje *1.25. He añadido lo siguiente, pero no sé si está bien. Aunque no me da error de compilación, al probar en el juego no noto gran diferencia entre pulsar el botón izquierdo o derecho. Quizás es porque debo aumentar también la velocidad de la animación "Run"??¿cómo se hace? desde el componente Animator en el código?
   
   
      Os dejo el Script aquí por si alguien puede echar un vistazo. Gracias
   
   
      using System.Collections;<br>using System.Collections.Generic;<br>using UnityEngine;<br>public class ControladorPersonaje : MonoBehaviour<br>{<br>    Rigidbody2D rigi;<br>    public float fuerzaSalto=100f;<br>    private bool enSuelo=true;<br>    public Transform comprobadorSuelo;<br>    float comprobadorRadio=0.07f;<br>    public LayerMask mascaraSuelo;<br>    private bool dobleSalto=false;<br>    private Animator animator;<br>    private bool corriendo=false;<br>    public float velocidad=1f;<br>    private AudioSource sonido;<br> <br>        // Start is called before the first frame update<br>    void Start()<br>    {<br>    }<br>    void Awake()<br>    {<br>        rigi=GetComponent<Rigidbody2D>();<br>        animator=GetComponent<Animator>();<br>        sonido = GetComponent<AudioSource>();<br>    }<br>    void FixedUpdate()<br>    {<br>        //Si está parado<br>        if (corriendo){<br>            //Aplicar velocidad en el eje X<br>            rigi.velocity= new Vector2(velocidad,rigi.velocity.y);<br>            animator.SetFloat("velX",rigi.velocity.x);<br>        }<br>        //Comprobamos si comprobadorSuelo+Radio tocan con los objetos <br>        //que pertenezcan a mascaraSuelo<br>        enSuelo=Physics2D.OverlapCircle(comprobadorSuelo.position,comprobadorRadio,mascaraSuelo);<br>        <br>        animator.SetBool("isGrounded",enSuelo);<br>        if (enSuelo){<br>            dobleSalto=false;<br>        }<br>    }<br>    // Update is called once per frame<br>    void Update()<br>    {<br>        if (Input.GetMouseButtonDown(0)){<br>            if(corriendo){<br>                if ((enSuelo || !dobleSalto)){<br>                    sonido.Play();<br>                    //Empujón hacia arriba<br>                    //Opción 1. Metodo up de Vector2<br>                    rigi.AddForce(Vector2.up*fuerzaSalto);<br>                    //Otra opción<br>                    //Aplicamos una fuerza de 0 en X y de fuerzasalto en Y<br>                    //rigi.AddForce(new Vector2(0,fuerzaSalto));<br>                    if(!dobleSalto && !enSuelo){<br>                        dobleSalto=true;<br>                    }<br>                }<br>            }<br>            else{<br>                corriendo=true;<br>                NotificationCenter.DefaultCenter().PostNotification(this, "PersonajeEmpiezaACorrer");<br>            }<br>        }<br>        if (Input.GetMouseButtonDown(1)){<br>            <br>            if (corriendo){<br>             <br>            }<br>            else{<br>                rigi.velocity = new Vector2(velocidad * 1.25f, rigi.velocity.y);<br>            }<br>            <br>        }<br>        <br>    }
   

using System.Collections;
using System.Collections.Generic;
using UnityEngine;
public class ControladorPersonaje : MonoBehaviour
{
    Rigidbody2D rigi;
    public float fuerzaSalto=100f;
    private bool enSuelo=true;
    public Transform comprobadorSuelo;
    float comprobadorRadio=0.07f;
    public LayerMask mascaraSuelo;
    private bool dobleSalto=false;
    private Animator animator;
    private bool corriendo=false;
    public float velocidad=1f;
    private AudioSource sonido;
 
        // Start is called before the first frame update
    void Start()
    {
    }
    void Awake()
    {
        rigi=GetComponent<Rigidbody2D>();
        animator=GetComponent<Animator>();
        sonido = GetComponent<AudioSource>();
    }
    void FixedUpdate()
    {
        //Si está parado
        if (corriendo){
            //Aplicar velocidad en el eje X
            rigi.velocity= new Vector2(velocidad,rigi.velocity.y);
            animator.SetFloat("velX",rigi.velocity.x);
        }
        //Comprobamos si comprobadorSuelo+Radio tocan con los objetos 
        //que pertenezcan a mascaraSuelo
        enSuelo=Physics2D.OverlapCircle(comprobadorSuelo.position,comprobadorRadio,mascaraSuelo);
        
        animator.SetBool("isGrounded",enSuelo);
        if (enSuelo){
            dobleSalto=false;
        }
    }
    // Update is called once per frame
    void Update()
    {
        if (Input.GetMouseButtonDown(0)){
            if(corriendo){
                if ((enSuelo || !dobleSalto)){
                    sonido.Play();
                    //Empujón hacia arriba
                    //Opción 1. Metodo up de Vector2
                    rigi.AddForce(Vector2.up*fuerzaSalto);
                    //Otra opción
                    //Aplicamos una fuerza de 0 en X y de fuerzasalto en Y
                    //rigi.AddForce(new Vector2(0,fuerzaSalto));
                    if(!dobleSalto && !enSuelo){
                        dobleSalto=true;
                    }
                }
            }
            else{
                corriendo=true;
                NotificationCenter.DefaultCenter().PostNotification(this, "PersonajeEmpiezaACorrer");
            }
        }
        if (Input.GetMouseButtonDown(1)){
            
            if (corriendo){
             
            }
            else{
                rigi.velocity = new Vector2(velocidad * 1.25f, rigi.velocity.y);
            }
            
        }
        
    }

   
       
   
   
      ControladorPersonaje.cs 2.51 kB · 0 downloads  
   

Hola Rootet.


Mirando el código, veo que hay un problema con tu actualización de la velocidad.


En el Update, estás mirando si pulsas el botón del ratón, entonces actualizas la velocidad a "velocidad * 1.25f", pero también, una instrucción similar está siendo llamada en el siguiente frame y poseriores, en FixedUpdate, y esa está utilizando sólo "velocidad".


No he probado tu código, pero creo, que esto puede llevarte al problema de que el cambio de velocidad no se llega a ejecutar realmente, que lo que ves al moverse, puede que sea el resultado del FixedUpdate. Tal y como lo veo yo (un poco por encimilla @_@) 
 

• 
     El personaje empieza quieto, el rigidbody no se mueve (ni en FixedUpdate ni en la llamada del ratón).
  


• 
     Se pulsa el botón del ratón, el personaje sigue sin moverse en ese frame, como la variable corriendo era falsa, el personaje no se mueve todavía, pero la variable "corriendo" se activa.
  


• 
     En el siguiente frame, la variable corriendo ya está activada, así que el velocity del FixedUpdate ya está en movimiento, pero el velocity del ratón ya no se activa, porque la función "GetMouseButtonDown" sólo da verdad en el frame (o ciclo) en que el botón pasa de no estar pulsado a estar pulsado, (que era el ciclo anterior por el que se activó la variable corriendo, pero ahora en este nuevo ciclo, el botón ya está pulsado y sigue pulsado, luego GetMouseButtonDown vuelve a ser falso).
  

De modo que si el cambio de velocidad venía por esa línea, creo que no funcionará.
 
 
 
Una manera rápida de arreglarlo es en vez de usar GetMouseButtonDown, utilizar simplemente GetMouseButton, su función es la misma, sólo que esta siempre se activa mientras el botón esté siendo pulsado, mientras que GetMouseButtonDown y GetMouseButtonUp, sólo se activan en los momentos de transición. (Son bastante útiles, pero quizás no en esta situación).
 
 
 
Otra posibilidad, es cambiar el valor de la variable velocidad cuando haces la activación de la variable corriendo.
 
 
 
Tutorial arriba, tutorial abajo, me parece que es buena idea intentar hacerlo por ti mismo, es una buena práctica para aprender a resolver y racionalizar los problemas. Ya habrá tiempo de pulirlo.

@Rootet en tu script hay cositas que están bien otras que están mal, por ej:
 
1) Update para inputs, FixedUpdate para físicas:
 

if (Input.GetMouseButtonDown(1)){
            
            if (corriendo){
             
            }
            else{
                rigi.velocity = new Vector2(velocidad * 1.25f, rigi.velocity.y);
            }
            
        }

 
En update, estás modificando una velocidad, cosa que debés hacer en FixedUpdate, esto se lee como "Si apreto el botón derecho y corriendo es false la velocidad X es el 25% más grande" y esta actualización sucede en un solo cuadro, no cuadro a cuadro. Además si la velocidad es un 25% más grande se supone que corriendo ahora debe ser true pero no, se setea true si apretás el otro botón (en FixedUpdate).
 
─────────────────────────────────────────────────────────────────────────────────────────────────────
 
2) Quién es responsable de corriendo?
 

if (Input.GetMouseButtonDown(0)){
            if(corriendo){
                if ((enSuelo || !dobleSalto)){
                    sonido.Play();
                    //Empujón hacia arriba
                    //Opción 1. Metodo up de Vector2
                    rigi.AddForce(Vector2.up*fuerzaSalto);
                    //Otra opción
                    //Aplicamos una fuerza de 0 en X y de fuerzasalto en Y
                    //rigi.AddForce(new Vector2(0,fuerzaSalto));
                    if(!dobleSalto && !enSuelo){
                        dobleSalto=true;
                    }
                }
            }
            else{
                corriendo=true;
                NotificationCenter.DefaultCenter().PostNotification(this, "PersonajeEmpiezaACorrer");
            }
        }

 
Cuando apretás el botón izq y no está corriendo se da la primer vuelta (el else), es decir que se pone corriendo = true. En el siguiente cuadro GetMouseButtonDown no se dispara (por razones obvias) entoces el "if(corriendo)" no se da. Sí se va a dar el "if(corriendo)" en el proximo click. Además, ¿quién pone corriendo en false?
 
Y volviendo al punto 1, solo si corriendo es falso la velocidad aumenta. Una vez que diste click izquierdo corriendo nunca vuelve a false (?). Puede ser que me haya olvidado de algo, muchas veces contesto a la ligera.
 
─────────────────────────────────────────────────────────────────────────────────────────────────────
 
█████████████████████████████████████████████████████████████████████████████████████████████████████
 
Podés resolver el problema parcheando if's y demás, pero te diría que de entrada la estructura es muy confusa para una tarea tan simple, tomalo como una oportundad de mejora y experimentá con una nueva, que tenga más lógica y sobretodo que alguien externo al código lo vea y lo entienda al instante, pensá que esto es un salto y una funcionalidad de correr, practicamente no es nada.
 
La lógica que te recomiendo para una mezcla Update + FixedUpdate en un platformer (y con cualquier personaje) es la siguiente (seguramente hay cosas que no definí, solo una idea muy por arriba):
 

// inputs - esto expandilo a todas las inputs que estás esperando, lo mismo con ejes
// En este ejemplo está solo correr "Run" (que elegí la leftShift)
bool runPressed = false;
bool runReleased = false;
bool running = false;
bool isGrounded = false;
 
void Update()
{
// Update puede pasar 100 veces más rápido que FixedUpdate (50 fps por default), así que 
// todos los KeyDown se acumulan, no se setean, así no perdés ninguno en el camino
// por acumular me refiero a usar una simple OR
runPressed |= Input.GetKeyDown( KeyCode.LeftShift );
runReleased |= Input.GetKeyUp ( KeyCode.LeftShift );
  
  
  

}
void FixedUpdate()
{
//Detectar suelo, determinar direcciones de movimiento, ángulos, etc
// Por supuesto acá es donde decís "suelo = true o false"
isGrounded = IsGrounded();
  
     // el vector objetivo cuadro a cuadro, no necesariamente el aplicado
     Vector2 targetVelocity = Vector2.Zero;
  
// bien general un "if( suelo ) ....
if( isGrounded )
    {
   //Todo lo que pasa si y solo si el jugador está grounded
   // en este caso la input a evaluar es run así que 
   if( runPressed )
        {
           running = true;
             targetVelocity.x = ...
        }
         else if( runReleased )
        {
           running = false;
             targetVelocity.x = ...
        }
        
         
    }
    else   //Todo lo que pasa si y solo si el jugador NO está grounded
    {
         //en el aire no puede correr
         running = false;
         
         targetVelocity.y = ...
   
    }
     //Una vez determinado el estado se aplica el vector obtenido a la velocidad (de la forma que quieras).
     rb.velocity = ...(dependiendo de targetVelocity)

// IMPORTANTE --> Resetear inputs, FixedUpdate ya las usó
runPressed = false;
runReleased = false;

}

 
 
 
 
 

Cita de: Rootet date=1566753372Quizás es porque debo aumentar también la velocidad de la animación "Run"?
   

Animación y movimiento poco tienen que ver, si no ves los efectos en pantalla tratá de exagerar los valores, en vez de 1.25f poné 5f. Igualmente creo que sí, desde el animator podés setear la velocidad del clip (revisá las referencias de Animator o google más rápido), aunque no recuerdo, estoy completamente seguro que para el sistema Legacy (viejo) se puede hacer, pero con el componente Animation.
 
 
 
Saludos

La solución de @lightbug es la que uso yo tal cual. A veces incluso uso un script aparte que se dedique exclusivamente a leer inputs (InputManager o algo así) y, luego, cualquier otro script que necesite saber acerca de esos inputs, use una referencia a este (InputManager).

Eso sí, lo de usar OR al asignar los inputs para acumularlos entre cada FixedUpdate; excelente añadido que me voy a copiar a partir de ahora.  Gracias por el aporte.

Cita de: Deaymon date=1566984927La solución de @lightbug es la que uso yo tal cual. A veces incluso uso un script aparte que se dedique exclusivamente a leer inputs (InputManager o algo así) y, luego, cualquier otro script que necesite saber acerca de esos inputs, use una referencia a este (InputManager).

      Eso sí, lo de usar OR al asignar los inputs para acumularlos entre cada FixedUpdate; excelente añadido que me voy a copiar a partir de ahora.  Gracias por el aporte.
   

Exacto, en realidad un script separado para inputs es lo ideal (separado del movimiento). Lo de las OR se lo rob... digo tome prestado a los de Unity , de la serie del training day de 2D (excelente por cierto, va desde personaje hasta audio):
 
Create A 2D Platformer (//%5Byoutube%5Dhttps://www.youtube.com/watch?v=j29NgzV8Dw4&amp;list=PLX2vGYjWbI0REfhDHPpdIBjjrzDHDP-xT%5B/youtube%5D)

Cita de: lightbug date=1566959924Cuando apretás el botón izq y no está corriendo se da la primer vuelta (el else), es decir que se pone corriendo = true. En el siguiente cuadro GetMouseButtonDown no se dispara (por razones obvias) entoces el "if(corriendo)" no se da. Sí se va a dar el "if(corriendo)" en el proximo click. Además, ¿quién pone corriendo en false?
   
   
      Y volviendo al punto 1, solo si corriendo es falso la velocidad aumenta. Una vez que diste click izquierdo corriendo nunca vuelve a false (?). Puede ser que me haya olvidado de algo, muchas veces contesto a la ligera.
   

@lightbug, sí efectivamente. Solo si está corriendo puede aumentar la velocidad y nunca vuelve a false, hasta que se cae o toca una bomba (fin de juego). Muchas gracias por la ayuda, lo del script para leer inputs es una gran idea.
 

Cita de: Antonio date=1566851072Una manera rápida de arreglarlo es en vez de usar GetMouseButtonDown, utilizar simplemente GetMouseButton, su función es la misma, sólo que esta siempre se activa mientras el botón esté siendo pulsado, mientras que GetMouseButtonDown y GetMouseButtonUp, sólo se activan en los momentos de transición. (Son bastante útiles, pero quizás no en esta situación).
   
   
       
   
   
      Otra posibilidad, es cambiar el valor de la variable velocidad cuando haces la activación de la variable corriendo.
   
   
       
   
   
      Tutorial arriba, tutorial abajo, me parece que es buena idea intentar hacerlo por ti mismo, es una buena práctica para aprender a resolver y racionalizar los problemas. Ya habrá tiempo de pulirlo.
   

Gracias Antonio, en ello estoy. El ejercicio me pide que use el GetMouseButtonDown. Ya casi lo tengo. Muchas gracias por el aporte
 

Cita de: Deaymon date=1566984927La solución de @lightbug es la que uso yo tal cual. A veces incluso uso un script aparte que se dedique exclusivamente a leer inputs (InputManager o algo así) y, luego, cualquier otro script que necesite saber acerca de esos inputs, use una referencia a este (InputManager).

      Eso sí, lo de usar OR al asignar los inputs para acumularlos entre cada FixedUpdate; excelente añadido que me voy a copiar a partir de ahora.  Gracias por el aporte.
   

Si, es muy buena solución, aunque pienso que nos piden algo mucho más sencilo. Gracias por contestar 
 
Buenas a todos, al final lo solucioné de la siguiente forma:
 
En la parte del Update solo añadí lo siguiente:
 

void Update()
    {
        if (Input.GetMouseButtonDown(0)){
            if(corriendo){
                if ((enSuelo || !dobleSalto)){
                    sonido.Play();
                    //Empujón hacia arriba
                    //Opción 1. Metodo up de Vector2
                    rigi.AddForce(Vector2.up*fuerzaSalto);
                    //Otra opción
                    //Aplicamos una fuerza de 0 en X y de fuerzasalto en Y
                    //rigi.AddForce(new Vector2(0,fuerzaSalto));
                    if(!dobleSalto && !enSuelo){
                        dobleSalto=true;
                    }
                }
            }
            else{
                corriendo = true;
                NotificationCenter.DefaultCenter().PostNotification(this, "PersonajeEmpiezaACorrer");
            }
        }
        if (Input.GetMouseButtonDown(1)){
            velocidad *= 1.25f;
            animator.speed *= 1.25F;
            if (corriendo){
             
            }
            else{
                rigi.velocity = new Vector2(velocidad * 1.25f, rigi.velocity.y);
                
            }
            
        }
        
    }

 
Al componente Animator, también le añado el mismo aumento de velocidad y ya funciona y se nota más al pulsar el botón derecho. Era sencillo, una vez que ves la lógica bien. Aunque m e quedo con lo del script para los imputs ;) Muchas gracias a los que habéis dedicado unos minutos en ayudarme.