Question

我正在尝试使用TDD开发一些代码，但偶然发现了一个奇怪的行为：setUp和tearDown似乎并没有＆＃34;清理＆＃34;每次测试后执行。我希望每个测试（用@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

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

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

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

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

正如你在混淆使用静态一般看看这个静态的很好的解释，当你有时间:)
Static variables - what are they?

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

2 个答案: