java - 测试数组是否为空 JUNIT

Question

我想编写一个方法来显示游戏中的一副牌。我初始化了一个包含 98 个元素的 Array

private int [] cards = new int[98];

我还创建了一个 get 和一个 set 方法

public int[] getCards() {
    return cards;
}

和

public void setCards(final int... cards) {
    this.cards = cards;
}

我编写的方法称为drawCard。如果我使用此方法，它应该从 Array 中删除第一个元素并返回它。

public int drawCard() throws IndexOutOfBoundsException {
    if (getCards().length == 0) {
        throw new IndexOutOfBoundsException("No Cards Left!");
    }
    setCards(ArrayUtils.removeElement(getCards(), 0));
    return getCards()[0];
}

然后我编写了 JUnit 测试。测试应删除所有 98 个元素，然后数组应为空 == 0。但测试总是停在 1。

@Test
public void testDrawCard() {
    Deck deck = new Deck();
    assertThat(deck.getCards().length).isEqualTo(98);

    for(int x = 98; x >= 0; x--){
        deck.drawCard();
    }
    assertThat(deck.getCards().length).isEqualTo(0);
}

错误消息:

java.lang.ArrayIndexOutOfBoundsException: 0
at edu.hm.hafner.java2.thegame.Deck.drawCard(Deck.java:35) 

35=Return Line

at edu.hm.hafner.java2.thegame.DeckTest.testDrawCard(DeckTest.java:40)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:498)
at org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50)
at org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
at org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47)
at org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
at org.junit.runners.ParentRunner.runLeaf(ParentRunner.java:325)
at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:78)
at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:57)
at org.junit.runners.ParentRunner$3.run(ParentRunner.java:290)
at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:71)
at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:288)
at org.junit.runners.ParentRunner.access$000(ParentRunner.java:58)
at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:268)
at org.junit.runners.ParentRunner.run(ParentRunner.java:363)
at org.junit.runner.JUnitCore.run(JUnitCore.java:137)
at com.intellij.junit4.JUnit4IdeaTestRunner.startRunnerWithArgs(JUnit4IdeaTestRunner.java:68)
at com.intellij.rt.execution.junit.IdeaTestRunner$Repeater.startRunnerWithArgs(IdeaTestRunner.java:51)
at com.intellij.rt.execution.junit.JUnitStarter.prepareStreamsAndStart(JUnitStarter.java:237)
at com.intellij.rt.execution.junit.JUnitStarter.main(JUnitStarter.java:70)

有人知道出了什么问题吗？

Answer 1

for(int x = 98; x >= 0; x--)

这会在 x >= 0 时执行循环。也就是说，如果我们递减 x 并且它得到值 0，我们会再次执行循环，因为 0 >= 0 为 true。然后，我们递减x，发现它是-1，因此循环停止。这意味着循环将执行 x = 98, 97, 96, ..., 2, 1, 0。此列表中有 99 个数字，因此循环执行 99 次。

由于您根本不使用 x，因此没有理由让它从顶部开始向下移动。如果唯一的目的是确保循环准确执行 98 次，那么就不要耍小聪明——只需使用标准的 for 循环习惯用法即可:

for (int i = 0; i < 98; i++)

这对程序的执行没有影响。但是，除非必要，否则通过坚持使用标准习惯用法，您将节省一些脑细胞，并最终减少需要解决的错误。