JUnit：setUp 和 tearDown 方法没有按预期工作

Question

我尝试使用 TDD 开发一些代码，但偶然发现了一个奇怪的行为：setUp 和 tearDown 似乎在每次测试执行后都没有“清理”。我希望每个测试（标有 @Test 注释）以随机顺序执行，一个接一个，而不会相互影响。考虑到这一点，我不明白发生了什么，因为似乎一个特定的测试（testaSomarMao）正在影响另一个特定的测试（testaSplit）。 testaSplit 测试在第一个断言上失败：我期望值为 6，但我得到 9。任何人都可以解释我发生了什么吗？

JogadorTest.java

import static org.junit.Assert.*;
import org.junit.After;
import org.junit.Before;
import org.junit.Test;

public class JogadorTest{
    private static Jogador p1;
    public JogadorTest(){

    }

    @Before
    public void setUp(){
        p1 = new Jogador();
    }

    @After
    public void tearDown(){
        p1 = null;
    }

    @Test
    public void testaGetNome(){
        assertEquals(null, p1.getNome());
    }

    @Test
    public void testaGetPontos(){
        assertEquals(0, p1.getPontos());
    }

    @Test
    public void testaSetNome(){
        p1.setNome("Lucas");
        assertEquals("Lucas", p1.getNome());
    }

    @Test
    public void testaSomarMao(){
        p1.comprarCarta(1);
        assertEquals(3, p1.somarMao(1));
    }

    @Test
    public void testaSplit(){  
        p1.comprarCarta(1);
        p1.comprarCarta(1);        
        assertEquals(6, p1.somarMao(1));
        p1.split();
        assertEquals(p1.somarMao(1), p1.somarMao(2));
    }
}

Jogador.java

public class Jogador {

    private static String nome;

    private int pontos;

    private static int mao[] = new int[13];

    private static int mao2[] = new int[13];

    public void parar() {

    }

    public  void setNome(String novoNome){    
        nome = novoNome;    
    }

    public static int getPontos() {
        return 0;
    }

    public static void split() { 
        //Usamos mao2 para garantir que soh ocorra um split.
        if(mao[0] == mao[1] && mao2[0] == 0){            
            mao2[0] = mao[1];
            mao[1] = 0;
        }
    }

    public void fecharJogo() {

    }

    public String getNome() {
        return nome;
    }

    public static int somarMao(int maoEscolhida){
        int soma = 0;
        int cartasNaMao[];

        if(maoEscolhida == 2)
            cartasNaMao = mao2;        
        else
            cartasNaMao = mao;

        for(int i = 0; i < cartasNaMao.length; i++)
            soma += cartasNaMao[i];
        return soma;
    }

    public static void comprarCarta(int maoEscolhida){
        int carta = 3; // random futuramente
        int cartasNaMao[];

        if(maoEscolhida == 2)
            cartasNaMao = mao2;        
        else
            cartasNaMao = mao;

        for(int i = 0; i < cartasNaMao.length; i++){
            if(cartasNaMao[i] == 0) {
                cartasNaMao[i] = carta;
                break;
            }
        }
    }

}

Answer 1

正如 Thevenin 指出的那样，您的问题的根本原因是 mao[] 和 mao2[] 使用了静态变量

您还需要从方法comprarCarta 和somarMao 中删除静态。在内部引用的所有变量都不再是静态之后，您将不再需要它。从测试中很明显，您没有以通常调用静态方法的方式使用它们，即Jogador.comprarCarta(1) 而不是p1.comprarCarta(1)。

我的猜测是你创建了那些静态方法，然后编译器抱怨你的变量不是静态的并且无法访问，所以你也更改了变量。

实际上，使用静态也会导致您程序的其他部分出现问题 - 您最好将 nome 变量更改为不是静态的。同样适用于getPontos() 方法 - 如果您尝试在那里返回 pontos 而不是 0，您将收到一个编译器错误，即 pontos 不是静态的。

由于您通常对使用 static 感到困惑，所以有空的时候看看这个关于什么是静态的精彩解释 :)
Static variables - what are they?

Answer 2

在comprarCarta() 中，您正在修改mao 或mao2，它们是静态数组。您对这些数组所做的任何更改都将在所有 Jogador 实例中持续存在。

由于这不是您所期望的，我怀疑您可能不希望这些是静态的。