我必须重构一个大型的c#应用程序,我发现了许多从未使用过的函数。我如何检查未使用的代码,所以我可以删除所有未使用的函数?


ReSharper在寻找未使用的代码方面做得很好。

在VS IDE中,你可以右键单击定义并选择“查找全部” 引用,尽管这只适用于解决方案级别。


这是一个很好的问题,但请注意,你正在踏入危险的水域。当你删除代码时,你必须确保你经常编译和测试。

我想到了一个很棒的工具:

NDepend - this tool is just amazing. It takes a little while to grok, and after the first 10 minutes I think most developers just say "Screw it!" and delete the app. Once you get a good feel for NDepend, it gives you amazing insight to how your apps are coupled. Check it out: http://www.ndepend.com/. Most importantly, this tool will allow you to view methods which do not have any direct callers. It will also show you the inverse, a complete call tree for any method in the assembly (or even between assemblies).

无论你选择什么工具,都不能掉以轻心。特别是当你在处理库类型程序集上的公共方法时,因为你可能永远不知道应用程序什么时候引用了它们。


Resharper是好的,就像其他人所说的。但是要注意,这些工具不会找到反射使用的代码,例如不能知道某些代码没有被反射使用。


FXCop是一个代码分析器…它的功能远不止查找未使用的代码。我用了FXCop一段时间,对它的建议太着迷了,所以我把它卸载了。

我觉得NDepend看起来更有可能。


事实上,这个工具永远不能给你一个100%确定的答案,但是覆盖率工具可以让你的钱花得很值。

如果您使用全面的单元测试套件,那么您可以使用测试覆盖工具来查看在测试运行期间哪些代码行没有执行。您仍然需要手动分析代码:要么消除您认为的死代码,要么编写测试以提高测试覆盖率。

其中一个这样的工具是NCover,它在Sourceforge上有开源前驱。另一种选择是PartCover。

在stackoverflow上看看这个答案。


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


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

为了详细说明,NDepend提议在LINQ查询上编写代码规则(CQLinq)。大约提出了200个默认代码规则,其中3个用于检测未使用/死亡的代码

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

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

但是这个规则很幼稚,会返回微不足道的假阳性。有很多情况下,一个方法从未被调用,但它并没有被使用(入口点,类构造函数,终结器……)这就是为什么3个默认规则更详细:

潜在的死亡类型(因此检测未使用的类,结构,接口,委托…) 可能死亡的方法 潜在的死亡领域

NDepend集成在Visual Studio 2022年、2019年、2017年、2015年、2013年、2012年、2010年,因此这些规则可以在IDE中检查/浏览/编辑。该工具还可以集成到CI流程中,它可以构建报告,显示违反的规则和罪魁祸首代码元素。NDepend也有一个VS团队服务扩展。

如果您点击上面指向这些规则源代码的这3个链接,您将看到有关类型和方法的内容有点复杂。这是因为它们不仅检测未使用的类型和方法,而且还检测仅由未使用的死类型和方法使用的类型和方法(递归)。

这是静态分析,因此规则名中有前缀potential。如果一个代码元素仅通过反射使用,这些规则可能认为它未使用,但事实并非如此。

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

免责声明:我为NDepend工作。


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


我还要提一下,使用IOC也就是Unity可能会让这些评估产生误导。我可能有错误,但几个非常重要的类,通过Unity实例化似乎没有实例化,就ReSharper可以告诉。如果我按照ReSharper的建议去做,我就会被洗掉!