sub shuffle {
  my @r = map rand, 0 .. $#_;
  @_[sort { $r[$a] <=> $r[$b] } 0 .. $#_] = @_;
}
[download]

_____________________________________________________
Jeff[japhy]Pinyan: Perl, regex, and perl hacker, who'd like a job (NYC-area)
s++=END;++y(;-P)}y js++=;shajsj<++y(p-q)}?print:??;

There was a recent thread on the ActiveState list which was never quite solved. In general it came down to assigning to @_ was causing some weirdness to occur and ActiveState perl 5.6.1 would slow down to a crawl after a few thousand iterations. Other perls like my OpenBSD 5.6.1 and Cygwin 5.8.0 behaved nicely.

I'm not sure what the issue was but there may be lurking bug in assigning to @_. I'm warning against it. (or at least be aware so if you do run into trouble later...)

Update: fixed some spelling

Don't fret -- an equally speedy solution is thus:

sub shuffle (\@) {
  my $d = shift;
  my @r = map rand, 0 .. $#$d;
  @$d[sort { $r[$a] <=> $r[$b] } 0 .. $#$d] = @$d;
}
[download]

It requires you send it an actual array, of course, but it is just as fast.

_____________________________________________________
Jeff[japhy]Pinyan: Perl, regex, and perl hacker, who'd like a job (NYC-area)
s++=END;++y(;-P)}y js++=;shajsj<++y(p-q)}?print:??;

You know (leaving the in-place assignment issue aside), it occured to me (duh) that this is a general purpose but faster rephrasing of the Schwartzian Transform:

sub sxsorta (&;) {
    my $sort_key = shift;
    my @rank = map &$sort_key, @_;
    @_[ sort { $rank[$a] cmp $rank[$b] } 0 .. $#_ ];
}

sub sxsortn (&;) {
    my $sort_key = shift;
    my @rank = map &$sort_key, @_;
    @_[ sort { $rank[$a] <=> $rank[$b] } 0 .. $#_ ];
}
[download]

The gains over the regular ST comes from the fact that it doesn't create hundreds of tiny anonymous arrays, and that the sort callback is slightly faster because it only has to do an array lookup but no dereferencing.

The drawback is that with a lexically scoped (as it should be) @rank, there's no way to pass a userspecified callback to the routines without inflicting a massive performance hit. To do more complex sorts, one therefor has to separately spell this idiom out in their own code.

Makeshifts last the longest.

That looks a lot like a method used in Effective Perl Programming (see item 14 in the idiomatic perl pdf)

The goal is to sort /etc/passwd by the third field:

  open PASSWD, "/etc/passwd" or die;
  @passwd = <PASSWD>;
  @key = map                            # Create an array @key contain
+ing
    { (split /:/)[2] } @passwd;         # just the keys (user ids).


  @by_uid_index = sort {                # Sort indices, using @key arr
+ay.
    $key[$a] <=> $key[$b]
  } 0 .. $#key;

  @by_uid = @passwd[@by_uid_index];     # Now, rearrange the contents 
+of
                                        # @passwd into sorted order.

Or just combine the last two statements:

  @by_uid = @passwd[sort { $key[$a] <=> $key[$b] } 0 .. $#key];
[download]

Well, this will work, and it's efficient enough, but there are prettier, more Perl-ish ways to present the solution.

It then goes on to describe the Orcish and Schwartzian under the heading of 'Advanced sorting: the cool ways'

-Blake

Just for the record this technique is sometimes called a Guttman Rosler Transform.


---
demerphq

_{<Elian> And I do take a kind of perverse pleasure in having an OO assembly language...}