通过


CA1839:使用 Environment.ProcessPath 而不是 Process.GetCurrentProcess().MainModule.FileName

属性
规则 ID CA1839
标题 使用 Environment.ProcessPath 而不是 Process.GetCurrentProcess().MainModule.FileName
类别 “性能”
修复会引起中断还是不会引起中断 非中断
在 .NET 10 中默认启用 作为一种建议
适用的语言 C# 和 Visual Basic

原因

使用 Process.GetCurrentProcess().MainModule.FileName 来获取启动进程的文件的路径(而不是 Environment.ProcessPath)。

规则说明

System.Diagnostics.Process.GetCurrentProcess().MainModule.FileName 成本较高:

  • 它分配 ProcessProcessModule 实例,通常只为了获取 FileName
  • 需要处置 Process 实例,这会影响性能。
  • 很容易忘记在 Process 实例上调用 Dispose()
  • 如果除了 FileName 之外没有其他东西在使用 Process 实例,那么由于引用类型图的扩大,链接大小也会因此不必要地增长。
  • 发现或查找此 API 有点困难。

System.Environment.ProcessPath 避免了所有这些缺点并生成相同的信息。

注意

System.Environment.ProcessPath 从 .NET 6 开始可用。

如何解决违规

可以手动解决冲突,或者在某些情况下,使用快速操作来修复 Visual Studio 中的代码。

以下两个代码片段显示了规则冲突及其解决方法:

using System.Diagnostics;

class MyClass
{
    void MyMethod()
    {
        string path = Process.GetCurrentProcess().MainModule.FileName; // Violation occurs
    }
}
Imports System.Diagnostics

Class MyClass
    Private Sub MyMethod()
        Dim path As String = Process.GetCurrentProcess().MainModule.FileName ' Violation occurs.
    End Function
End Class
using System.Diagnostics;

class MyClass
{
    void MyMethod()
    {
        string path = System.Environment.ProcessPath; // Violation fixed
    }
}
Imports System.Diagnostics

Class MyClass
    Private Sub MyMethod()
        Dim path As String = System.Environment.ProcessPath ' Violation fixed.
    End Function
End Class

提示

Visual Studio 中为此规则提供了代码修补程序。 若要使用它,请将光标置于违规上,然后按 "Ctrl+.(句点)"。 从显示的选项列表中选择“使用Environment.ProcessPath”。

针对 CA1839 的代码修复:使用 Environment.ProcessPath

何时禁止显示警告

如果你不担心因ProcessProcessModule实例的不必要分配和最终处置对性能造成的影响,那么忽略此规则的违反是可以放心的。

抑制警告

如果只想抑制单个冲突,请将预处理器指令添加到源文件以禁用该规则,然后重新启用该规则。

#pragma warning disable CA1839
// The code that's violating the rule is on this line.
#pragma warning restore CA1839

若要对文件、文件夹或项目禁用该规则,请在none中将其严重性设置为

[*.{cs,vb}]
dotnet_diagnostic.CA1839.severity = none

有关详细信息,请参阅如何禁止显示代码分析警告

另请参阅