for循环中ArrayList的奇怪错误

Question

我有一个很奇怪的问题，这个问题只在案例 3 和 4 中出现。

                for(Sector sector : sectoren)
    {
        switch(sector.getCode())
        {
            case 1:
                for(int i = 0; i<3; i++)
                {
                    for(int j=0;j<3; j++)
                    {
                        Gebiedskaart gbk = gebiedskaarten.get(r.nextInt(gebiedskaarten.size()));
                        vakken[i][j].setGbk(gbk);
                        gebiedskaarten.remove(gbk);
                    }
                vakken[2][2].setGbk(null);
                }
            case 2:
                for(int i = 0; i<3; i++)
                {
                    for(int j=4;j<7; j++)
                    {
                        Gebiedskaart gbk = gebiedskaarten.get(r.nextInt(gebiedskaarten.size()));
                        vakken[i][j].setGbk(gbk);
                        gebiedskaarten.remove(gbk);
                    }
                }
                vakken[2][4].setGbk(null);
            case 3:
                for(int i=4; i<7; i++)
                {
                    for(int j=0;j<3; j++)
                    {
                        System.out.println(gebiedskaarten.size());
                        Gebiedskaart gbk = gebiedskaarten.get(r.nextInt(gebiedskaarten.size()));
                        vakken[i][j].setGbk(gbk);
                        gebiedskaarten.remove(gbk);
                    }
                }
                vakken[4][2].setGbk(null);
           case 4:
                for(int i = 4; i<7; i++)
                {
                    for(int j=4;j<7; j++)
                    {
                        Gebiedskaart gbk = gebiedskaarten.get(r.nextInt(gebiedskaarten.size()));
                        vakken[i][j].setGbk(gbk);
                        //gebiedskaarten.remove(gbk); doet iets raar? moeten we nog naar kijken, hij blijft gewoon verwijderen tot de lijst leeg is
                    }
                }
                vakken[4][4].setGbk(null);
        }
    }

在代码中，您可以看到我为数组分配了一个值，然后我从 ArrayList 中删除了该值。你可以看到我已经把 System.out.println 放在那里，看看它做了什么，这是输出：

34
33
32
31
30
29
28
27
26
16
15
14
13
12
11
10
9
8
7
6
5
4
3
2
1
0

然后错误很明显：

Exception in thread "main" java.lang.IllegalArgumentException: n must be positive
    at java.util.Random.nextInt(Random.java:300)
    at domein.Spelbord.wijsGebiedskaartenToe(Spelbord.java:123)
    at domein.Spelbord.<init>(Spelbord.java:19)
    at domein.Spel.setWereldkaart(Spel.java:18)
    at domein.DomeinController.maakWereldkaart(DomeinController.java:39)
    at ui.Uc1.maakWereldkaart(Uc1.java:42)
    at ui.ConsoleApplicatie.behandelUc1(ConsoleApplicatie.java:67)
    at ui.ConsoleApplicatie.startSpel(ConsoleApplicatie.java:29)
    at StartUp.main(StartUp.java:12)
Java Result: 1

我不知道发生了什么，也许你们知道？

Answer 1

因为你这样做了：

 r.nextInt(gebiedskaarten.size());

当gebiedskaarten.size() == 0

==> 你尝试得到一个从 0 到 0 的随机数 ==> 异常

更新：您似乎也忘记了 switch-case 中的 break; 语句！我猜你想成为你的代码逻辑是这样的：

 switch(sector.getCode()) {
 case 1: /*do something*/ break;
 case 2: /*do something else */ break;
 //...
 default: //do some default action
 }

另见相关问题：Why do we need break after case statements?

Answer 2

您正在调用 Random.nextInt()，值为 0，这是不受支持的。

正如您从System.out 的输出中看到的那样，数组的大小最终为0，并且您将数组的大小传递给nextInt()。

另见http://docs.oracle.com/javase/7/docs/api/java/util/Random.html#nextInt(int)

Answer 3

我不知道为什么您的循环运行了这么多次，而预期只有 9 (3 * 3) 次，但假设您发布的代码从 gebiedskaarten.size() == 9 开始，它会执行 9 次迭代：

• System.out.println(gebiedskaarten.size())
  • 打印当前尺寸。
• Gebiedskaart gbk = gebiedskaarten.get(r.nextInt(gebiedskaarten.size()))
  • 首先执行r.nextInt(...)和gebiedskaarten.size()，获得一个新的随机值。
  • 然后使用该随机数调用gebiedskaarten.get(...)，并将其分配给gbk。
• vakken[i][j].setGbk(gbk);
  • 在vakken 的i, j 条目上调用setGbk(gbk)。
• gebiedskaarten.remove(gbk);
  • 从gebiedskaarten 集合中删除gbk，将大小减小一。

哪里出错了，是某个点，gebiedskaarten.size() 变成了0，然后你尝试通过r.nextInt(...) 在0 上获得一个新的随机数，这是不可能的，因为它需要严格积极的。
但是请注意，调用 gebiedskaarten.get(0) 是有效的，因此您会遇到一次性错误。

我还强烈建议您从荷兰语命名约定转变为英语命名约定，因为这将对您的职业未来和在线发布问题有很大帮助。

另外，为了完整起见，正如@donfuxx 所指出的，您的switch 语句不符合标准，这在99% 的情况下会导致错误，应该是这样的：

switch (variable) {
    case 1:
        //do interesting things
        break;
    case 2:
        //do interesting things
        break;
    case 3:
        //do interesting things
        break;
    default:
        //either allowable base case, or may-not-happen error case
        break;

如您所见，我在每个case 的末尾添加了一个break，这是防止它通过语句的必要条件。如果case 1 的末尾没有break，那么case 2 将也被执行。