次の方法で共有


CA1874: 'Regex.IsMatch' を使用する

プロパティ 価値
ルール識別子 CA1874
タイトル Regex.IsMatch を使用する
カテゴリ パフォーマンス
修正が破壊的であるか非破壊的であるか Non-breaking
.NET 10 で既定で有効 提案として

原因

Regex.Matchからの結果のSuccessプロパティは、パターンが一致するかどうかを確認するために使用されます。

ルールの説明

Regex.IsMatch は、 Regex.Match(...).Successよりもシンプルで高速です。 IsMatchメソッドは、一致するものではなく、一致が存在するかどうかを知る必要がある場合にのみ最適化されています。 Match()を呼び出してからSuccessを確認すると、パフォーマンスに影響する可能性のある不要な作業が行われます。

違反を修正する方法

Regex.Match(...).Successの呼び出しをRegex.IsMatch(...)に置き換えます。

この変換を自動的に実行する コード修正 プログラムを使用できます。

Example

次のコード スニペットは、CA1874 の違反を示しています。

using System.Text.RegularExpressions;

class Example
{
    public bool IsValidEmail(string email)
    {
        // Violation
        return Regex.Match(email, @"^[^@\s]+@[^@\s]+\.[^@\s]+$").Success;
    }
}
Imports System.Text.RegularExpressions

Class Example
    Public Function IsValidEmail(email As String) As Boolean
        ' Violation
        Return Regex.Match(email, "^[^@\s]+@[^@\s]+\.[^@\s]+$").Success
    End Function
End Class

次のコード スニペットは、違反を修正します。

using System.Text.RegularExpressions;

class Example
{
    public bool IsValidEmail(string email)
    {
        // Fixed
        return Regex.IsMatch(email, @"^[^@\s]+@[^@\s]+\.[^@\s]+$");
    }
}
Imports System.Text.RegularExpressions

Class Example
    Public Function IsValidEmail(email As String) As Boolean
        ' Fixed
        Return Regex.IsMatch(email, "^[^@\s]+@[^@\s]+\.[^@\s]+$")
    End Function
End Class

警告を抑制するタイミング

パフォーマンスが問題でない場合は、この規則からの警告を抑制しても安全です。

警告を抑制する

単一の違反を抑制するだけの場合は、ソース ファイルにプリプロセッサ ディレクティブを追加して無効にしてから、規則をもう一度有効にします。

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

ファイル、フォルダー、またはプロジェクトのルールを無効にするには、その重大度をnoneに設定します。

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

詳細については、「 コード分析の警告を抑制する方法」を参照してください。

こちらも参照ください