Question

以下代码在Eclipse中给出了一个“死代码”警告：

private void add(Node<E> n, E element) {
        Node<E> e = new Node<E>(element);
        if (n == null)
            root = e;
        else if (n.compareTo(e) > 0)
            if (n.hasLeft())
                add(n.getLeft(), element);
            else
                n.setLeft(e);
        else if (n.hasRight())
            add(n.getRight(), element);
        else
            n.setRight(e);
        balance(e);
    }

警告显示在显示root = e;的行中。

我查了死代码，发现代码帽无效，因此会被java编译器忽略。

但是，这个root是我班级中的一个私有字段，因此我的程序功能必须这样做。

编译器真的会忽略这个吗？我怎么能阻止它？为什么它认为它是死代码？

Answer 1

如果root是您的类中包含您发布的add方法的私有字段，那么，正如您所说，Eclipse root = e;行不应被视为死代码IDE。

编译器应该正常工作......它只是一个IDE警告。

我的猜测是Eclipse会执行某种代码行走（类似于Cyclomatic复杂性工具）来确定代码路径并找到“死代码”和“无法访问的代码”。

我会尝试刷新，然后干净并在IDE中构建。如果这不能解决它，Eclipse可能会对死代码的警告产生“误报”。这不是第一次......我同时使用Eclipse和IntelliJ IDEA，并且看到两个IDE之前都错误地警告过代码。但是，尽管有IDE警告，我的代码仍可正常编译。

Answer 2

可能存在两个问题：第一：问题是线根不是“用于”任何东西。在findbugs中，相同的错误被称为“死存储”，每findbugs表示：

该指令为局部变量赋值，但不会在任何后续指令中读取或使用该值。通常，这表示错误，因为计算的值从未使用过。

请注意，此处的关键字是“常常”。

我会检查并确保root正在按照您的预期使用，如果可能的话，正如Philip Tenn所提到的那样，是误报。

第二：您的问题可能与this issue有关。

Answer 3

由于源不足以找到确切的原因，我猜它与eclipse中与死代码相关的问题有关。您可以检查一些问题here和here。

Answer 4

看看你的条件。

Node<E> e = new Node<E>(element);
if (n == null) {
    // dead code here
}

此代码已失效，因为n == null 总是为假。您刚刚在前一行（无条件地）创建了Node<E>的新实例。新对象永远不会为空。

Eclipse中不需要的死代码警告

4 个答案: