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 trial

Game Development How to Make a Video Game Player Input and Cameras Move the Player with Animation

The 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);
    }


}

}

Do you get any compile errors? If yes, what do they say?

5 Answers

Marcus Gray
Marcus Gray
1,837 Points

I think you need to add an ending curly brace to the Fixed Update method. So...

playerAnimator.SetFloat ("Speed", 0f); }

}

}

It is still not working, can you give me the exact code that should work?

Marcus Gray
Marcus Gray
1,837 Points

I'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'.

I fixed it but it is still not working.

Anthony Endrina
Anthony Endrina
1,779 Points

Hi 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.