如何更正我的GCD功能单元测试

Question

我编写了一个简单的GCD函数，以实现Euclid算法来计算最大公约数gcd（m，n），这是除以m和n的最大整数k。

我成功编写的函数可以编译：

   public static int gcd(int m, int n) {
      if (n == 0) return m;
      return gcd(n, m%n);
   }

但是，当我在GCD上编写单元测试时遇到错误：

   @Test public void gcdTest() {
      for (int m = 0; m < 15; m++) {
      for (int n = 0; n < 15; n++) {
         assertEquals("Divide m,n", m/n%m, Recursion.gcd(m,n));
         }
      }
   }

错误出现在“ assertEquals”行中。 我不确定是否可能通过写m / n％m错误地计算了此方法。

有任何提示或建议吗？ 提前致谢。

Answer 1

除了这里的“数学”之外，您不应该立即在单元测试中使用循环。

我的意思是：在考虑在一个循环中迭代并执行多个断言的测试用例之前，请先执行以下操作

@Test
public void gcdTest1_1() {
   assertThat(Recursion.gcd(1,1), is(1));
}

换句话说：编写仅测试一件事的简单测试用例。 当第一个通过时，写下下一个。 然后，当您更有信心时，请考虑使用这种循环解决方案。

这样一来，您可能会想到除以0 ...并不是测试中不可缺少的东西！

根据您的评论进行编辑：使用see，单元测试的核心思想是它们可以帮助您发现并修复被测代码中的错误。 因此，从您的示例来看，最大的障碍是...您正在打印字符串“ m，n”。 那什么都没告诉你。 您已经知道您的变量称为m和n。 最好在断言失败的情况下打印m和n的值 。

最后：我更改为assertThat ； 我发现其他一些断言风格会导致“更具可读性”的代码。 使用该功能时，您将不得不使用is（）之类的hamcrest匹配器（google是您的朋友）。