| プロパティ | 価値 |
|---|---|
| ルール識別子 | 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
詳細については、「 コード分析の警告を抑制する方法」を参照してください。
こちらも参照ください
.NET