我一直得到错误“索引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 out of length 6”。

Answer 1

请考虑行：

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

i从0变化到noofcards-1随机数表达式从0变化到noofcards-2所以s从0变化到(2*noOfCards)-3

则array[s]将在s>=noOfCards时引发异常

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