I keep getting the response "Unexpected Token else" to this code. Does anyone have a suggestion?

Question

0.00/5 (No votes)

See more:

function myChildren(name){
if (name = "Dave")
return "Oldest"
if (name = "Michelle")
return "Middle";
}

else {
return "Not my kid!";
}

Posted 14-Dec-15 8:32am

Heschel

Add a Solution

Comments

Richard Deeming 14-Dec-15 14:41pm

You're missing a semi-colon on the return "Oldest" line, but Javascript is relatively lax about that.

You're also missing a closing brace (}) on the else block, and the opening brace ({) on the preceding if block.

You don't really need the else {, since your previous if block has an unconditional return.

2 solutions

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

W∴ Balboos, *GHB* · Answer 1 · 2015-12-14T08:52:00

Solution 1

Looks to me like your else block is OUTSIDE of your function.

That and the missing semicolon noted above by RD.

Posted 14-Dec-15 8:52am

W∴ Balboos, GHB

phil.o · Answer 2 · 2015-12-14T08:57:00

I think this code block should better read:

JavaScript

function myChildren(name) {
   if (name == "Dave") {
      return "Oldest";
   }
   else if (name == "Michelle") {
      return "Middle";
   }
   else {
      return "Not my kid!";
   }
}

Identing your code correctly is the first thing to do when you want to quickly detect misplaced braces.
You should also try to be consistent with the way you are using if blocks. For the two first ones, you do not use braces, whereas you use them for the else part.
I also corrected your equality tests; as far as I know, conditional operator in javascript is ==, not =. = is the assignment operator.

Hope this helps.