java - 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 variables - what are they?