18

如果没有从任何地方调用 foo()，我想删除 foo()。

4

7 回答 7

21

宪兵将检测没有上游调用者的私有方法。它可以跨平台使用，最新版本处理“ AvoidUncallPrivateCodeRule ”。

FxCop将检测没有上游调用者的公共/受保护方法。但是，FxCop 不会检测没有上游调用者的所有方法，因为它旨在检查您的代码是否属于库的一部分，因此会忽略公共成员。您可以使用 NDepend 搜索没有上游调用者的公共成员，我在另一个 StackOverflow 答案中对此进行了详细说明。

（编辑：添加了关于宪兵的信息，实际上是按照提问者的要求做的）

于 2008-09-15T18:47:50.473 回答
7

NDepend还将报告可能未使用的代码。

于 2008-09-15T19:38:04.347 回答
5

请记住，如果方法被标记，Resharper（可能还有其他类似的工具）不会突出显示未使用的方法public。静态代码分析工具无法检查您的程序集的方法是否被您的解决方案之外的其他程序集使用。因此，淘汰未使用方法的第一步是降低它们对privateor的可见性internal。

于 2008-09-15T19:04:17.643 回答
3

Resharper做到了这一点，而不仅仅是方法。它也可以通过 using 语句、变量等来实现。

于 2008-09-15T18:47:43.547 回答
3

是的，MZ-Tools插件具有审查死代码功能。

于 2008-09-15T18:48:01.410 回答
3

NDepend工具可以帮助在 .NET 代码库中查找未使用的代码。免责声明：我是这个工具的开发者之一。

NDepend 建议在 LINQ 查询 (CQLinq) 上编写代码规则。提出了大约200 条默认代码规则，其中 3 条专用于未使用/死代码检测：

NDepend 集成在 Visual Studio 中，因此可以在 IDE 中检查/浏览/编辑这些规则。该工具还可以集成到您的 CI 流程中，它可以构建报告，显示违反规则和罪魁祸首代码元素。

如果你点击这三个链接指向这些规则的源代码，你会发现关于类型和方法的部分有点复杂。这是因为它们不仅检测未使用的类型和方法，还检测仅由未使用的死类型和方法（递归）使用的类型和方法。

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

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

于 2010-10-19T13:19:56.403 回答
2

好吧，如果 VS 本身不这样做，一个简单的方法是右键单击该方法并选择“查找所有引用”。如果只有 1 个引用（声明它的地方），它很可能不会在其他任何地方使用。

于 2008-09-15T18:48:43.640 回答