2016-06-27 29 views
7

IntelliJでlintを実行すると、「ブールメソッドは常に反転します」という警告が表示されます。私はコードベースにいくつか同様の警告があります。どの基本的なコーディングスタイル、私は行方不明ですか?IntelliJ IDEA:ブールメソッドXXは常に反転されています

public static boolean isBlueToothEnabled(){ 
    final BluetoothAdapter bluetoothAdapter = BluetoothAdapter.getDefaultAdapter(); 
    if(bluetoothAdapter != null) 
     return bluetoothAdapter.isEnabled(); 
    return false; 
} 
+1

https://www.jetbrains.com/help/idea/2016.1/invert-boolean.html '戻ろう'' '' 'bluetoothAdapter'''がnullの場合は' false'''を返します。そうでない場合は '' 'isEnabled()' '' –

答えて

9

bluetoothAdapterがnullの場合はそう続きを読むisEnabled()

public static boolean isBlueToothEnabled(){ 
    final BluetoothAdapter bluetoothAdapter = BluetoothAdapter.getDefaultAdapter(); 
    if(bluetoothAdapter == null){ 
     return false; 
    } 
    return bluetoothAdapter.isEnabled(); 
} 

の出力を返すfalseを返すようにしてください:クリーンコードで

、ロバート・マーティンは書き込み、「ネガだけです陽性よりも理解するためには、少し難しい 。したがって、可能な場合、条件式は を陽性として表現する必要があります。 "(Martin、[G29]) IntelliJ IDEAには、あなたが肯定的な態度を保つのに役立つ検査が3つあります( )。

https://blog.jetbrains.com/idea/2014/09/the-inspection-connection-issue-2/ (エントリー#4負条件文を回避する)

https://www.jetbrains.com/help/idea/2016.1/invert-boolean.html

+0

の出力を返しますが、戻り値を切り替える際のアイデアや目的は何ですか?あなたが投稿しました)。 – mobileDev

+0

編集された回答を表示 –

関連する問題