.net - 是否有自定义 FxCop 规则可以检测未使用的 PUBLIC 方法？

Question

我刚试过 FxCop。它确实检测到未使用的私有(private)方法，但未检测到未使用的公共(public)方法。是否有我可以下载的自定义规则，插件将检测未从同一程序集中调用的公共(public)方法？

Answer 1

Corey，我对使用 FxCop 的回答假设您有兴趣删除未使用的私有(private)成员，但是要解决其他情况下的问题，您可以尝试使用 NDepend .下面是一些用于检测未使用的公共(public)成员的 CQL(改编自下面列出的文章):

// <Name>Potentially unused methods</Name>
WARN IF Count > 0 IN SELECT METHODS WHERE
 MethodCa == 0 AND            // Ca=0 -> No Afferent Coupling -> The method 
                              // is not used in the context of this
                              // application.

 IsPublic AND                 // Check for unused public methods

 !IsEntryPoint AND            // Main() method is not used by-design.

 !IsExplicitInterfaceImpl AND // The IL code never explicitely calls 
                              // explicit interface methods implementation.

 !IsClassConstructor AND      // The IL code never explicitely calls class
                              // constructors.

 !IsFinalizer                 // The IL code never explicitely calls
                              // finalizers.

来源: Patrick Smacchia's "Code metrics on Coupling, Dead Code, Design flaws and Re-engineering .本文还讨论了检测死字段和类型。

(编辑:使答案更容易理解)

编辑 2012 年 6 月 11 日:解释有关未使用代码的新 NDepend 设施。免责声明:我是这个工具的开发者之一。

由于NDepend v4于2012年5月发布，该工具建议写 Code Rule over LINQ Query (CQLinq) .左右 200 default code rules建议，其中 3 个专用于未使用/死代码检测:

Potentially dead Types (因此检测未使用的类、结构、接口(interface)、委托(delegate)...)

Potentially dead Methods (因此检测未使用的方法、ctor、属性 getter/setter...)

Potentially dead Fields

这些 CQLinq 代码规则比以前的 CQL 更强大。如果你点击上面这3个链接看这些规则的源代码，你会发现关于类型和方法的部分有点复杂。这是因为它们不仅检测未使用的类型和方法，还检测仅由未使用的死类型和方法(递归)使用的类型和方法。

这是静态分析，因此规则名称中有潜在的前缀。如果仅通过反射使用代码元素，则这些规则可能会将其视为未使用，但事实并非如此。

除了使用这 3 条规则之外，我还建议通过测试来衡量代码覆盖率，并努力实现完全覆盖。通常，您会看到测试无法覆盖的代码实际上是可以安全丢弃的未使用/死代码。这在不清楚代码分支是否可达的复杂算法中特别有用。