コードスタイルJava：プライベートメソッドからの戻り値のバブリング

Question

私は、プライベートメソッドでいくつかの計算を行い、それがnullでないときにプライベートメソッドの戻り値をバブルアップする状況に直面することがよくあります。私が書きたい何

public String myMethod(int value) { 
    String tmp = strategy1(value); 
    if (tmp != null) { 
     return tmp; 
    } 
    tmp = strategy2(value) 
    if (tmp != null) { 
     return tmp; 
    } 
    return "default result" 
} 

private String strategy1(int value) { 
    return value > 1? "Good value" : null; 
} 

はこのような何か（動作しないJavaコード）

public String myMethod(int value) { 
    return if(strategy1(value) != null); 
    return if(strategy2(value) != null); 
    return "default result" 
} 

うまくこれを達成するための任意のベストプラクティスを次のとおりです。私はこれを行うよりもよりよい方法があるはずと感じ？

Answer 1

valueをメソッドに渡す必要がある場合は、nullの戻り値よりも例外をスローする必要がある場合は、nullを返してください。 で有効なJavaを読んでください。あなたは何かを検証しようとした場合、あなたはヌルで取引をすると、それにそれが動作しない外部のコードをご確認くださいより良いリターンブールだろう

public String myMethod(int value) { 
     if (value < 1) { 
      throw new IllegalArgumentException(); 
     } 
} 

：これを考えてみましょう、あなたのコードは、そのようなものでなければなりません。さらに良いアプローチは、あなたが値を取得しようとすると、それは悪いそれが例外をスローすることを、悪いときインターフェースValue

interface Value<T> { 

    boolean isValid(); 

    T getValue() throws IllegalStateException; 

} 

一定値をincapsulateと値が良いとき trueを返し

とfalseを作成することです

public boolean isValid(int value) { 
      return (value > 1); 
} 

;あなたが最初の戦略が戻った後、短circuiting--サポートすることを可能にするインターフェイスを使用して

public interface Strategy { 
    String evaluate(int); 
} 

public String firstResult(int candidate, String defaultValue, Strategy... strategies) { 
    for (Strategy strategy : strategies) { 
     String result=strategy.evaluate(candidate); 
     if (result!=null) return result; 
    } 
    return defaultValue; 
} 

：あなたの戦略は「戦略」インタフェースの実装であれば

Answer 2

は、あなたは、このようなユーティリティメソッドを使用することができますNULL以外の結果は、他の戦略のどれも処理されません。

のJava 8の使用：

System.out.println(firstResult(12, "snot-nosed kid", 
      age -> age >= 18 ? "adult" : null, 
      age -> age > 12 ? "teen" : null, 
      age -> age < 5 ? "pre-school" : null 
    )); 

のJava 7使用法：

Strategy strategy1=new Strategy() { 
     public String evaluate(int candidate) { 
      return candidate >=18 ? "adult" : null; 
     } 
    }; 
    Strategy strategy2=new Strategy() { 
     public String evaluate(int candidate) { 
      return candidate >12 ? "teen" : null; 
     } 
    }; 
    Strategy strategy3=new Strategy() { 
     public String evaluate(int candidate) { 
      return candidate < 5 ? "pre-school" : null; 
     } 
    }; 

    System.out.println(firstResult(12, "snot-nosed kid", strategy1, strategy2, strategy3));