如何更正GCD函数的单元测试

Question

我编写了一个简单的GCD函数来实现欧几里德算法，用于计算最大公约数GCD（m，n），即最大整数k除以m和n。

我编写的函数成功编译：

   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；一些其他风格的断言，我发现这导致了“更可读”的代码。当使用它时，你必须使用hamcrest匹配器，比如is（）虽然（谷歌是你的朋友）。