.net - null / 空文字列の扱い

Question

.NET 4 がリリースされる前は、null/空の文字列をチェックするために次のようなことを行っていました。

String s;
if ((s + "").Trim().Length == 0)
{
    //do something
}

上記の方法は私たちにとって非常にうまく機能しますが、.NET 4 ではIsNullOrWhiteSpace同様のことを行う方法があることを知っています。

私の質問は、どちらが良いですか? 代わりに組み込み関数を使用するように切り替える必要がありますか?

score 4 · Accepted Answer

.NET 4 では、間違いなく組み込みメソッドを使用します。結局のところ、それはあなたがやろうとしていることを正確に言います.

.NET 4 より前で行き詰まっている場合は、次を使用してください。

if (s == null || s.Trim() == "")

または

if (s == null || s.Trim().Length == 0)

? どちらもあなたが達成したいことを言っています。

ここでは、文字列連結は絶対に使用しません。パフォーマンスはさておき、文字列の連結には関心がありません。達成しようとしていることの一部ではないことをコードが実行しているのを見つけたときはいつでも、可読性のためにそれを改善するように努めるべきです。

score 2 · Accepted Answer

IsNullOrWhiteSpace.NET 4より前の機能を実装するためのヘルパーメソッドを作成してみませんか？と同じように

public static boolean IsNullOrWhiteSpace(string input)
{
    return string.IsNullOrEmpty(input) || input.Trim() == string.Empty;
}

ジョンが言ったように、ここでは連結を使用しないでください。null/空をチェックするのは良い習慣ではありません。

score 2 · Accepted Answer

私は個人的にを使用していますIsNullOrWhiteSpace。主な理由は、これを使用するとコードが読みやすくなり、より多くのケース (WhiteSpace部分) を処理できるためです。どちらの方法もほとんど同じことを行うため、好みによって異なります。

3 に答える 3