错误而洗牌在Java中

Question

数组这是我迄今为止

int[] question = new int[25]; 

for (int i = 0; i < question.length; i++){ 
    question[i] = i+1; 
} 

Random rand = new Random(); 

int max = question.length-1, min = 1; 

for(int i = 0; i < question.length; i++){ 
    int idx = rand.nextInt((max - min) + 1) + min; 
    randg[i] = idx; 
    question[i] ^= question[idx]; 
    question[idx] ^= question[i]; 
    question[i] ^= question[idx]; 

    if(question[i] == 0){ 
     System.out.println("Something went wrong!" + i + " " + idx); 
    } 
} 

所以，问题似乎是当兰特值（IDX）等于我，为交换......它只是用0替换该值。

如何解决该问题？

Answer 1

我知道最快的修复方法;使用Integer[]，Arrays.asList(T...)和Collections.shuffle(List)像

Integer[] question = new Integer[25]; 
for (int i = 0; i < question.length; i++) { 
    question[i] = i + 1; 
} 
System.out.println(Arrays.toString(question)); 
Collections.shuffle(Arrays.asList(question)); 
System.out.println(Arrays.toString(question)); 

，或者用代码，你可以添加

for(int i = 0; i < question.length; i++){ 
    int idx = rand.nextInt((max - min) + 1) + min; 
    if (idx == i) { 
    i--; 
    continue; 
    } 

Answer 2

不建议将您正在使用的XOR交换，它可以零出位， 你的时候通过相同的变量两次，所以交换（a，a）可能会提供0： 更多信息请参阅：https://softwareengineering.stackexchange.com/questions/182037/is-this-xor-value-swap-algorithm-still-in-use-or-useful

你没有得到XOR Swap，它既不聪明也不fa韦伯斯特。 这是汇编langugae保存一个变量的技巧。

的推荐方法来交换：

在阵列交换两个值的[]索引i和j：A [1]和a [j]的：

int help = a[i]; 
a[i] = a[j]; 
a[j] = help;