Welcome to the Treehouse Community
Want to collaborate on code errors? Have bugs you need feedback on? Looking for an extra set of eyes on your latest project? Get support with fellow developers, designers, and programmers of all backgrounds and skill levels here with the Treehouse Community! While you're at it, check out some resources Treehouse students have shared here.
Looking to learn something new?
Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and join thousands of Treehouse students and alumni in the community today.
Start your free trialDiana Anderson
510 PointsThe movement script that I wrote is still not working. Can you help me?
using UnityEngine; using System.Collections;
public class PlayerMovement : MonoBehaviour {
private Animator playerAnimator;
private float moveHorizontal;
private float moveVertical;
public Vector3 movement;
// Use this for initialization
void Start () {
playerAnimator = GetComponent<Animator> ();
}
// Update is called once per frame
void Update () {
moveHorizontal = Input.GetAxisRaw ("Horizontal");
moveVertical = Input.GetAxisRaw ("Vertical");
movement = new Vector3 (moveHorizontal, 0.0f, moveVertical);
}
void FixedUpdate () {
if (movement != Vector3.zero) {
playerAnimator.SetFloat ("Speed", 3f);
} else
{
playerAnimator.SetFloat ("Speed", 0f);
}
}
}
5 Answers
Marcus Gray
1,837 PointsI think you need to add an ending curly brace to the Fixed Update method. So...
playerAnimator.SetFloat ("Speed", 0f); }
}
}
Diana Anderson
510 PointsIt is still not working, can you give me the exact code that should work?
Marcus Gray
1,837 PointsI'm further on in the course and more stuff has been added to the script, so before I do that your movement variable should be 'private' instead of 'public'.
Diana Anderson
510 PointsI fixed it but it is still not working.
Anthony Endrina
1,779 PointsHi Diana, looking at your code I see you are missing a "}" at the end of your code. There should be three curly braces at the end. One for the else statement, one for the Void Update, and one for the entire script. Should look like this at the end:
playerAnimator.SetFloat ("Speed", 0f); } }
}
After you save it, it should work. I had to save a couple of time for the controls to work. Probably, lag from transferring script from the MonoDevelop to the Unity engine software.
Hope this helps.
Jennifer Gerbl
8,772 PointsJennifer Gerbl
8,772 PointsDo you get any compile errors? If yes, what do they say?