这样的排序方法有毛病吗？

Question

// 打乱数组
function shuffle(data){
    let startIndex = 0,
        endIndex = length = data.length - 1;
    while(endIndex){
        let index = Math.floor(Math.random()*length);
        [data[index], data[endIndex]] = [data[endIndex], data[index]];
        endIndex--;
    }
}

// 具有负数或重复值的数据除外，使用这种方式似乎比其他排序更快一些。
var obj = {};
var arr = Array.from({length:1e5},(item,index)=>index);

shuffle(arr);
console.time();
arr.forEach(item => { obj[item] = item });
console.log(Object.values(obj));
console.timeEnd();

Answer 1

没什么毛病，就是洗牌算法，不过洗牌算法乘以length会有重复的洗牌过程，所以可以直接

let index = Math.floor(Math.random()*endIndex);