What is Wrong With This Script?

Questions & Answers
1

function PlayerAnims()

{

if (PlayerState == 0);
{

PlayerAnimSec.animation.CrossFade("IDLE ANIMATION");

else if (PlayerState == 1);
{

PlayerAnimSec.animation.CrossFade("WALKING ANIMATION");

    }

else if (PlayerState == 2);
{

PlayerAnimSec.animation.CrossFade(“SPRINTING ANIMATION”);

    }

}

Here’s what the console says:

Assets/Player ANIMATION.js(37,60): BCE0044: expecting :, found ‘;’.

Line 37 is the line that says:
PlayerAnimSec.animation.CrossFade(“IDLE ANIMATION”);

It is most likely to do with the closed bracket and the semi-colon but that’s just what the console says. I’m not sure if it is pin-point accurate because I’m not that experienced.
I think that the problem is throughout the other two lines as well which look almost exactly the same as the one mentioned:

PlayerAnimSec.animation.CrossFade("SPRINTING ANIMATION");

AND
PlayerAnimSec.animation.CrossFade(“WALKING ANIMATION”);

I’m relatively new to programming and Unity3d; so the simpler the better,
I’ve definitely posted the codes right but some of the brackets may have been posted in this forum wrongly; but it should be okay.

Any Ideas?
Thanks In Advance!

2

Remove the semicolon in the line that says

if (PlayerState == 0); {

It should say

if (PlayerState == 0) {

And the same in the lines

else if (PlayerState == 1); {

and

else if (PlayerState == 2); {

Should be no semi-colons in any of them.

3

Ok, I will remove the semi-colons again but , Seth what do you mean by I am missing a closing bracket?
P.S: A lot of this is from tutorials

4

Here is the code that you want

function PlayerAnims()
   {
   if (PlayerState == 0)
     {
     PlayerAnimSec.animation.CrossFade("IDLE ANIMATION");
     }
   else if (PlayerState == 1)
     {
     PlayerAnimSec.animation.CrossFade("WALKING ANIMATION");
     }
   else 
     {
     PlayerAnimSec.animation.CrossFade("SPRINTING ANIMATION");
     }
   }

If you find yourself writing lots of if/if else/else blocks like this, consider replacing with a Switch() statement instead. But this should get you started for now.