Issue with Javascript For loop

Question

Consider the Code below:

function splicer()
{
 var arrayElements = ["elem1","elem2","elem3","elem4"];
 for(var index in arrayElements)
 {
  arrayElements.splice(index,1);
 }
 alert("Elements: "+arrayElements);
}

The above function is supposed to remove all the elements from the array "arrayElements". But it won't.

Javascript engine maintains the "index" as it is and doesn't mind the array being modified. People might expect something like "for each" loop that doesn't have this kind of issue

even the following code doesn't seem to work:

function splicer()
{
...
 for(var index in arrayElements)
 {
  arrayElements.splice(index--,1);
 }
...
}

even when changing the value of the variable "index" doesn't seem to work. the changed value is available inside the "for(...){...}" block but, as the loop reaches the next iteration, the value gets reset and continues from the next index as clockwork.

so it seems code like this might be the only solution:

function splicer()
{
 var arrayElements = ["elem1","elem2","elem3","elem4"];
 for(var index=0;index<arrayElements.length;index++)
 {
  arrayElements.splice(index--,1);
 }
 alert("Elements: "+arrayElements);
}

Tested in: Firefox 16 Beta.

But placing a unary Operator inside a "splice()" method seems to be misleading at first sight.

This might be worth considering to the "W3C" or whomever it may concern so that they come up with a nice solution.

Possible duplicate of stackoverflow.com/questions/500504/… — Sergii Kudriavtsev
– Sergii Kudriavtsev, Commented Sep 28, 2012 at 8:34

Konstantin Dinev · Accepted Answer · 2012-09-28 08:36:49Z

2

You may want to refer to John Resig's array.remove() link.

// Array Remove - By John Resig (MIT Licensed)
Array.prototype.remove = function(from, to) {
  var rest = this.slice((to || from) + 1 || this.length);
  this.length = from < 0 ? this.length + from : from;
  return this.push.apply(this, rest);
};

answered Sep 28, 2012 at 8:36

Konstantin Dinev

35k14 gold badges79 silver badges102 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

Nokia808Freak Over a year ago

D This piece of code Looks nice. But modifying the "out of the Box" Array Object doesn't sound like a good idea. and BTW the code snippet isn't user friendly at first sight. The guy who coded this might've carefully done this he must be really good in it...

Konstantin Dinev Over a year ago

@CrystalPaladin the link provides you with an example where the prototype is not used.

web-nomad · Accepted Answer · 2012-09-28 08:38:16Z

0

Try this:

*Splice modifies the original array, hence tge loop skips the alternate values. *

var arrayElements = ["elem1","elem2","elem3","elem4"];
arrayElements.splice(0,arrayElements.length);
alert("Elements: "+arrayElements)

answered Sep 28, 2012 at 8:38

web-nomad

6,0133 gold badges37 silver badges49 bronze badges

3 Comments

Nokia808Freak Over a year ago

Not alternate values it just skips progressively. skips one for first iteration and two for second not including the first one it already skipped. and so on...

Nokia808Freak Over a year ago

I'm not emphasizing on emptying an array! I expected the viewers might think of a possible solution/explanation of the JavaScript behavior in such scenario

web-nomad Over a year ago

If you see the splice docs - w3schools.com/jsref/jsref_splice.asp, it is clearly written that This method changes the original array. Let me create a fiddle for you which will hopefully make things clear. I'll post it in some time.

Collectives™ on Stack Overflow

Issue with Javascript For loop

2 Answers 2

2 Comments

3 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

3 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related