note
kwaping
I had the benefit of a direct discussion with the OP about this, which brought much clarity as to his intentions. He has an array of ~4000 elements and wants to select five non-duplicates out of there. So basically, his code can be reduced to this:
<c>
use List::Util 'shuffle';
my %unique = map { $_ => 1 } @weighteddiv;
my @shuffled = shuffle(keys %unique);
@selected = @shuffled[0..4];
</c>
I noticed he was randomizing the array, then selecting random indices from there, which was not only reduntdant but created the issue of potentially selecting the same element.
<br><br>
<b>Update:</b> The big array is *not* already internally unique after all, so I had to add the line to unique it.
<br><br>
<b>Update 2:</b> My solution is no longer relevant, after the OP updated with more information. However, I'm leaving my code as-is because it might help a future monk looking for a similar solution.
<!-- Node text goes above. Div tags should contain sig only -->
<div class="pmsig"><div class="pmsig-253850">
<font size="-1"><i>
<br>---<br>
It's all fine and dandy until someone has to look at the code.
</i></font>
</div></div>
618798
618798