查找未使用的代码

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

1> Jarrett Meye..：

---

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

---

这很棒.没有足够的人知道这一点.您还必须启用解决方案范围分析才能显示所有内容.

---

Resharper是一个很好的工具,但我发现这个任务不可靠.我有一个公共方法,我删除了所有引用.如果我右键单击该方法并选择Show Usages,则没有,但Resharper的代码问题不会将其列为未使用.

---

@ user890155那是因为该方法是公共的,该库可能被不在当前解决方案中的另一个应用程序使用.我相信它只会将内部和私有方法标记为代码问题(如果未使用).

---

我们正在使用依赖注入.结果,所有东西看起来都用于重新收缩,因为即使是未使用的类型仍然使用统一注册.

---

@elggarc关于依赖注入,请看一下这里提到的Agent Mulder插件:http://blogs.jetbrains.com/dotnet/2012/08/resharper-70-plug-ins/项目主页:http://hmemcpy.github .com/AgentMulder/Agent Mulder - 支持依赖注入框架,如Autofac,Castle Windsor,Unity.由于ReSharper不了解这些容器,因此类通常可以标记为未使用或未实例化.代理Mulder在使用这些类时告诉ReSharper,并为每个类的注册点提供导航.

2> Jeff Schumac..：

---

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

一个伟大的工具浮现在脑海:

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

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

---

另一个警告,如果您的应用程序是asp.net,使用NDepend,您将需要预编译您的站点,以便您可以分析代码隐藏和NDepend无法覆盖/了解来自aspx页面的调用(即ObjectDataSources中的方法调用和喜欢)

3> mmiika..：

---

像其他人所说的那样,Resharper对此有好处.但要小心,这些工具找不到反射使用的代码,例如无法知道反射是否使用了某些代码.

4> Patrick from..：

---

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

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

基本上这样的规则来检测未使用的方法,例如:

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

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

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

可能死的方法

可能死的菲尔兹

NDepend集成在Visual Studio 2017,2015,2013,2012,2010中,因此可以在IDE内部检查/浏览/编辑这些规则.该工具还可以集成到您的CI流程中,它可以构建报告,显示违反规则和罪魁祸首的代码元素.NDepend还有一个VS Team Services扩展.

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

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

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

免责声明:我为NDepend工作.

5> 小智..：

---

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