我必须重构一个大型的c#应用程序,我发现了许多从未使用过的函数。我如何检查未使用的代码,所以我可以删除所有未使用的函数?
当前回答
这是一个很好的问题,但请注意,你正在踏入危险的水域。当你删除代码时,你必须确保你经常编译和测试。
我想到了一个很棒的工具:
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).
无论你选择什么工具,都不能掉以轻心。特别是当你在处理库类型程序集上的公共方法时,因为你可能永远不知道应用程序什么时候引用了它们。
其他回答
我还要提一下,使用IOC也就是Unity可能会让这些评估产生误导。我可能有错误,但几个非常重要的类,通过Unity实例化似乎没有实例化,就ReSharper可以告诉。如果我按照ReSharper的建议去做,我就会被洗掉!
是的,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工作。
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).
无论你选择什么工具,都不能掉以轻心。特别是当你在处理库类型程序集上的公共方法时,因为你可能永远不知道应用程序什么时候引用了它们。
推荐文章
- .Net HttpWebRequest.GetResponse()在返回http状态码400(坏请求)时引发异常
- 在using块的中间返回
- 找到未使用的代码
- 是什么让Visual Studio调试器停止评估ToString重写?
- ContentControl和ContentPresenter有什么区别?
- 一个字符串的字符串列表
- 无法确定条件表达式的类型,因为'int'和<null>之间没有隐式转换
- 从实体框架中删除单个记录?
- 创建一个完成的任务
- 字典的字面符号在c# ?
- 随机字符串生成器返回相同的字符串
- 为什么Func<T,bool>而不是Predicate<T>?
- .NET中的Map和Reduce
- 我如何能使一个组合框不可编辑的。net ?
- .NET反射的成本有多高?