Code quits before creating a 2-d array

[u/wbport1]

I am trying to add a feature to a webpage that simulates an election. "bordaTally" is defined as new Array(); in the second copied text below. "alert 0" is the only one that shows.

Thanks in advance.

function bordaItem(name) {

this.name = name;

this.total = 0;

this.tally = new array();

}

  if (document.tournform.rr2[4].checked == true) { //Borda count
  for (i=0;i < n; i++) {
alert("alert 0");
    bordaTally[i] = new bordaItem(i);
alert("alert 1");
for (j=0; j < n;j++) {
  alert("alert 2");
  bordaTally[i][j].tally = 0; }
}
  for (i=0; i < totBallots; i++)
    for (j=0; j < n; j++)
  break;     //placeholder for now
    
  }   //End Borda Count

[u/CelebrityChess]

the other thing that'll bite you once the array typo is fixed is the line inside the loop, bordaTally[i][j].tally = 0. bordaTally[i] is a bordaItem object, so it has no [j] and that will throw a "cannot set properties of undefined" error as soon as it runs. you want bordaTally[i].tally[j] = 0 instead. you could also just build it up front in the constructor with new Array(n).fill(0) so you don't need the inner loop at all. and if the alerts stop showing again, open the browser console with F12 because the error message there will point at the exact line.

[u/wbport1 → u/CelebrityChess]

Thank you--moving the [j] to tally performed as advertised. I don't know how many tallys I'll need until run time so I'll fix the inner loop for now, but I'll definitely check out how to use fill.

[u/gimmeslack12]

this.tally = new array();

This is incorrect. Try new Array or more simply just [].

[u/Discordbothost]

yep that lowercase array is the first crash — constructors are case sensitive so it has to be new Array() or just []. once that runs, the next landmine is bordaTally[i][j].tally like the other comment said; that should be bordaTally[i].tally[j]. hit f12 and the console will name the exact line instead of just stopping after alert 0.