10

What's wrong with this code?

var divarray = document.getElementById("yui-main").getElementsByTagName("div");
var articleHTML = array();
var absHTML;
var keyHTML;
var bodyHTML = array();
var i = 0;
for ( var j in divarray) {
    if(divarray[i].className == "articleBody"){
  alert("found");
  articleHTML = divarray[i];
  break;
 }
 bodyHTML[i] = '';
 if(articleHTML[i].className == "issueMiniFeature"){continue;}
 if(articleHTML[i].className == "abstract"){absHTML = articleHTML[i]; continue;}
 if(articleHTML[i].className == "journalKeywords"){keyHTML = articleHTML[i]; continue;}
 bodyHTML[i] = articleHTML[i];
 i++;
}

This is the error I am getting: 

ReferenceError: array is not defined

I am using Google Chrome if it helps any.

Arlen Beiler
  • 14,021
  • 32
  • 88
  • 133

8 Answers8

25

It's not php - you should use

var variable_name = new Array()

or even better

var variable_name = []
Andris
  • 27,029
  • 3
  • 33
  • 37
8

That's not how to declare variables as an empty array. You should be using:

var articleHTML = [];

See this previous question for reasoning of using this method instead of new Array()

Community
  • 1
  • 1
Chad Birch
  • 71,186
  • 23
  • 148
  • 148
2

Note! Javascript IS case sensitive you have to use upper-case A in word Array.

var myarr = new array(); //THIS IS WRONG! and will result in error not defined

So these are the correct ways:

var myarr = new Array(); //THIS IS CORRECT (note the "big" A) :)
var myarr = []; //and this is correct too
jave.web
  • 12,324
  • 10
  • 80
  • 104
2

It's [] in ECMAScript; this isn't PHP. The interpreter is right - array is not defined, which is why you're getting that.

meder omuraliev
  • 177,923
  • 69
  • 381
  • 426
2
var articleHTML = new Array();
Wh1T3h4Ck5
  • 8,295
  • 9
  • 56
  • 77
derek
  • 4,851
  • 1
  • 22
  • 25
1

Instead of 

var articleHTML = array();

and 

var bodyHTML = array();

do

var articleHTML = [];

and

var bodyHTML = [];
D'Arcy Rittich
  • 160,735
  • 37
  • 279
  • 278
1

You first need to define 

var divarray = new Array();
Mark Pieszak - Trilon.io
  • 52,747
  • 13
  • 77
  • 93
Pinu
  • 6,940
  • 15
  • 51
  • 77
0

You also don't need to use var six times, you can do:

var divarray = document.getElementById("yui-main").getElementsByTagName("div"),
    articleHTML = [],
    absHTML = [],
    keyHTML = [],
    bodyHTML = [],
    i = 0;

Which works just as well as your six vars but looks much nicer.

Also there are a number of compelling reasons not to use new in instantiate an array (besides []; is much shorter than new Array();)

IndieInvader
  • 9
  • 1