Both if and else statements running JavaScript

Question

When i press 1 to go to "Start" it runs start BUT it also runs "instructions". So after it says "Welcoe22222" it would say "HAHAHAH" right after it but i only want the "start" function.

<script type="text/javascript">


alert("Welcome to the BattleShip Game!!!")
name = prompt("What is your name: ")
alert("Welcome "+name)
option = prompt("1. Start Game \n \n 2. Instructions \n \n Select: ")
if (option ==1)
    start()
else (option ==2)
    instructions()

function start() {

    alert("Welcoe22222")

}

function instructions() {

    alert("HAHAHAH")

}

</script>

This is why you should use semicolons, at least until you fully understand JS syntax. — SLaks
– SLaks, Commented Dec 21, 2016 at 20:25
@Saiid: No; I mean semicolons (although braces would also help). You can't rely on ASI until you fully understand how JS syntax works. — SLaks
– SLaks, Commented Dec 21, 2016 at 20:27
The semicolons aren't the problem, though. If they had used the brackets in the if/else, it wouldn't be doing the same thing. — TrojanByAccident
– TrojanByAccident, Commented Dec 21, 2016 at 20:29

SLaks · Accepted Answer · 2016-12-21 20:25:03Z

4

Your else syntax is completely wrong.

If you want else if, you need to write that.

Your code is parsed using automatic semicolon insertion as

if (option ==1)
    start()
else
    (option ==2);
instructions()

answered Dec 21, 2016 at 20:25

SLaks

891k182 gold badges1.9k silver badges2k bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Nina Scholz · Accepted Answer · 2016-12-21 20:25:06Z

0

You are missing an if

else if (option == 2)
//   ^^

Better to use block statement { /* ... */ }, like

if (option == 1) {
    start();
} else if (option == 2) {
    instructions();
}

answered Dec 21, 2016 at 20:25

Nina Scholz

388k26 gold badges367 silver badges417 bronze badges

Comments

Mohammad Kermani · Accepted Answer · 2016-12-26 10:09:08Z

0

First I think for not getting confusing, or making your teammates confused about your code, it is better to use block statement (I mean if you think in the future you may want to add more lines of codes in the if statement.

You can't have parentheses after else. if you sayelse (x==y)`, you will get the error:

Uncaught SyntaxError: Unexpected token {

It should be:

if (option == 1) {
    start();
} else if (option == 2) {
    instructions()
}

answered Dec 26, 2016 at 10:09

Mohammad Kermani

5,4368 gold badges39 silver badges63 bronze badges

Collectives™ on Stack Overflow

Both if and else statements running JavaScript

3 Answers 3

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related