コード検査: 文字列比較を明示的に指定する

このインスペクションは、文字列変換で明示的にカルチャを指定すると同様に、異なるカルチャ設定のマシンでコードを実行する際の問題を回避できます。

String.ToUpper()/String.ToLower() によって返された文字列結果が等価演算子 (==/!=) を使用して比較される場合、比較結果はマシンのロケールによって異なる場合があります。標準的な例はトルコ語で、ドットなしの小文字の「ı」と対応する大文字の「I」、および小文字の「i」とドット付きの大文字の「İ」があります。

大文字と小文字を区別しない比較に String.ToUpper()/String.ToLower() を使用する場合は、次のようになります。

public void Foo(string a, string b)
{
  if (a.ToUpper() != b.ToUpper()) { }

  // or

  if (a.ToLower() != b.ToLower()) { }
}
        

  
Public Sub Foo(a As String, b As String)
    If a.ToUpper() <> b.ToUpper() Then
    End If

    ' or

    If a.ToLower() <> b.ToLower() Then
    End If
End Sub

ReSharper は、String.Equals() のオーバーロードに置き換えることを提案しています。

if (!String.Equals(a, b, StringComparison.CurrentCultureIgnoreCase)) {}

If Not String.Equals(a, b, StringComparison.CurrentCultureIgnoreCase) Then
End If
                

文化の設定を考慮して文字列が比較されることがすぐに分かります。

比較が明示的にカルチャに依存しない場合は、比較ルールパラメーターで String.Equals() と InvariantCultureIgnoreCase を使用することもお勧めします。例:

if (a.ToLowerInvariant() == b.ToLowerInvariant()) {}

If a.ToLowerInvariant() = b.ToLowerInvariant() Then
End If
        

と置き換えることができた

if (String.Equals(a, b, StringComparison.InvariantCultureIgnoreCase)) {}

If [String].Equals(a, b, StringComparison.InvariantCultureIgnoreCase) Then
End If
        

2024 年 2 月 11 日

関連ページ：

コードインスペクション設定を構成する

設計時コードインスペクションを構成する：デフォルトでは、ReSharper は、サポートされている言語に対応するすべてのファイルで設計時コードインスペクションを有効にします。必要に応じて、無効にすることができます。設計時コードインスペクションが有効かどうかに関係なく、コードインスペクションは常に特定のスコープで実行できます。メインメニューからを選択するか、Alt+R O を押してから、左側のを選択します。設計時コードインスペクションを切り替えるには、コード分析を有効にするチェックボックスを使用し...

コード検査: 明示的に文字列を指定する

データ構造のテキストへのアドホック変換は、現在のカルチャに大きく依存しており、ロケールが元の開発者と異なるマシンでコードを実行すると、意図しない結果につながる可能性があります。あいまいさを防ぐため、ReSharper は、このような問題が発生する可能性のあるコードのインスタンスを警告します。例: 次のコードを取る void Test(float foo) { Console.WriteLine(foo.ToString()); ; } はカルチャに依存しないと思われるかもしれませんが、実際にはそうで...