CODEWITHSUNDEEP
javascriptjquerysorting
xorinzor
xorinzor

Reputation: 6467

Sort array by value from sub-array item

Im trying to sort the items from an object/array by their "name" item,

I used this page for reference Sort array of objects and build the piece of code below:

var alphabet = {
    a: 1,
    b: 2,
    c: 3,
    d: 4,
    e: 5,
    f: 6,
    g: 7,
    h: 8,
    i: 9,
    j: 10,
    k: 11,
    l: 12,
    m: 13,
    n: 14,
    o: 15,
    p: 16,
    q: 17,
    r: 18,
    s: 19,
    t: 20,
    u: 21,
    v: 22,
    w: 23,
    x: 24,
    y: 25,
    z: 26
}

var test = {
    item: {
        name: "Name here",
        email: "[email protected]"
    },

    item2: {
        name: "Another name",
        email: "[email protected]"
    },

    item3: {
        name: "B name",
        email: "[email protected]"
    },

    item4: {
        name: "Z name",
        email: "[email protected]"
    }
};
test.sort(function (a, b) {return alphabet[a.name.charAt(0)] - alphabet[b.name.charAt(0)]});

console.log(test);

Unfortunately no errors are returned and the console.log doesn't returns anything either. Any help is greatly appreciated!

EDIT: After the answer has been given, it seemed the variable "test" was required to be an array, however, the variable was generated dynamically in an external library, therefor I made this little piece of code. If anyone has the same issue, feel free to use it.

var temp = [];
$.each(test, function(index, value){
    temp.push(this);
});

//temp is the resulting array

Upvotes: 0

Views: 2026

Answers (2)

Marcelo Cantos
Marcelo Cantos

Reputation: 185852

test is an object, not an array. Perhaps you want this:

var test = [
    {
        name: "Name here",
        email: "[email protected]"
    },
    ⋮
];

If you need the item, item1, … to be retained against each object, you could add them as fields of each object:

var test = [
    {
        id: "item",
        name: "Name here",
        email: "[email protected]"
    },
    ⋮
];

To sort alphabetically, you need a case-insensitive comparator (and forget the alphabet object):

compareAlpha = function(a, b) {
  a = a.name.toUpperCase(); b = b.name.toUpperCase();
  return a < b ? -1 : a > b ? 1 : 0;
};

Upvotes: 4

pbeardshear
pbeardshear

Reputation: 1000

Firstly, test should be an array, not an object. Secondly, I think you are missing a call to .toLowerCase() after selecting the character.

test.sort(function (a, b) {
    return alphabet[a.name.charAt(0).toLowerCase()] - alphabet[b.name.charAt(0).toLowerCase()];
});

Upvotes: 1

Related Questions