java 中的异常不会因 `if` 条件不满足而自动触发;必须显式调用 `throw` 语句才能抛出异常,进而被 `catch` 捕获。当前代码中 `try` 块内从未实际抛出 `nosuchelementexception`,因此 `catch` 永远不会执行。
在 Java 中,“执行一个异常”这一说法本身并不准确——异常不是被“执行”的,而是被显式抛出(throw) 并由运行时机制传递给匹配的 catch 块处理。你提供的 remove(int value) 方法声明了 throws NoSuchElementException,但这仅表示该方法可能向上级抛出该异常,并不意味着它会自动抛出;真正的抛出动作必须由开发者主动编写。
以你的需求为例——当 value 不存在于链表中时应抛出 NoSuchElementException,那么逻辑应是:
@Override
public void remove(int value) throws NoSuchElementException {
if (!contains(value)) {
throw new NoSuchElementException("Element " + value + " not found.");
}
// 执行删除逻辑(无需包裹在 try-catch 中)
ListItem index = head;
if (head == tail) {
head = tail = null;
} else if (head.value == value) {
head = head.next;
} else {
while (index.next != null && index
.next.value != value) {
index = index.next;
}
if (index.next != null) {
index.next = index.next.next;
if (index.next == null) {
tail = index; // 维护 tail 指针(如需)
}
}
}
}? 注意:原代码中 try-catch 完全多余——NoSuchElementException 是未检查异常(unchecked)(继承自 RuntimeException),即使不声明 throws 也能编译通过;但若你刻意使用它(如本例),更推荐直接 throw 而非依赖空 catch,因为:catch 中又 throw 同一异常,属于冗余包装;try 块内无任何可能抛出该异常的语句,catch 永远不可达(编译器甚至可能警告 unreachable catch block)。
要让 catch 生效,必须确保:
回到你的场景:删除前先校验存在性,不存在则立即 throw;存在则安全执行删除。这才是清晰、健壮且符合 Java 异常设计意图的做法。