2016-06-27 1 views
7

Ich bekomme eine Warnung "Boolean Methode ist immer invertiert" beim Ausführen von Flusen in IntelliJ. Ich habe mehrere ähnliche Warnungen in meiner Codebasis. Welchen grundlegenden Programmierstil vermisse ich?IntelliJ IDEA: Boolesche Methode XX ist immer invertiert

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 versuchen 'zurückzukehren' 'false''' wenn' '' bluetoothAdapter''' ist null sonst gib die Ausgabe von '' 'isEnabled()' '' –

Antwort

9

versuchen zurückzukehren false wenn bluetoothAdapter null sonst die Ausgabe von isEnabled()

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

Rückkehr Lesen Sie mehr:

In Clean Code, Robert Martin schreibt: „Negatives nur sind etwas härter zu verstehen als positive. Also, wenn möglich, sollten die Bedingungen als positiv ausgedrückt werden. "(Martin, [G29]). IntelliJ IDEA hat drei Inspektionen, um Ihnen zu helfen, positiv zu bleiben.

https://blog.jetbrains.com/idea/2014/09/the-inspection-connection-issue-2/ (Eintrag # 4 Vermeidung negativer Conditionals)

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

+0

zurück aber was ist die Idee/der Zweck beim Umschalten des Rückgabewertes (ich bekomme es nicht von der Verbindung du hast geposted). – mobileDev

+0

siehe bearbeitete Antwort –