干し草の針の空白文字列を確認するにはどうすればよいですか？

Question

状況：私は自分のコードで多くの小切手を渡しています。そして私はそれらを減らす方法を知りたいと思います。

if(needle!=null && haystack!=null) 
{ 
    if(needle.length()==0) 
    return true; 
    else 
    { 
    if(haystack.length()==0) 
    return false; 
    else 
    { 
    // Do 2 for loops to check character by character comparison in a substring 
    } 
} 

} 
else 
return false; 

Answer 1

おそらく、異なるコードスタイルは、あなたのコードの可読性を高め、あなたの小切手のすべてのためにネストされたif文の量が減少する。：

if (needle == null || haystack == null || haystack.isEmpty()) 
    return false; 

if (needle.isEmpty()) 
    return true; 

// compare strings here and return result. 

Answer 2

その後、長さ== 0の使用は常にはisValidにfalseを返しNull Object（）が解消するかどうかを確認すること、彼らにはisValid（）のような機能を追加し、文字列のためのラッパークラスを作成することができますヌルチェック。

あなたはあなたではなくヌルあなたのコード全体チェックする必要があり、文字列を渡すよりも、何をすべきか教えてクラスを作成することができた場合は、あなたが得るより多くのresuseable結果：あなたはにそのロジックを統合でき

class Haystack { 
    private static final Haystack NULL_HAYSTACK = new Haystack(""); 
    private final String value; 

    public Haystack(String value) { 
     this.value = value; 
    } 

    public boolean containsNeedle(String needle) { 
     return needle != null && value.contains(needle); 
    } 
} 

Answer 3

シングルトンの「StringFunctions」クラスのメソッドを1つ作成し、共通メソッドを使用するように用途を更新します。