bug, array swaping values about crystal HOT 3 CLOSED

I also added Array#swap(index0, index1) that should be faster if you care about performance (if that's your bottleneck). This only does two bounds check. The other way:

a[0], a[1] = a[1], a[0]

gets translated to:

temp1 = a[1]
temp2 = a[0]
a[0] = temp1
a[1] = temp2

so that's four bounds check.

from crystal.

thanks, maybe better to call it swap! because it changes array state.

from crystal.

I don't think it makes sense to clone the array and swap the elements. There's also no "pop!" in Ruby and "pop" modifies self, so...

from crystal.