可読性の点でどちらが優れていますか？

Question

あなたは3つの条件をチェックする必要があり、場合によっては、この例のように、同じ値を返す場合：

 if (designatorType != AttributeDesignator.ENVIRONMENT_TARGET) 
      return new EvaluationResult(BagAttribute. 
             createEmptyBag(attributeType)); 

     if (! attributeId.toString().equals("processor-load")) 
      return new EvaluationResult(BagAttribute. 
             createEmptyBag(attributeType)); 

     if (! attributeType.toString().equals(IntegerAttribute.identifier)) 
      return new EvaluationResult(BagAttribute. 
             createEmptyBag(attributeType)); 

は、この

 if (designatorType != AttributeDesignator.ENVIRONMENT_TARGET) || 
      (! attributeId.toString().equals("processor-load")) || 
      (! attributeType.toString().equals(IntegerAttribute.identifier)) || 
      return new EvaluationResult(BagAttribute. 
             createEmptyBag(attributeType)); 

このようにそれを書き直すほうがよいのではないだろうおそらく非常に主観的ですが、私はまだ誰かが有効な答えを与えることを願っています:-)。

Answer 1

public void someMethod() 
{ 
    if (isConditionMet()) 
    return new EvaluationResult(BagAttribute.createEmptyBag(attributeType)); 
} 

private boolean isConditionMet() 
{ 
    if (designatorType != AttributeDesignator.ENVIRONMENT_TARGET)) 
    return true; 
    if (! attributeId.toString().equals("processor-load")) 
    return true; 
    if (! attributeType.toString().equals(IntegerAttribute.identifier)) 
    return true; 
    return false; 
} 

Answer 2

全く混乱しません。同じ事後条件（結果）につながる3つの独立した前提条件があります。

Answer 3

私は条件のための別の方法を作成します。

その後、

private boolean shouldCreateEmptyBag(...){ 

    return (designatorType != AttributeDesignator.ENVIRONMENT_TARGET) || 
      (! attributeId.toString().equals("processor-load")) || 
      (! attributeType.toString().equals(IntegerAttribute.identifier)) || ... 
} 

：

if(shouldCreateEmptyBag(...)){ 
    return new EvaluationResult(BagAttribute. 
             createEmptyBag(attributeType)); 
} 

Answer 4

私は最初のものを好む - 私は1つの文では、それらの倍数があるときにあまりにも多くの人々が彼らの事業者が混乱してき見てきました。

Answer 5

if (isTest1(designatorType) || isTest2(attributeId) || isTest3(attributeType)){ 
    return new EvaluationResult(BagAttribute.createEmptyBag(attributeType)); 
} 

Answer 6

これらのコードはどちらも同じです。最初はもっと明確で、互いに独立した条件で分かれています。しかし、第2のものは、これらの条件の組み合わせのみです。 IMOは、最初の方が柔軟性があり理解できるものです。将来変更した場合は、変更する場所と変更する場所を理解しやすくなります。