blob: 521dab2a25007806d72e4d42b5ca7727f0c8e044 [file] [log] [blame] [view] [edit]
Catching an exception and throwing another is a common pattern. This is often
used to supply additional information, or to turn a checked exception into an
unchecked one.
```java
try {
ioLogic();
} catch (IOException e) {
throw new IllegalStateException(); // BAD
}
```
Throwing a new exception without supplying the caught one as a cause means the
stack trace will terminate at the `catch` block, which will make debugging a
possible fault in `ioLogic()` far harder than is necessary.
Prefer wrapping the original exception instead,
```java
try {
ioLogic();
} catch (IOException e) {
throw new IllegalStateException(e); // GOOD
}
```
Suppress false positives with `@SuppressWarnings("UnusedException")` on the
ignored exception. Consider also adding a comment to explain why the exception
should not be propagated.
```java
try {
...
} catch (@SuppressWarnings("UnusedException") IOException e) {
throw new IllegalStateException();
}
```