如果没有从任何地方调用foo(),我想删除foo()。
答案 0 :(得分:21)
Gendarme将检测没有上游调用者的私有方法。它可以跨平台使用,最新版本处理“AvoidUncalledPrivateCodeRule”。
FxCop将检测没有上游调用者的公共/受保护方法。但是,如果没有上游调用者,FxCop不会检测所有方法,因为它是为了检查您的代码是否是库的一部分,因此公共成员被排除在外。 You can use NDepend to do a search for public members with no upstream callers, which I detail here in this other StackOverflow answer
(编辑:添加了关于Gendarme的信息,实际上提出了提问者的要求)
答案 1 :(得分:7)
NDepend还会报告可能未使用的代码。
答案 2 :(得分:5)
请记住,如果方法标记为public
,Resharper(以及可能还有其他类似工具)将不会突出显示未使用的方法。静态代码分析工具无法检查解决方案之外的其他程序集是否使用了程序集的方法。因此,清除未使用方法的第一步是降低其对private
或internal
的可见性。
答案 3 :(得分:3)
Resharper执行此操作,而不仅仅是使用方法。它也可以使用语句,变量等来实现。
答案 4 :(得分:3)
是的,MZ-Tools插件具有审核死代码功能。
答案 5 :(得分:3)
工具NDepend可以帮助在.NET代码库中查找未使用的代码。 免责声明:我是此工具的开发人员之一。
NDepend建议写Code Rule over LINQ Query (CQLinq)。提议大约200 default code rules,其中3个专门用于未使用/死代码检测:
NDepend集成在Visual Studio中,因此这些规则可以是checked/browsed/edited right inside the IDE。该工具还可以集成到您的CI流程中,它可以构建reports,显示违反的规则和罪魁祸首代码元素。
如果单击这些规则的源代码的这3个链接,您将看到有关类型和方法的链接有点复杂。这是因为它们不仅检测未使用的类型和方法,还检测未使用的死类型和方法(递归)仅使用 的类型和方法。
这是静态分析,因此规则名称中的前缀可能。如果代码元素仅通过反射 使用,则这些规则可能会将其视为未使用,而不是这种情况。
除了使用这三条规则外,我还建议通过测试来测量代码覆盖率并努力实现全面覆盖。通常,您会看到测试无法覆盖的代码实际上是 unused / dead 代码,可以安全地丢弃。这在复杂的算法中尤其有用,因为不清楚代码的分支是否可达。
答案 6 :(得分:2)
好吧,如果VS本身不这样做,一个简单的方法是右键单击该方法并选择“查找所有引用”。如果只有一个引用(声明它的位置),则很可能不会在其他地方使用它。