Question

我做了一个简单的ADT，它有一个方法（Add3），它给给定的int增加了3。代码如下所示：

public class TestADT 
{
    private final int x;

    public TestADT (int x)
    {
        this.x = x;
    }

    public static TestADT Add3(TestADT num)
    {
        int ex = (num.x + 3);
        return (new TestADT(ex));
    }

    public String toString()
    {
        return(x + "");
    }

    public static void main(String[] args)
    {
        TestADT test = new TestADT(2);
        System.out.println(Add3(test));
    }
}

我想要做的是创建一个JUnit测试来检查Add3方法是否正常工作，到目前为止我有这个：

import static org.junit.Assert.*;

import org.junit.After;
import org.junit.Before;
import org.junit.Test;

public class Add3Test 
{
    private TestADT test;
    private TestADT expected;

    @Before
    public void setUp() throws Exception 
    {
        test = new TestADT(2);
        expected = new TestADT(5);
    }

    @Test
    public void test() 
    {
        TestADT result = TestADT.Add3(test);
        assertEquals(expected, result);
    }
}

当我运行测试课时，它失败了，但我不确定为什么。如果我在运行测试之前打印了两个值（预期和结果），则它们都打印5.

我对JUnit和ADT相对较新，所以我不确定我是否正确地做这件事。我将尝试在JUnit上找到一些教程视频，看看我是否能找到解决方案。任何帮助将不胜感激！

Answer 1

您尚未在equals中覆盖hashcode和TestADT。默认情况下，equals是通过引用完成的。这意味着它要检查第一个是否与第二个实例相同。

您需要覆盖equals和hashcode并让他们按x进行比较以检查是否相等：

public class TestADT
{
    private final int x;

    public TestADT(int x)
    {
        this.x = x;
    }

    public static TestADT Add3(TestADT num)
    {
        int ex = (num.x + 3);
        return (new TestADT(ex));
    }

    public String toString()
    {
        return(x + "");
    }

    public static void main(String[] args)
    {
        TestADT test = new TestADT(2);
        System.out.println(Add3(test));
    }

    @Override
    public boolean equals(Object o) {
        if (this == o) return true;
        if (o == null || getClass() != o.getClass()) return false;

        TestADT testADT = (TestADT) o;

        if (x != testADT.x) return false;

        return true;
    }

    @Override
    public int hashCode() {
        return x;
    }
}

在我看来，这是比@ azbarcea更好的解决方案。如果您想将TestADT放入Set或Map并希望它能正常运行，您需要实施这些方法。

Answer 2

修复代码的最快方法是使用：

@Test
public void test() 
{
    TestADT result = TestADT.add3(test);
    assertEquals(expected.toString(), result.toString());
}

最佳做法是使用小写方法名称：http://docs.oracle.com/javase/tutorial/java/javaOO/methods.html

初学者ADT＆J和4

2 个答案: