.net 是否有用于检测未使用的公共方法的自定义FxCop规则?

.net 是否有用于检测未使用的公共方法的自定义FxCop规则?,.net,code-analysis,fxcop,public-method,.net,Code Analysis,Fxcop,Public Method,我刚试过。它检测未使用的私有方法,但不检测未使用的公共方法。是否有我可以下载的自定义规则,即可以检测未从同一程序集中调用的公共方法的插件?这类事情是您的朋友它如何知道公共方法未使用 通过将方法标记为public,任何引用您的库的应用程序都可以访问该方法。如果某个方法未使用且public FxCop假定您已将其公开,以便外部对象访问 如果未使用的公共方法导致FxCop警告,那么编写API之类的东西将是一件痛苦的事情——对于您打算让其他人使用的方法,您将收到大量FxCop警告 如果您不需要任何外部访

我刚试过。它检测未使用的私有方法,但不检测未使用的公共方法。是否有我可以下载的自定义规则,即可以检测未从同一程序集中调用的公共方法的插件?

这类事情是您的朋友

它如何知道公共方法未使用


通过将方法标记为public,任何引用您的库的应用程序都可以访问该方法。

如果某个方法未使用且public FxCop假定您已将其公开,以便外部对象访问

如果未使用的公共方法导致FxCop警告,那么编写API之类的东西将是一件痛苦的事情——对于您打算让其他人使用的方法,您将收到大量FxCop警告

如果您不需要任何外部访问您的程序集/exe,请考虑用<代码>内部< /代码>替换<代码>公共< /代码>。您的应用程序将运行相同的方法,FxCop将能够找到未引用的内部方法

如果您确实需要外部访问,请找到真正需要外部访问的方法,并将其余方法都设置为内部访问


任何使外部可见的方法也可以进行单元测试。

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

//可能未使用的方法
如果选择方法中的计数>0,则发出警告,其中
MethodCa==0和//Ca=0->无传入耦合->方法
//未在本文上下文中使用
//应用程序。
IsPublic和//检查未使用的公共方法
!设计未使用IsEntryPoint和//Main()方法。
!IsExplicitInterfaceImpl和//IL代码从不显式调用
//显式接口方法实现。
!IsClassConstructor和//IL代码从不显式调用类
//构造器。
!IsFinalizer//IL代码从不显式调用
//终结器。
来源:。本文还讨论了死区和类型的检测

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


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

自2012年5月发布NDepend v4以来,该工具建议编写。提出了四种方法,其中三种专门用于未使用/死代码检测:

  • (因此检测未使用的类、结构、接口、委托…)
  • (因此检测未使用的方法、ctor、属性getter/setter…)
这些CQLinq代码规则比以前的CQL规则更强大。如果你点击上面的3个链接来查看这些规则的源代码,你会发现关于类型和方法的链接有点复杂。这是因为它们不仅检测未使用的类型和方法,还检测仅由未使用的死类型和方法使用的类型和方法(递归)

这是静态分析,因此规则名称中可能包含前缀。如果只通过反射使用代码元素,则这些规则可能会认为它是未使用的,而不是这样。p>
除了使用这3条规则外,我还建议通过测试测量代码覆盖率,并努力实现完全覆盖。通常,您会看到测试无法覆盖的代码实际上是可以安全丢弃的未使用/死代码。这在复杂算法中尤其有用,因为在复杂算法中,不清楚代码分支是否可访问。

NDepend非常棒,但我不知道它如何检测未使用的公共方法。@Ian Nelson:我在下面回答了如何使用NDepend搜索这类东西。我编辑了我的问题,使其更清晰。我只是对我的一个程序集感兴趣,一个exe,顺便说一下,不是dll。
// <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.