我一直得到错误"索引5出界长度5"但不知道要改变什么

Question

我对编码很陌生，所以请耐心听我说，我只是在尝试洗牌数组的方法，以便最终在我Java创建的纸牌游戏中洗牌。我知道索引5超出了长度5的范围，但令我困惑的是错误并不总是会弹出。有时我试图运行我的代码，它工作得很好，有时我得到了错误，即使我在运行它的两次之间没有改变任何东西。

public class Card{

    public static void shuffle(Object[] array) {

        int noOfCards = array.length;

        for (int i = 0; i < noOfCards; i++) {

            int s = i + (int)(Math.random() * (noOfCards - 1));

            Object temp = array[s]; //this is the first line it says has a problem
            array[s] = array[i];
            array[i] = temp;
        }
    }

    public static void main(String[] args) {

        String[] strOfCards = {"A","B","C","D","E"};

        Card.shuffle(strOfCards); //this is the second line that has a problem
        for(int i = 0; i < strOfCards.length; i++) {
            System.out.println(strOfCards[i] + " ");
        }
    }
}

我不知道如何更改有缺陷的线条，欢迎提出任何建议！***我尝试过更改字符串中的字母数，但错误随之改变，即“索引6超出长度6的界限”

Answer 1

以下是导致问题的原因：

int s = i + (int)(Math.random() * (noOfCards - 1));

只要s值为

例如i可以是3，如果Math.random（）返回类似于0.5的东西，那么3（int）（0.5*4）等于5，这是出界的（0...4）

这种情况并不总是发生，因为有时Math.random（）生成的数字足够小，所以s的求值小于5。

为了确保s的值总是在范围内（0...4），您应该将结果取模为5。

这条线应该是这样的：

int s = (i + (int)(Math.random() * (noOfCards - 1))) % 5;

Answer 2

长度为N的数组的最大索引为（N-1）。代码应该是这样的：

public static void shuffle(Object[] array) {
    int noOfCards = array.length;
    Random random = new Random();
    for (int i = 0; i < noOfCards; i++) {
        int s = random.nextInt(noOfCards);

        Object temp = array[s];
        array[s] = array[i];
        array[i] = temp;
    }
}

Answer 3

考虑行：

for (int i = 0; i < noOfCards; i++) {
    int s = i + (int)(Math.random() * (noOfCards - 1));
    Object temp = array[s]; //this is the first line it says has a problem

你的随机数表达式从0到noOfCards-2不等，所以s从0到（2*noOfCards）-3不等

然后，array[s]将在

并不是每次你运行它都会发生，因为有时候随机数都碰巧在noOfCards下

如果您想与其他随机卡交换，则可以尝试：

Random random = new Random();
int s = (i + random.nextInt(noOfCards - 1)) % noOfCards;

我知道你在利用这个机会学习，但如果你不知道的话，这里有一个集合。shuffle方法，可用于在一个命令中洗牌集合，如ArrayList。