IsNullOrEmptyを使用しない理由

Question

客先のレビューにて、下記のコードが指摘を受けました。
指摘内容が気になり質問をさせて頂きます。

private void button1_Click(object sender, EventArgs e)
{
  if (!String.IsNullOrEmpty(textBox1.Text))   // ※
  {
    // 処理
    // 処理
    // 処理
  }
}

※の箇所が指摘されました。
指摘内容は、呼出コストを考えると
if (textBox1.Text != "")
{
}
のようにするべきと指摘されました。
単純に考えるとメソッドを呼び出すので
コストがかかると思いますが、気にするほどとは思えない。

textBox1.Text != ""が問題なく比較できても
私は、勘違いや誤解を招くこと防ぐ為
IsNullOrEmptyを使用しています。

呼出コスト以外でも
if (textBox1.Text != "")
の方が良いことがありますでしょうか？

Answer 1

下記コードをMono 4.2.1 (OS X 10.11.3)で試して見ましたが、!=の方が3倍ぐらい遅いです。また、!=もString.Inequality 演算子 (String, String)というメソッドなので、メソッド呼び出しになると思われます。また、nullの場合に結果が異なります。

と言うことで、良い事なんて無いようです。下記コードをレビュアーに叩きつけてやって下さい。
※ Visual C#ではは未確認なので、ご注意を。

using System;
using System.Diagnostics;

class Yobidashi
{
    [STAThread]
    public static void Main(string[] args)
    {
        const long times = 1000000000;
        String text = "a";
        Console.WriteLine("TimeMethod: " + TimeMethod(times, text));
        Console.WriteLine("TimeEqual: " + TimeEqual(times, text));
        text = "";
        Console.WriteLine("TimeMethod: " + TimeMethod(times, text));
        Console.WriteLine("TimeEqual: " + TimeEqual(times, text));
        text = null;
        Console.WriteLine("TimeMethod: " + TimeMethod(times, text));
        Console.WriteLine("TimeEqual: " + TimeEqual(times, text));
    }
    public static TimeSpan TimeMethod(long times, String text) {
        Stopwatch sw = new Stopwatch();
        long count = 0;
        sw.Start();
        for (long i = 0; i < times; i++) {
            if (!String.IsNullOrEmpty(text)) {
                count++;
            }
        }
        sw.Stop();
        Console.WriteLine(count == times ? "ok" : "ng");
        return sw.Elapsed;
    }
    public static TimeSpan TimeEqual(long times, String text) {
        Stopwatch sw = new Stopwatch();
        long count = 0;
        sw.Start();
        for (long i = 0; i < times; i++) {
            if (text != "") {
                count++;
            }
        }
        sw.Stop();
        Console.WriteLine(count == times ? "ok" : "ng");
        return sw.Elapsed;
    }
}

Comments

[ozwk]

VC#で試しましたが!=のほうが遅いですね
ReferenceSourceを見ても
!=のほうが処理重そうです。

[sho_cs]

IsNullOrWhiteSpaceを使った場合でも!=と同等か少し遅くなるくらいです。
そもそもボタンのイベントから呼び出される単発処理のようなのでコストは無視できるでしょう。

[ai_2013_dev]

raccy さん
ご回答ありがとうございます。
よく考えると!=は展開されることを忘れていました。
IsNullOrEmptyより複雑なコードを展開するようです。

ozwk さん
確かに、ReferenceSourceで調べると
処理が重そうです。

sho_cs さん
>ボタンのイベントから呼び出される単発処理のようなのでコストは無視できるでしょう。
単発処理ですのでコストは無視できますね。

回答を踏まえて、客先のレビュアーへの回答とさせて頂きます

[toraneko]

!=よりも==のが速いのでどうしてもコストうんぬん気になるなら、
IsNullOrEmptyメソッドも↓と同じ処理なので、このようにした方が速いようです。
if (string == null || string.Length == 0)
{
}