Javascript for loop with asignment in comparison section

Question

Is this shortcut

var cars = [ "ford", "toyota"];

for (var i = 0, car; i < cars.length, car = cars[i]; i++) {
    console.log(car);
}

for

var cars = [ "ford", "toyota"];
var car;

for (var i = 0; i < cars.length; i++) {
    car = cars[i];

    console.log(car);
}

good or bad practice?

I have been using longer code notation for some time now, because the processed array name isn't always that short (cars), but now I tried to minimize it and found the solution.

Answer 1

It's not a good practice.

If you not wanna to access by using index try to use forEach instead

like this

var cars = [ "ford", "toyota"];

cars.forEach(function(car,index){
   console.log(car);
})

If wanna to use break then use some or every

Try like this

SOME

var cars = [ "ford", "toyota"];

cars.some(function(car,index){
   console.log(car);
   return true; // break on return true
})

Every

var cars = [ "ford", "toyota"];

cars.every(function(car,index){
   console.log(car);
   return false; // break on return false
})

JSFIDDLE

Answer 2

I prefer this:

 cars.forEach(function(car, i) {
     ...
 });

since you've then got completely enclosed scope for the parameters car and i .

Answer 3

You have already got solutions using .forEach() , here is solution with while loop:

 var cars = ["ford", "toyota"]; var car, i = 0; while (car = cars[i++]) console.log(car);