到達不能コード検出（なぜ？）

Question

だから、なぜそれがこのコードで検出されないコードがあるということです。

 public bool GetKeyPressed(KeyCode key) 
    { 
     for (int i = 0; i < keys.Count; i++) 
      if (keys[i].key == key && keys[i].pressed) 
       return true; 
      else 
       return false; 

     return false; 
    } 

インデックス（i）が明らかに到達できている...なぜ？

Answer 1

あなたのコードは一度評価ループ、意志常にリターンしたがって、最初の反復を持っています。

それはあなただけで、欲しいしかし、あなたはは、アレイ内の任意のは、あなたのテストを満たしている場合にtrueを返すようにする（私はここに疑いある）場合には、使用

return keys[0].key == key && keys[0].pressed; 

、と言うものだ場合LINQのAny()、jdphenix答えを上の拡大

return keys.Any(k => k.key == key && k.pressed); 

Answer 2

あなたのコードは2つのコードパスを持ち、1つはifです。もう1つはelseです。これは、コントロールがこれらの2つの方法で機能を残すことを意味します。したがって、elseの後のreturn文はトリガされません。そのため、コンパイラは到達不能なコードとしてそれを指しています。これは、次のコードを使用することで回避できます。

public bool GetKeyPressed(KeyCode key) 
{ 
    for (int i = 0; i < keys.Count; i++) 
     if (keys[i].key == key && keys[i].pressed) 
      return true;   
     return false; 
} 

Answer 3

は、私たちはなぜあなたのループを理解する必要があると思います「は一度と評価されます」というメッセージが表示され、「到達不能なコード」がある理由を理解することができます。

public bool GetKeyPressed(Keys key) 
{ 
    for (int i = 0; i < keys.Count; i++) 
     if (keys[i].key == key && keys[i].pressed) 
      return true; 
     else 
      return false; 

    return false; 
} 

forループは体を構成する単一if-elseの文を持っています。満足すればifガードはtrueを返します。それ以外の場合はelseステートメントを実行してfalseを返します。最終結果は、多くの場合、ループの1サイクルが実行され、keysの項目の数に関係なく、制御が呼び出しメソッドに戻されるということです。

それは我々がJetBrainsのReSharperのを経由して、コードを見れば、より明らかです。ミスをしないでください

public bool GetKeyPressed(Keys key) 
{ 
    for (int i = 0; i < keys.Count;) // Look Ma, no i++ !!! 
     if (keys[i].key == key && keys[i].pressed) 
      return true; 
     else 
      return false; 

    return false; 
} 

：

コードも同様のように書くことができます。最後のreturn falseは、シナリオ中であるため、このメソッドの最後にある必要はありません。keys.Count == 0

0もちろん

、もう少しきれいにコードをフォーマットすると、最初のreturn falseが冗長であるとun-lucky's answerあたりのように単純化することができないという問題を明らかに長い道のりを行く：