Solved

# Fixing Array to be in order of most occurances and alphabetical

Posted on 2014-03-12
111 Views
Below you find code that I am using to group together values and find how many times it occured.  However, the format it is coming out is not what I would like.

I am getting this:
John,Paul,Peter,2,4,2

I would like to get it like this:
Paul (4)
John (2)
Peter (2)

Basically in order of most occurnaces and then alphabetical if possible with the number or occurances in ().

``````foo(["John","Paul","Peter","Peter","Paul","Paul","Paul","John"])

function foo(arr) {
var a = [], b = [], prev;

arr.sort();
for ( var i = 0; i < arr.length; i++ ) {
if ( arr[i] !== prev ) {
a.push(arr[i]);
b.push(1);
} else {
b[b.length-1]++;
}
prev = arr[i];
}
//return a + "(" + b + );
return [a, b];
}
``````
0
Question by:flfmmqp
• 6
• 5

LVL 58

Expert Comment

ID: 39924448
``````var testArray = new Array("John","Paul","Peter","Peter","Paul","Paul","Paul","John")
testArray.sort()
var newArray = compressArray(testArray);

console.log(newArray)
function compressArray(original) {
var sorted = [];
current=""
count=1
for (var i = 0; i < original.length; i++) {
if(current==original[i]){
count++
}

if((current!=original[i]&&current!="")||i==original.length-1){
var a = new Object();
a.value = original[i-1];
a.count = count;
sorted.push(a);
count=1
}
current=original[i]
}
sorted.sort(function(a,b) {
return  a.count-b.count;
});
return sorted;
};
``````
0

Author Comment

ID: 39924497
I'm pretty new to Javascript.  How should I call this?  I've tried a few different approaches but am getting undefined errors.  I thought this would work.

I am using this in a html page using IE9 if that makes a differences.  Do I need anything special in my script to use the Console.log  piece?

``````foo(["John","Paul","Peter","Peter","Paul","Paul","Paul","John"])

function foo(testArray) {
//var testArray = new Array("John","Paul","Peter","Peter","Paul","Paul","Paul","John")
testArray.sort()
var newArray = compressArray(testArray);

console.log(newArray)
function compressArray(original) {
var sorted = [];
current=""
count=1
for (var i = 0; i < original.length; i++) {
if(current==original[i]){
count++
}

if((current!=original[i]&&current!="")||i==original.length-1){
var a = new Object();
a.value = original[i-1];
a.count = count;
sorted.push(a);
count=1
}
current=original[i]
}
sorted.sort(function(a,b) {
return  a.count-b.count;
});
return sorted;
};
}
``````
0

LVL 58

Expert Comment

ID: 39924514
Why are you nesting the functions?  Just use the code I posted.

In Developer Tools (F12) you should see the Console tab (cannot remember the layout).
The console.log is just there for testing so you can see the returned array, it can be removed
0

Author Comment

ID: 39924526
I need to call it though.  I am grabbing the values from another spot.  They will not be hard coded
0

Author Comment

ID: 39924537
I'm not getting any values.  Just this:

[object Object],[object Object],[object Object],[object Object]

Even using this - var testArray = new Array("John","Paul","Peter","Peter","Paul","Paul","Paul","John")
0

LVL 58

Expert Comment

ID: 39924541
Call what
All you need to do is pass the array to the function

``````var testArray = ["John","Paul","Peter","Peter","Paul","Paul","Paul","John"]
testArray.sort()

var newArray = compressArray(testArray);
...
...
``````

http://jsfiddle.net/GaryC123/rfW6x/3/
0

Author Comment

ID: 39924555

``````        function selectInBuffer(response){
var feature;
var features = response.features;
var inBuffer = [];
//filter out features that are not actually in buffer, since we got all points in the buffer's bounding box
for (var i = 0; i < features.length; i++) {
feature = features[i];
if(circle.contains(feature.geometry)){
inBuffer.push(feature.attributes[featureLayer.objectIdField]);
}
}
var query = new Query();
query.objectIds = inBuffer;
//use a fast objectIds selection query (should not need to go to the server)
featureLayer.selectFeatures(query, FeatureLayer.SELECTION_NEW, function(results){
var r = "";
r = "5 Mile Aggregate Browser:" +"<br> Retailer Summary: " + GetUnique(results) + "<BR> Unigue arrary: " + [b]compressArray(testArray[/b]);
dom.byId("messages").innerHTML = r;
});
}

var testArray = new Array("John","Paul","Peter","Peter","Paul","Paul","Paul","John")
testArray.sort()
var newArray = compressArray(testArray);

//console.log(newArray)
function compressArray(original) {
var sorted = [];
current=""
count=1
for (var i = 0; i < original.length; i++) {
if(current==original[i]){
count++
}

if((current!=original[i]&&current!="")||i==original.length-1){
var a = new Object();
a.value = original[i-1];
a.count = count;
sorted.push(a);
count=1
}
current=original[i]
}
sorted.sort(function(a,b) {
return  a.count-b.count;
});
return sorted;
};
``````
0

LVL 58

Expert Comment

ID: 39924567
What is that supposed to mean to me?
0

Author Comment

ID: 39924570
BTW this is not coming out how I was hoping.  That alert box is not coming out like this order or looking like this:

I would like to get it like this:
Paul (4)
John (2)
Peter (2)

Thanks for trying to help.  But currently the result does not look much different then what I already have posted code for.
0

LVL 58

Accepted Solution

Gary earned 500 total points
ID: 39924589
0

Author Closing Comment

ID: 39926031
I'm awarding the points as I think you answered my question here for me.  However, I'm not completely done with this project.  So if you would like to take a crack at my last issue with that here is the link to my next question.   Thanks for all of your help.

http://www.experts-exchange.com/Programming/Languages/Scripting/JavaScript/Q_28387540.html
0

## Featured Post

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

Things That Drive Us Nuts Have you noticed the use of the reCaptcha feature at EE and other web sites? Â It wants you to read and retype something that looks like this.Insanity! Â It's not EE's fault - that's just the way reCaptcha works. Â But it is â€¦