Solved

array.sort() being ignored in a for... in loop

Posted on 2007-11-26
3
157 Views
Last Modified: 2010-04-03
I have an array whihc I have put a sort on, when I sue a for in loop to loop through the array, it acts as if the array hadn't been sorted. I noticed that using a regular for loop solves this problem, however, however I need this to work with the for in loop. I am aware that a for in will go through an array in reverse order, however this isn't the issue.

Here is the code greatly simplified code, and below, the results of the traces

Many thanks for helping solve this problem

function layout() {
      var cats = ["40artist", "30still", "50fashion"];
      trace("---- Part 1: "+cats);
      cats.sort();
      trace("---- Part 2: "+cats+"\n");
      for (var i in cats) {
            trace("---- Part 3: "+cats[i]);
      }
      trace("\n---- Part 4: "+cats+"\n");
}

layout();


---- Part 1: 40artist,30still,50fashion
---- Part 2: 30still,40artist,50fashion

---- Part 3: 40artist
---- Part 3: 30still
---- Part 3: 50fashion

---- Part 4: 30still,40artist,50fashion

0
Comment
Question by:cursive
  • 2
3 Comments
 
LVL 23

Expert Comment

by:Ashish Patel
ID: 20355861
Try changing, though your code seems working for me.
cats.sort();
to
cats = cats.sort();
0
 
LVL 34

Expert Comment

by:Aneesh Chopra
ID: 20356084
cats.sort() is working fine in your code,
problem is that you are tracing the part3 using "for in" loop which does not gets effected with Array.sort();
you should use "for()" loop

here is the fixed code and it works fine:

function layout()

{

	var cats = ["40artist", "30still", "50fashion"];

	trace("---- Part 1: "+cats);

	cats.sort();

	trace("---- Part 2: "+cats+"\n");

	for (var i = 0; i<cats.length; i++)

	{

		trace("---- Part 3: "+cats[i]);

	}

	trace("\n---- Part 4: "+cats+"\n");

}
 

layout();

Open in new window

0
 
LVL 34

Accepted Solution

by:
Aneesh Chopra earned 500 total points
ID: 20356105
If you have to have "for in" sorting then you can do it with using an extra temp Array

here is the code which will do the trick for you
function layout()

{

	var cats = ["40artist", "30still", "50fashion"];

	trace("---- Part 1: "+cats);

	cats.sort();

	trace("---- Part 2: "+cats+"\n");
 

	var tmp_arr = [];

	for (var i = 0; i<cats.length; i++)

	{

		tmp_arr.push(cats[i]);

	}

	cats = tmp_arr;

	for (var i in cats)

	{

		trace("---- Part 3: "+cats[i]);

	}

	trace("\n---- Part 4: "+cats+"\n");

}
 

layout();

Open in new window

0

Featured Post

How your wiki can always stay up-to-date

Quip doubles as a “living” wiki and a project management tool that evolves with your organization. As you finish projects in Quip, the work remains, easily accessible to all team members, new and old.
- Increase transparency
- Onboard new hires faster
- Access from mobile/offline

Join & Write a Comment

I come across a lot of question about how to access things in the document class from a movieclip, or accessing something from a movieclip in the document class. It took me a while to figure this out but once I did it makes life so much easier. …
This article describes a solution to a problem of subloading one movie into another when they have different SWF versions. Sometime back, I was working on an ActionScript project while I came across an interesting fact which I would like to share…
In this tutorial viewers will learn how to create a basic shape tween animation in Flash including shape hints for smooth animation Open a new document in Flash: Draw a shape: Select another frame (how long you want the tween to be): Right click and…
The goal of the tutorial is to teach the user how to how to record live broadcast.

758 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question

Need Help in Real-Time?

Connect with top rated Experts

22 Experts available now in Live!

Get 1:1 Help Now