C# では文字列の比較に Equals を使うな

C# では文字列の比較に Equals を使わない方がいい。== を使った方がコンパイラの型チェックの恩恵を受けられるし、間違いを少なくすることができる。

Equals を使うな。使う事を推奨するな。
http://www.ailight.jp/blog/kazuk/archive/2006/01/31/11043.as ...

Equals はタイプセーフではない。このため、型のエラーの発見が遅れる。

例：

int a;
string b;
bool result = a.Equals( b );

int a;
string b;
bool result = (a==b);    // CS0019: 演算子 '==' を 'int' と 'string' 型のオペランドに適用することはできません。

よっぽどの理由が無い限り == で比較せよ。

Visual Studio 2003 環境で試してみた。int 型の 0 と string 型の 0 を比較する。

int i = 0;
string s = "0";

Console.WriteLine(i.ToString() == s);
Console.WriteLine(string.Equals(i.ToString(), s));
Console.WriteLine(string.Equals(i, s));
Console.WriteLine(s.Equals(i));

実行結果。上記コードを書いた「プログラマ」としては、比較結果はすべて True になって欲しいのだが、そうなっていない。

True
True
False
False

string.Equals() メソッドにはオーバーロードがあり、string 型同士を比較するものと、Object 型として比較するものがある。Object 型同士の比較だと int 型のオブジェクトと string 型でのオブジェクトという違いがあるので false になってしまう。

これ、バグの元だなあ。Perl だったら型変換があっても気にしないけど、C# というせっかく型の恩恵を受けられる言語を使ってるんだから、最大限に活用した方がいい。

Equals の方が速いとのことだが、上記の動きを理解した上で、100万回のループの中などの速度が重要な部分で使うならいい。でも、日常的に Equals を使うのは避けた方が間違いを減らせる。「速いらしいから」という理由で Equals を常用していた後輩は、上記の動きを知らなかった。