查找未使用的代码

Question

我必须重构一个大的C＃应用​​程序，我发现了很多从未使用过的函数。如何检查未使用的代码，以便删除所有未使用的功能？

Answer 1

是的，ReSharper会这样做。右键单击您的解决方案并选择“查找代码问题”。其中一个结果是“未使用的符号”。这将显示未使用的类，方法等。

Answer 2

这是一个很好的问题，但请注意，你在这里遇到危险的水域。当您删除代码时，您必须确保经常进行编译和测试。

想到一个很棒的工具：

NDepend - 这个工具真是太神奇了。需要一段时间才能理解，在前10分钟之后，我认为大多数开发人员只是说“拧它！”并删除该应用程序。一旦您对NDepend有了良好的感觉，它将为您提供有关应用程序耦合方式的惊人见解。看看：http://www.ndepend.com/。最重要的是，此工具将允许您查看没有任何直接调用方的方法。它还将显示逆向，即程序集中任何方法（甚至是程序集之间）的完整调用树。

无论你选择什么工具，轻描淡写都不是一项任务。特别是如果您正在处理库类型程序集的公共方法，因为您可能永远不知道应用程序何时引用它们。

Answer 3

Resharper就像其他人所说的那样对此有好处。但请注意，这些工具找不到反射使用的代码，例如无法知道反射是否使用了某些代码。

Answer 4

正如Jeff所指出的，工具NDepend可以帮助找到未使用的方法，字段和类型。

为了详细说明，NDepend建议写Code Rule over LINQ Query (CQLinq)。建议大约200 default code rules，其中3个专门用于未使用/死代码检测

基本上这样一个检测未使用方法的规则如下：

// <Name>Dead Methods</Name>
warnif count > 0 
from m in Application.Methods where !m.MethodsCallingMe.Any()
select m

但是这条规则很幼稚，会带来琐碎的误报。在许多情况下，一个方法永远不会被调用但它没有被使用（入口点，类构造函数，终结器......）这就是为什么3个默认规则更精细：

• Potentially dead Types（因此检测未使用的类，结构，接口，委托......）
• Potentially dead Methods
• Potentially dead Fields

NDepend在Visual Studio 2017,2015,2013,2012,2010中集成，因此这些规则可以是checked/browsed/edited right inside the IDE。该工具还可以集成到您的CI流程中，它可以构建reports，显示违反规则和罪魁祸首的代码元素。 NDepend还有VS Team Services extension。

如果单击上面这三条链接指向这些规则的源代码，您会看到有关类型和方法的链接有点复杂。这是因为它们不仅检测未使用的类型和方法，还检测未使用的死类型和方法（递归）仅使用 的类型和方法。

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

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

免责声明：我为NDepend工作。

Answer 5

我还要提到使用IOC aka Unity可能会使这些评估产生误导。我可能有错，但是通过Unity实例化的几个非常重要的类似乎没有ReSharper可以告诉的实例化。如果我遵循ReSharper的推荐，我就会受到冲击！

Answer 6

ReSharper可以很好地找到未使用的代码。

在VS IDE中，您可以右键单击定义并选择“全部查找” 引用'，虽然这只适用于解决方案级别。

Answer 7

事实是，该工具永远不会给你一个100％肯定的答案，但覆盖工具可以为你提供相当不错的运行。

如果您使用全面的单元测试套件，那么您可以使用测试覆盖率工具查看测试运行期间未执行的代码行。您仍然需要手动分析代码：要么删除您认为的死代码，要么编写测试以提高测试覆盖率。

其中一个工具是NCover，Sourceforge上有开源前体。另一种选择是PartCover。

在stackoverflow上查看此answer。

Answer 8

我遇到过AXTools CODESMART ..尝试一次。 在评论部分使用代码分析器。它将列出死的本地和全局函数 其他问题。

Answer 9

FXCop是一个代码分析器...它不仅仅是找到未使用的代码。我使用FXCop已经有一段时间了，并且在我的卸载建议中迷失了它。

我认为NDepend看起来更像是候选人。