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

在comprarCarta()您要修改mao或mao2 ，它們是靜態數組。 您對這些數組所做的任何更改將在Jogador的所有實例中保留。

因為這不是您期望的，所以我懷疑您可能不希望它們是靜態的。

Answer 2

如戴維南所指出的，導致問題的根本原因是對mao[]和mao2[]使用靜態變量

您還需要從方法comprarCarta和somarMao刪除靜態somarMao 。 在內部引用的所有變量不再是靜態的之后，您將不需要它。 從測試中可以明顯Jogador.comprarCarta(1) ，您沒有以通常調用靜態方法的方式使用它們，即Jogador.comprarCarta(1)而不是p1.comprarCarta(1) 。

我的猜測是您創建了那些靜態方法並編譯器，然后抱怨您的變量不是靜態的並且無法訪問，因此您也更改了變量。

實際上，使用靜態也會導致程序其他部分出現問題-最好將nome變量更改為非靜態。 同樣適用於getPontos()方法-如果您嘗試在那里返回ponto而不是0，則會出現編譯器錯誤，即pontos不是靜態的。

當您普遍混淆靜態的使用時，請在有空的時候看一下關於靜態的解釋：
靜態變量-它們是什么？