Tell me more ×
Stack Overflow is a question and answer site for professional and enthusiast programmers. It's 100% free, no registration required.
up vote 0 down vote favorite

I am trying to declare variables using for loop and then test if the cols and rols are the same. If they are, it will run a recursive function. However, I am having trouble doing do in javascript. Can anyone help?

Right now, it says unexpected variable == when comparing col.1 and col.2. I also tried col+j inside the for loop, but it was invalid left-hand side assignment

for (var i = 0; i < 2; i++) {
  var col = {};
  col.i = Math.floor(Math.random() * cols);
  col.i = Math.floor(Math.random() * rows);
}

if (col.1 == col.2 && row.1 == row.2) {
  return this.getRandomBlock();
}
share|improve this question

2 Answers

up vote 6 down vote accepted
  1. You should declare col and row, you declare only col.
  2. You should declare col and row outside of the loop, now it declares each time when the body of the loop executes.
  3. You assign to col.i twice.
  4. When you write col.i it is equel to col['i'], so you should use brackets instead.
  5. You don't need object ({}), use array ([]).
  6. You cannot write col.1 it is not legal, number after dot is not legal, use col[1].
  7. You have 0 and 1, not 1 and 2.

.

var col = [];
var row = [];

for (var i = 0; i < 2; i++) {
  col[i] = Math.floor(Math.random() * cols);
  row[i] = Math.floor(Math.random() * rows);
}

if (col[0] == col[1] && row[0] == row[1]) {
  return this.getRandomBlock();
}
share|improve this answer
Great Speransky.I just writing down the same. – Sunny Sep 5 '12 at 5:27
up vote 0 down vote

The col variable has declared inside the for loop which means that variable instantiated every iteration. So, at the end of the for loop, the col variable has only one property that is 1 (col.1).

After the loop you are trying to access col.2 which is not a property of col object. Also, if you are trying to access col object using for loop iteration variable (i or j) the same problem occurs because the value of i or j after iteration is 2

Try the below it will work like a pro.

var col = {};
var row = {};
for (var i = 0; i < 2; i++) {
  col.i = Math.floor(Math.random() * cols);
  row.i = Math.floor(Math.random() * rows);
}
if (col.0 == col.1 && row.0 == row.1) {
  return this.getRandomBlock();
}
share|improve this answer

Your Answer

 
discard

By posting your answer, you agree to the privacy policy and terms of service.

Not the answer you're looking for? Browse other questions tagged or ask your own question.