Wayback Machinekoobas.hobune.stream
May JUN Jul
Previous capture 12 Next capture
2021 2022 2023
1 capture
12 Jun 22 - 12 Jun 22
sparklines
Close Help
  • Products
  • Solutions
  • Made with Unity
  • Learning
  • Support & Services
  • Community
  • Asset Store
  • Get Unity

UNITY ACCOUNT

You need a Unity Account to shop in the Online and Asset Stores, participate in the Unity Community and manage your license portfolio. Login Create account
  • Blog
  • Forums
  • Answers
  • Evangelists
  • User Groups
  • Beta Program
  • Advisory Panel

Navigation

  • Home
  • Products
  • Solutions
  • Made with Unity
  • Learning
  • Support & Services
  • Community
    • Blog
    • Forums
    • Answers
    • Evangelists
    • User Groups
    • Beta Program
    • Advisory Panel

Unity account

You need a Unity Account to shop in the Online and Asset Stores, participate in the Unity Community and manage your license portfolio. Login Create account

Language

  • Chinese
  • Spanish
  • Japanese
  • Korean
  • Portuguese
  • Ask a question
  • Spaces
    • Default
    • Help Room
    • META
    • Moderators
    • Topics
    • Questions
    • Users
    • Badges
  • Home /
  • Help Room /
avatar image
0
Question by animeman0 · Mar 05, 2016 at 07:23 PM · c#2dmovementrigidbody2d

Player won't stop moving when key is released

As it says in the title player won't stop moving when key is released and I can't figure out why. using UnityEngine; using System.Collections;

 public class Controller : MonoBehaviour {
 
     public float Speed = 5f;
     Rigidbody2D rb;
     // Use this for initialization
     void Start () {
         rb = GetComponent<Rigidbody2D> ();
 
     }
 
     void FixedUpdate () {
 
         if (Input.GetKeyDown (KeyCode.W)) 
         {
             rb.velocity = new Vector2 (rb.velocity.x, Speed);
             if (Input.GetKeyUp (KeyCode.W)) 
             {
                 rb.velocity = new Vector2 (0,0);
             }
         }
         if (Input.GetKeyDown (KeyCode.S)) 
         {
             rb.velocity = new Vector2 (rb.velocity.x, Speed * -1);
             if (Input.GetKeyUp (KeyCode.S)) 
             {
                 rb.velocity = new Vector2 (0,0);
             }
         }
         if (Input.GetKeyDown (KeyCode.A)) 
         {
             rb.velocity = new Vector2 (Speed * -1,rb.velocity.y);
             if (Input.GetKeyUp (KeyCode.A)) 
             {
                 rb.velocity = new Vector2 (0,0);
             }
         }
         if (Input.GetKeyDown (KeyCode.D)) 
         {
             rb.velocity = new Vector2 (Speed, rb.velocity.y);
             if (Input.GetKeyUp (KeyCode.D)) 
             {
                 rb.velocity = new Vector2 (0,0);
             }
         }
     }
 }
Comment
Add comment
10 |3000 characters needed characters left characters exceeded
▼
  • Viewable by all users
  • Viewable by moderators
  • Viewable by moderators and the original poster
  • Advanced visibility
Viewable by all users

1 Reply

· Add your reply
  • Sort: 
avatar image
2
Best Answer

Answer by mahdiii · Mar 05, 2016 at 07:37 PM

you wrote two if into together. It is incorrect if(){ if(){ } } you must write as: if (Input.GetKeyDown (KeyCode.W)) { ... } if (Input.GetKeyUp (KeyCode.W)) { rb.velocity = new Vector2 (0,0); } Also you must do the same work for others.

Comment
Add comment · Show 2 · Share
10 |3000 characters needed characters left characters exceeded
▼
  • Viewable by all users
  • Viewable by moderators
  • Viewable by moderators and the original poster
  • Advanced visibility
Viewable by all users
avatar image rorydriscoll · Mar 05, 2016 at 08:10 PM 0
Share

To be clear, Get$$anonymous$$eyDown fires the event when the key becomes pressed, and Get$$anonymous$$eyUp is fired when it is released. These can't be both true so the inner if can never be hit. Pull out your inner if statements to the FixedUpdate body.

avatar image animeman0 · Mar 05, 2016 at 09:31 PM 0
Share

Thanks both of you!

Your answer

Hint: You can notify a user about this post by typing @username

Up to 2 attachments (including images) can be used with a maximum of 524.3 kB each and 1.0 MB total.

Follow this Question

Answers Answers and Comments

136 People are following this question.

avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image

Related Questions

Why won't my 2D Sprite Move? 1 Answer

Player Movement Not Always Responding 1 Answer

How do I keep enemies (rigidbodies) from pushing each other? 1 Answer

How can I make the character jump between the boxes? 0 Answers

Rotating in Z-Axis while still moving using rigidbody2D.velocity,Rotating in z axis while still moving using rigidbody2d 2 Answers


Enterprise
Social Q&A

Social
Subscribe on YouTube social-youtube Follow on LinkedIn social-linkedin Follow on Twitter social-twitter Follow on Facebook social-facebook Follow on Instagram social-instagram

Footer

  • Purchase
    • Products
    • Subscription
    • Asset Store
    • Unity Gear
    • Resellers
  • Education
    • Students
    • Educators
    • Certification
    • Learn
    • Center of Excellence
  • Download
    • Unity
    • Beta Program
  • Unity Labs
    • Labs
    • Publications
  • Resources
    • Learn platform
    • Community
    • Documentation
    • Unity QA
    • FAQ
    • Services Status
    • Connect
  • About Unity
    • About Us
    • Blog
    • Events
    • Careers
    • Contact
    • Press
    • Partners
    • Affiliates
    • Security
Copyright © 2020 Unity Technologies
  • Legal
  • Privacy Policy
  • Cookies
  • Do Not Sell My Personal Information
  • Cookies Settings
"Unity", Unity logos, and other Unity trademarks are trademarks or registered trademarks of Unity Technologies or its affiliates in the U.S. and elsewhere (more info here). Other names or brands are trademarks of their respective owners.
  • Anonymous
  • Sign in
  • Create
  • Ask a question
  • Spaces
  • Default
  • Help Room
  • META
  • Moderators
  • Explore
  • Topics
  • Questions
  • Users
  • Badges