jquery reverse array

Question

I have the following snippet which returns some youtube id's. Now I want to reverse the output (because now it is the last first)

if (options.slideshow) {
var links = [];
var $lis = holder.parents('#yt_holder').find('li');
var $as = $lis.children('a');
for(var count = $lis.length-1, i = count; i >= 0; i--){
    links.push(youtubeid($as[i].href));
    }
 slideshow = '&playlist=' + links + '';
 alert(slideshow);
}

I tried .reverse() but some items seems to be missing then

links.reverse().push(youtubeid($as[i].href));

Any help will be appreciated. Ceasar

Instead of moving backwards through the array of $as (decrementing towards 0) and then reversing the array, why not go forwards through the array of $as, by incrementing towards i<$lis.length? — David Thomas
– David Thomas, Commented Jan 2, 2012 at 16:03
Where did you call links.reverse().push(youtubeid($as[i].href));? In your for loop? You should just call reverse once. — kba
– kba, Commented Jan 2, 2012 at 16:06

Alnitak · Accepted Answer · 2012-01-02 16:47:16Z

14

You should reverse the list after you've accumulated it:

for ( ... ) {
    ...
}
links = links.reverse();

but it would be better to just put the elements into the array in the right order in the first place.

edited Jan 2, 2012 at 16:47

answered Jan 2, 2012 at 16:15

Alnitak

341k72 gold badges420 silver badges503 bronze badges

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

2 Comments

ceasar Over a year ago

This also works. What is the best solution Jason's or yours ? And do you know how to skip the first one in the array ? Thanks

Alnitak Over a year ago

@ceasar to remove the first element - var first = links.shift();

Jason Gennaro · Accepted Answer · 2012-01-02 16:06:18Z

2

Try adding the videos in the reverse order, so instead of this

for(var count = $lis.length-1, i = count; i >= 0; i--){
    links.push(youtubeid($as[i].href));
    }

Do this

for(var i = 0, count = $lis.length; i < count; i++){
    links.push(youtubeid($as[i].href));
    }

answered Jan 2, 2012 at 16:06

Jason Gennaro

34.9k9 gold badges67 silver badges86 bronze badges

2 Comments

ceasar Over a year ago

He jason. That works :-) Now how can I remove only the first one in the array Thanks for your help

ceasar Over a year ago

links.shift(); is the answer to my own question :-)

Mr7-itsurdeveloper · Accepted Answer · 2014-07-02 06:03:42Z

0

Hi you after reversing the links array ,you have to assign it to other array and hence it will work.

var slideshow = [];
slideshow = links.reverse();

edited Jul 2, 2014 at 6:03

Mr7-itsurdeveloper

1,6412 gold badges17 silver badges24 bronze badges

answered Jul 2, 2014 at 5:28

Aakash Dhoundiyal

37 bronze badges

Collectives™ on Stack Overflow

jquery reverse array

3 Answers 3

2 Comments

2 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

2 Comments

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related