是否有自定义FxCop规则来检测未使用的PUBLIC方法？

我刚试过FxCop.它确实检测未使用的私有方法,但不检测未使用的公共 是否有可以下载的自定义规则,插件将检测未在同一程序集中调用的公共方法？

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

// Potentially unused methods
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的"关于耦合,死代码,设计缺陷和重新设计的代码指标.该文章还涉及检测死区和类型.

(编辑:回答更容易理解)

编辑2012年6月11日:解释有关未使用代码的新NDepend工具.免责声明:我是此工具的开发人员之一.

自2012年5月发布NDepend v4以来,该工具建议在LINQ查询(CQLinq)上编写代码规则.提出了大约200个默认代码规则,其中3个专用于未使用/死代码检测:

可能死的类型(因此检测未使用的类,结构,接口,委托...)

可能死的方法(因此检测未使用的方法,ctor,属性getter/setter ......)

可能死的菲尔兹

这些CQLinq代码规则比以前的CQL代码规则更强大.如果单击上面这三条链接指向这些规则的源代码,您将看到有关类型和方法的链接有点复杂.这是因为它们不仅检测未使用的类型和方法,还检测仅由未使用的死类型和方法(递归)使用的类型和方法.

这是静态分析,因此在规则名称中可能是潜在的前缀.如果仅通过反射使用代码元素,则这些规则可能会将其视为未使用,而不是这种情况.

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

如果方法未使用且公共FxCop假定您已将其公开以供外部访问.

如果未使用的公共方法导致FxCop警告编写API等等将是一件痛苦的事情 - 您需要为其他人使用的方法提供大量FxCop警告.

如果您不需要任何外部访问您的程序集/ exe,请考虑public使用internal.您的应用程序将运行相同,FxCop将能够找到未引用的内部方法.

如果你确实需要外部访问,那么找到真正需要哪些方法是外部的,并将所有其他方法都放在内部.

您在外部可见的任何方法也可以进行单元测试.