if、else if、else文の機能性と可読性

Question

タイトルの言葉の正確な言い方ではありませんが、一般的な質問はこれらの2つのコードブロックを比較しています。最初のものは元のもので、2番目のものはそれを置き換えるものです。

これは実際にコードを改善しますか、それは効率か、可読性を向上させるだけでしょうか？私は何かが欠けていない限り、彼らは機能的に異なるとは思わない。

1ST：

if(stl1YVal < stl2YVal){ 
     return -1; 
    }else if(stl1YVal > stl2YVal){ 
     return 1; 
    }else{ 
     if(stl1XVal < stl2XVal){ 
      return -1; 
     }else if(stl1XVal > stl2XVal){ 
      return 1; 
     }else{ 
      return 0; 
     } 
    } 

2ND：

if(stl1YVal < stl2YVal || stl1XVal < stl2XVal){ 
     return -1; 
    }else if(stl1YVal > stl2YVal || stl1XVal > stl2XVal){ 
     return 1; 
    }else { 
     return 0; 
    } 

編集：私は場合...それは人々に少しより多くの意味を行うことができるように全体の方法を入れています人々はそれをもっと読みやすくて機能的にするための答え/解決策を持っています（2番目のものは同等ではないようです）。

（彼らがあるため、単純なテキスト行のSTL命名されている - 私はそれらに名前を付けていなかった）

public int compare(SimpleTextLine stl1, SimpleTextLine stl2){ 
     //0 -> stl1 and stl2 are equivalent, maintain current order 
     //-1 -> stl1 comes before stl2 
     //1 -> stl2 comes before stl1 

     float stl1XVal = Math.round(stl1.getxLinePos()); 
     float stl1YVal = Math.round(stl1.getyLinePos()); 
     float stl2XVal = Math.round(stl2.getxLinePos()); 
     float stl2YVal = Math.round(stl2.getyLinePos()); 

     if(stl1YVal < stl2YVal){ 
      return -1; 
     }else if(stl1YVal > stl2YVal){ 
      return 1; 
     }else{ 
      if(stl1XVal < stl2XVal){ 
       return -1; 
      }else if(stl1XVal > stl2XVal){ 
       return 1; 
      }else{ 
       return 0; 
      } 
     } 
    } 

Answer 1

は、私は元のコードの「クリーン」バージョンは、これらの一つだと思い：

// Option 1 (eliminating redundant 'else' clauses and nesting) 
if (stl1YVal < stl2YVal) 
    return -1; 
if (stl1YVal > stl2YVal) 
    return 1; 
if (stl1XVal < stl2XVal) 
    return -1; 
if (stl1XVal > stl2XVal) 
    return 1; 
return 0; 

// Option 2 (using conditional ternary operator) 
return (stl1YVal < stl2YVal ? -1 : 
     stl1YVal > stl2YVal ? 1 : 
     stl1XVal < stl2XVal ? -1 : 
     stl1XVal > stl2XVal ? 1 : 0); 

私はオプション2が素晴らしいと思います。オプション1が中括弧（{}）を必要とするかどうかは、あなたのコーディング基準に従います。

Answer 2

何かが紛失していない限り、機能的に異なるとは思いません。

この簡単なテスト検討してください：

public class Test { 

    public static void main(String[] args) { 

     for(int y1=-1;y1<2;y1++){ 
      for(int y2=-1;y2<2;y2++){ 
       for(int x1=-1;x1<2;x1++){ 
        for(int x2=-1;x2<2;x2++){ 
         System.out.format(" %2d %2d %2d %2d ", y1, y2, x1, x2); 
         System.out.print(" ---- "); 
         int first = first(y1, y2, x1, x2); 
         int second = second(y1, y2, x1, x2); 
         System.out.format(" %2d %2d %b\n", first, second, first == second); 
        } 
       } 
      } 

     } 
    } 

    public static int first(int stl1YVal, int stl2YVal, int stl1XVal, int stl2XVal) { 
     if (stl1YVal < stl2YVal) { 
      return -1; 
     } else if (stl1YVal > stl2YVal) { 
      return 1; 
     } else { 
      if (stl1XVal < stl2XVal) { 
       return -1; 
      } else if (stl1XVal > stl2XVal) { 
       return 1; 
      } else { 
       return 0; 
      } 
     } 
    } 

    public static int second(int stl1YVal, int stl2YVal, int stl1XVal, int stl2XVal) { 
     if (stl1YVal < stl2YVal || stl1XVal < stl2XVal) { 
      return -1; 
     } else if (stl1YVal > stl2YVal || stl1XVal > stl2XVal) { 
      return 1; 
     } else { 
      return 0; 
     } 
    } 
} 

を結果が等しくないいくつかの例が含まれ、その結果を検討してください：

-1 -1 -1 -1 ---- 0 0 true 
-1 -1 -1 0 ---- -1 -1 true 
-1 -1 -1 1 ---- -1 -1 true 
-1 -1 0 -1 ---- 1 1 true 
-1 -1 0 0 ---- 0 0 true 
-1 -1 0 1 ---- -1 -1 true 
-1 -1 1 -1 ---- 1 1 true 
-1 -1 1 0 ---- 1 1 true 
-1 -1 1 1 ---- 0 0 true 
-1 0 -1 -1 ---- -1 -1 true 
-1 0 -1 0 ---- -1 -1 true 
-1 0 -1 1 ---- -1 -1 true 
-1 0 0 -1 ---- -1 -1 true 
-1 0 0 0 ---- -1 -1 true 
-1 0 0 1 ---- -1 -1 true 
-1 0 1 -1 ---- -1 -1 true 
-1 0 1 0 ---- -1 -1 true 
-1 0 1 1 ---- -1 -1 true 
-1 1 -1 -1 ---- -1 -1 true 
-1 1 -1 0 ---- -1 -1 true 
-1 1 -1 1 ---- -1 -1 true 
-1 1 0 -1 ---- -1 -1 true 
-1 1 0 0 ---- -1 -1 true 
-1 1 0 1 ---- -1 -1 true 
-1 1 1 -1 ---- -1 -1 true 
-1 1 1 0 ---- -1 -1 true 
-1 1 1 1 ---- -1 -1 true 
    0 -1 -1 -1 ---- 1 1 true 
    0 -1 -1 0 ---- 1 -1 false 
    0 -1 -1 1 ---- 1 -1 false 
    0 -1 0 -1 ---- 1 1 true 
    0 -1 0 0 ---- 1 1 true 
    0 -1 0 1 ---- 1 -1 false 
    0 -1 1 -1 ---- 1 1 true 
    0 -1 1 0 ---- 1 1 true 
    0 -1 1 1 ---- 1 1 true 
    0 0 -1 -1 ---- 0 0 true 
    0 0 -1 0 ---- -1 -1 true 
    0 0 -1 1 ---- -1 -1 true 
    0 0 0 -1 ---- 1 1 true 
    0 0 0 0 ---- 0 0 true 
    0 0 0 1 ---- -1 -1 true 
    0 0 1 -1 ---- 1 1 true 
    0 0 1 0 ---- 1 1 true 
    0 0 1 1 ---- 0 0 true 
    0 1 -1 -1 ---- -1 -1 true 
    0 1 -1 0 ---- -1 -1 true 
    0 1 -1 1 ---- -1 -1 true 
    0 1 0 -1 ---- -1 -1 true 
    0 1 0 0 ---- -1 -1 true 
    0 1 0 1 ---- -1 -1 true 
    0 1 1 -1 ---- -1 -1 true 
    0 1 1 0 ---- -1 -1 true 
    0 1 1 1 ---- -1 -1 true 
    1 -1 -1 -1 ---- 1 1 true 
    1 -1 -1 0 ---- 1 -1 false 
    1 -1 -1 1 ---- 1 -1 false 
    1 -1 0 -1 ---- 1 1 true 
    1 -1 0 0 ---- 1 1 true 
    1 -1 0 1 ---- 1 -1 false 
    1 -1 1 -1 ---- 1 1 true 
    1 -1 1 0 ---- 1 1 true 
    1 -1 1 1 ---- 1 1 true 
    1 0 -1 -1 ---- 1 1 true 
    1 0 -1 0 ---- 1 -1 false 
    1 0 -1 1 ---- 1 -1 false 
    1 0 0 -1 ---- 1 1 true 
    1 0 0 0 ---- 1 1 true 
    1 0 0 1 ---- 1 -1 false 
    1 0 1 -1 ---- 1 1 true 
    1 0 1 0 ---- 1 1 true 
    1 0 1 1 ---- 1 1 true 
    1 1 -1 -1 ---- 0 0 true 
    1 1 -1 0 ---- -1 -1 true 
    1 1 -1 1 ---- -1 -1 true 
    1 1 0 -1 ---- 1 1 true 
    1 1 0 0 ---- 0 0 true 
    1 1 0 1 ---- -1 -1 true 
    1 1 1 -1 ---- 1 1 true 
    1 1 1 0 ---- 1 1 true 
    1 1 1 1 ---- 0 0 true