尝试 Catch 块有效但测试 assertThrows 失败(Junit 5)
Try Catch block works but test assertThrows fail (Junit 5)
我正在尝试学习本教程
我使用 Java 10、IntelliJ 2018 和 Junit 5。
我制作了一个可以将 2 个分数相加的计算器应用程序。它检查输入的分母是否为 0。
当我 运行 测试异常消息被打印出来 "Undefined Math Expression" 但我的 IDE 说 "Expected java.lang.Throwable to be thrown, but nothing was thrown." 我认为我的代码范围存在一些问题?我是新手,请多多关照。我在下面提供了代码和测试:
public class Calculator {
public static int[] calculate (int firstNumerator, int firstDenominator, int secondNumerator, int secondDenominator) {
String exceptionMessage = "Undefined Math Expression";
int resultNumerator;
int resultDenominator;
int[] result = new int[2];
resultNumerator = (firstNumerator * secondDenominator) +
(secondNumerator * firstDenominator);
resultDenominator = firstDenominator * secondDenominator;
try {
if (resultDenominator == 0) {
throw (new Throwable(exceptionMessage));
} else {
result[0] = resultNumerator;
result[1] = resultDenominator;
}
} catch (Throwable e) {
System.out.println(e.getMessage());
}
return result;
}
}
测试:
class CalculatorTest {
@Test
void denominatorContainsZero() {
assertThrows(Throwable.class, () -> {
Calculator.calculate(0,0,0,0);
});
}
}
Throwable
被 try catch
块捕获,因此 Junit 无法访问它。尝试删除 try catch
块。
你实际上并没有抛出异常,你正在捕获它。为此,您应该删除 try catch 块。
这里的误解似乎在于 JUnit 实际可以看到什么。
JUnit 并不神奇:它只是普通的旧 Java。它看不到你的方法内部,看不到它们在做什么。它所能看到的是任何其他代码在执行方法时所能看到的:return 值和未捕获的异常(以及该方法的任何副作用,如果它们对调用代码可见的话)。
从调用者的角度来看,您的方法不会抛出异常:在内部,它会抛出异常,但会捕获并处理它。
如果您希望 JUnit 测试是否抛出了异常,则不需要捕获该异常。
抛出异常然后自己捕获并处理它从来都不是 (*) 正确的做法。重点是什么?你可以简单地做你做的事情来处理它,而不会抛出异常。抛出异常代价高昂,因为需要捕获整个堆栈跟踪。
Throwable
永远不会 (*) 抛出正确的异常。它是 returning Object
的异常 "equivalent":它没有向调用者传达有关异常的类型信息,调用者要么必须做大量工作来尝试处理它;要么或者,更现实地说,应该自己传播它。如果您确实需要抛出(而不是捕获)异常,IllegalArgumentException
是在此处抛出的正确异常。
Throwable
很少是正确的捕捉对象。 Throwable
是 Exception
和 Error
的超类型,因此您可能会无意中捕捉到 Error
,例如 OutOfMemoryError
,它不应该被捕捉,因为有除了让你的程序崩溃之外,没有什么可以做的。尽可能抓住最具体的类型;这也意味着您应该尽可能抛出最具体的类型(或者至少是适合抽象的类型)。
(*) 这是 "never",如 "ok, there are a limited number of circumstances where it may be appropriate"。但是除非你明白这些是什么,否则不要。
我正在尝试学习本教程
我使用 Java 10、IntelliJ 2018 和 Junit 5。
我制作了一个可以将 2 个分数相加的计算器应用程序。它检查输入的分母是否为 0。
当我 运行 测试异常消息被打印出来 "Undefined Math Expression" 但我的 IDE 说 "Expected java.lang.Throwable to be thrown, but nothing was thrown." 我认为我的代码范围存在一些问题?我是新手,请多多关照。我在下面提供了代码和测试:
public class Calculator {
public static int[] calculate (int firstNumerator, int firstDenominator, int secondNumerator, int secondDenominator) {
String exceptionMessage = "Undefined Math Expression";
int resultNumerator;
int resultDenominator;
int[] result = new int[2];
resultNumerator = (firstNumerator * secondDenominator) +
(secondNumerator * firstDenominator);
resultDenominator = firstDenominator * secondDenominator;
try {
if (resultDenominator == 0) {
throw (new Throwable(exceptionMessage));
} else {
result[0] = resultNumerator;
result[1] = resultDenominator;
}
} catch (Throwable e) {
System.out.println(e.getMessage());
}
return result;
}
}
测试:
class CalculatorTest {
@Test
void denominatorContainsZero() {
assertThrows(Throwable.class, () -> {
Calculator.calculate(0,0,0,0);
});
}
}
Throwable
被 try catch
块捕获,因此 Junit 无法访问它。尝试删除 try catch
块。
你实际上并没有抛出异常,你正在捕获它。为此,您应该删除 try catch 块。
这里的误解似乎在于 JUnit 实际可以看到什么。
JUnit 并不神奇:它只是普通的旧 Java。它看不到你的方法内部,看不到它们在做什么。它所能看到的是任何其他代码在执行方法时所能看到的:return 值和未捕获的异常(以及该方法的任何副作用,如果它们对调用代码可见的话)。
从调用者的角度来看,您的方法不会抛出异常:在内部,它会抛出异常,但会捕获并处理它。
如果您希望 JUnit 测试是否抛出了异常,则不需要捕获该异常。
抛出异常然后自己捕获并处理它从来都不是 (*) 正确的做法。重点是什么?你可以简单地做你做的事情来处理它,而不会抛出异常。抛出异常代价高昂,因为需要捕获整个堆栈跟踪。
Throwable
永远不会 (*) 抛出正确的异常。它是 returning Object
的异常 "equivalent":它没有向调用者传达有关异常的类型信息,调用者要么必须做大量工作来尝试处理它;要么或者,更现实地说,应该自己传播它。如果您确实需要抛出(而不是捕获)异常,IllegalArgumentException
是在此处抛出的正确异常。
Throwable
很少是正确的捕捉对象。 Throwable
是 Exception
和 Error
的超类型,因此您可能会无意中捕捉到 Error
,例如 OutOfMemoryError
,它不应该被捕捉,因为有除了让你的程序崩溃之外,没有什么可以做的。尽可能抓住最具体的类型;这也意味着您应该尽可能抛出最具体的类型(或者至少是适合抽象的类型)。
(*) 这是 "never",如 "ok, there are a limited number of circumstances where it may be appropriate"。但是除非你明白这些是什么,否则不要。