Javascript bool variable is out of scope

Question

I try to declare bool variable and then check it, inside function, but I get error that there is unexpected identifier pointing to line where daysCreated is comparing to false

<script type="text/javascript">

var daysCreated = false;

function createDays() {
 if daysCreated == false {
    //do something
    }
    daysCreated = true;
  }
}

function createDays is being called on button click inside document.

This is not valid syntax. The condition in the `if` needs to be wrapped in brackets. — Nelewout, Jun 11 '17 at 14:56
Possible duplicate of [javascript if statement syntax (need help)](https://stackoverflow.com/questions/37758365/javascript-if-statement-syntax-need-help) — Michelangelo, Jun 11 '17 at 14:58
I would likely have passed the parameter to the function rather than working with a global `daysCreated` - and returned that "internal" value, keeps the code a bit more modular perhaps. — Mark Schultheiss, Jun 11 '17 at 15:08

score 4 · Answer 1 · answered Jun 11 '17 at 14:56

You forgot your parenthesis.

function createDays() {
 if (daysCreated === false) {
    //do something
  }
  daysCreated = true;
}

Also, you have a stray closing curly brace, and it's probably better to check for strict equality (e.g. ===).

Using a linter will catch things like this out of the gate. Here's a tutorial on using ESLint, for example.

score 1 · Answer 2 · answered Jun 11 '17 at 14:56

1

This has nothing to do with scope.

The if test must be between parenthesis: if (condition)

if (daysCreated === false) {

answered Jun 11 '17 at 14:56

Quentin

914,110
126
1,211
1,335

score 0 · Answer 3 · answered Jun 11 '17 at 14:57

0

This is not valid. You need brackets for your if statement like this:

function createDays() {
 if (daysCreated == false) {
    //do something
  }
  daysCreated = true;
}

answered Jun 11 '17 at 14:57

Tom el Safadi

6,164
5
49
102

score 0 · Answer 4 · answered Jun 11 '17 at 14:57

0

You have a typo

function createDays() {
   if (daysCreated === false) {
       //do something
   }
   daysCreated = true;
}

There was a stray {

answered Jun 11 '17 at 14:57

Jonathan Bartlett

512
4
12

Javascript bool variable is out of scope

4 Answers4