致命错误：在Swift 2.0中不支持将位置与其自身交换

Question

我有这个扩展，它会从给定的数组中随机创建一组新数组。

extension Array {
    var shuffle:[Element] {
        var elements = self
        for index in 0..<elements.count {
            swap(&elements[index], &elements[ Int(arc4random_uniform(UInt32(elements.count-index)))+index ])
        }
        return elements
    }
    func groupOf(n:Int)-> [[Element]] {
        var result:[[Element]]=[]
        for i in 0...(count/n)-1 {
            var tempArray:[Element] = []
            for index in 0...n-1 {
                tempArray.append(self[index+(i*n)])
            }
            result.append(tempArray)
        }

        return result
    }
}

"我是这样使用它的："

let mainArr = Array(1...60)
let suffeldArr = mainArr.shuffle.groupOf(10)
print(suffeldArr)

"并且它会打印出以下内容："

[[10 random element between 1 to 60], [10 random element between 1 to 60], [10 random element between 1 to 60], [10 random element between 1 to 60], [10 random element between 1 to 60], [10 random element between 1 to 60]]

但是它在运行时在这一行给我一个错误：

swap(&elements[index], &elements[ Int(arc4random_uniform(UInt32(elements.count-index)))+index ])

这段话的意思是：

“致命错误：不支持将位置与其自身交换。在1.2版本中运作正常，但是在2.0版本中却无法正常工作。我不知道该如何解决这个问题。”

- Dharmesh Kheni

---

请注意，https://dev59.com/yWAg5IYBdhLWcg3wDHU4#24029847 中的代码已更新以解决此问题：https://dev59.com/yWAg5IYBdhLWcg3wDHU4#Ua2gEYcBWogLw_1boBuG - Martin R

@DharmeshKheni更新了原始问题https://dev59.com/k14d5IYBdhLWcg3wCOyD#27261991 - Leo Dabus

Answer 1

您正在尝试将元素与其自身交换，您需要执行检查以查看是否尝试将元素交换到数组中的相同位置，如下所示：

您正在尝试将元素与其自身交换，您需要执行检查以查看是否尝试将元素交换到数组中的相同位置，如下所示：

extension Array {
    var shuffle:[Element] {
        var elements = self
        for index in 0..<elements.count {
            let newIndex = Int(arc4random_uniform(UInt32(elements.count-index)))+index
            if index != newIndex { // Check if you are not trying to swap an element with itself
                swap(&elements[index], &elements[newIndex])
            }
        }
        return elements
    }
    func groupOf(n:Int)-> [[Element]] {
        var result:[[Element]]=[]
        for i in 0...(count/n)-1 {
            var tempArray:[Element] = []
            for index in 0...n-1 {
                tempArray.append(self[index+(i*n)])
            }
            result.append(tempArray)
        }

        return result
    }
}

- vrwim

---

@DharmeshKheni 还可以查看这个干净简单的 Fisher-Yates 洗牌算法的答案：https://dev59.com/yWAg5IYBdhLWcg3wDHU4#24029847 - vrwim

@DharmeshKheni 好的，没注意到 :) - vrwim

@Cœur 我搞错了，第一次没有正确识别它。 - Alexander