2012-01-31 16 views
3

ブール関数内のswitch文にはこれがあります。私は休憩を追加するか、それは私が非常に悪いことを暗示していますか?偽を返した後に中断する必要がありますか?

case Stop: 
default: 
    return false; 
//break;?????? 
+2

通常、 'default'はスイッチの最後のケースであるため、' break'は二重に冗長になります。それは戻りのために到達不能であり、いずれにしても 'switch 'の最後のものとしての' break 'は、とにかく" fall out "する同じ場所にジャンプするだけです。スイッチ。しかし、何かをしているかどうかにかかわらず、いくつかのスタイルガイドは、 'case'が' break'か、 'fall through 'のようなコメントを持つことを望んでいます。それが必要とされています。 –

+1

@SteveJessopもちろん、 'default'はswitch文の最後にする必要はありません。そして何らかの理由で多くの人が 'switch 'の最後に新しい' case'ステートメントを追加するだけなので、最終的なケース/デフォルトフォールスルーを終わらせないようにするのが良い予防策と思われます。 –

答えて

8

defaultから返ってもらえない場合は、改行記号は必要ありません。

break文は、操作するすべてのケースの後でのみ追加する必要があります。それ以外の場合は、関数から戻り値が返されます。

+0

私はこれを "あなたがどんなケースからも戻ってくるなら"に修正します。 –

+0

もちろん、そうです。 –

4

これは暗黙のうちではありませんが、コードは決してそこに到達しないので、break;を書く必要はありません。

3

ちょうど冗長で、returnで十分です。

5

returnは、関数で最後に実行されるステートメントであるため、改行は必要ありません。

3

前にreturnステートメントがある場合、プログラムの実行はbreakステートメントに到達しません。

2

復帰後、プログラムはブレークに達しないので、そこからステートメントを削除できます。

関連する問題