CA1837: Gunakan Environment.ProcessId, bukan Process.GetCurrentProcess().Id

Properti Nilai
ID Aturan CA1837
Judul Gunakan Environment.ProcessId, bukan Process.GetCurrentProcess().Id
Kategori Performa
Perbaikan bersifat merusak atau tidak merusak Tidak terputus
Diaktifkan secara default di .NET 10 Sebagai saran
Bahasa yang berlaku C# dan Visual Basic

Penyebab

Aturan ini menemukan panggilan ke System.Diagnostics.Process.GetCurrentProcess().Id dan menyarankan penggunaan System.Environment.ProcessId sebagai gantinya, karena lebih efisien.

Deskripsi aturan

System.Diagnostics.Process.GetCurrentProcess().Id mahal:

  • Ini mengalokasikan instans Process, yang umumnya hanya digunakan untuk mendapatkan Id.
  • Instans Process perlu dihapus karena memengaruhi performa.
  • Mudah untuk lupa memanggil Dispose() pada instans Process.
  • Jika tidak ada metode lain selain Id yang menggunakan instans Process, maka ukuran yang ditautkan akan bertambah secara tidak perlu dengan memperbesar grafik tipe yang dirujuk.
  • Cukup rumit untuk menemukan atau mendapatkan API ini.

System.Environment.ProcessId hindari semua hal di atas.

Catatan

Aturan CA1837 tersedia sejak .NET 5.0.

Cara memperbaiki pelanggaran

Pelanggaran dapat diperbaiki secara manual, atau, dalam beberapa kasus, menggunakan Tindakan Cepat untuk memperbaiki kode di Visual Studio.

Dua cuplikan kode berikut menunjukkan pelanggaran aturan dan cara memperbaikinya:

using System.Diagnostics;

class MyClass
{
    void MyMethod()
    {
        int pid = Process.GetCurrentProcess().Id;
    }
}
Imports System.Diagnostics

Class MyClass
    Private Sub MyMethod()
        Dim pid As Integer = Process.GetCurrentProcess().Id
    End Function
End Class
using System.Diagnostics;

class MyClass
{
    void MyMethod()
    {
        int pid = System.Environment.ProcessId;
    }
}
Imports System.Diagnostics

Class MyClass
    Private Sub MyMethod()
        Dim pid As Integer = System.Environment.ProcessId
    End Function
End Class

Tips

Perbaikan kode tersedia untuk aturan ini di Visual Studio. Untuk menggunakannya, posisikan kursor pada pelanggaran dan tekan Ctrl+. (titik). Pilih Gunakan 'Environment.ProcessId', bukan 'Process.GetCurrentProcess().Id' dari daftar opsi yang ditampilkan.

Perbaikan kode untuk CA1837 - Gunakan 'Environment.ProcessId' alih-alih 'Process.GetCurrentProcess(). Id'

Kapan harus menekan peringatan

Anda dapat mengabaikan pelanggaran aturan ini jika tidak khawatir tentang dampak performa akibat alokasi yang tidak diperlukan dan pada akhirnya penghapusan instans Process.

Menyembunyikan peringatan

Jika Anda hanya ingin menyembunyikan satu pelanggaran, tambahkan arahan praprosedur ke file sumber Anda untuk dinonaktifkan lalu aktifkan kembali aturannya.

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

Untuk menonaktifkan aturan untuk file, folder, atau proyek, atur tingkat keparahannya ke none dalam file konfigurasi.

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

Untuk informasi selengkapnya, lihat Cara menyembunyikan peringatan analisis kode.

Lihat juga