2016-06-27 16 views
7

Otrzymuję ostrzeżenie "Metoda Boolean jest zawsze odwrócona" na uruchomionych kłaczkach w IntelliJ. Mam kilka podobnych ostrzeżeń w mojej bazie kodów. Jakiego podstawowego stylu kodowania brakuje?IntelliJ IDEA: Metoda Boolean XX jest zawsze odwrócona

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 próbują powrócić ' 'false''' jeśli' 'bluetoothAdapter''' ma wartość null, w przeciwnym razie zwraca wynik działania' isEnabled() '' ' –

Odpowiedz

9

spróbować wrócić false jeśli bluetoothAdapter jest zerowa inaczej powrócić wyjście isEnabled()

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

Więcej:

w czystej Kodeksu Robert Martin pisze: „Negatywne to tylko nieco trudniejsze zrozumieć, niż pozytywy. Tak więc, jeśli to możliwe, czynniki warunkujące powinny być wyrażone jako pozytywne. "(Martin, [G29]). IntelliJ IDEA ma trzy inspekcje , które pomogą Ci zachować pozytywny wynik.

https://blog.jetbrains.com/idea/2014/09/the-inspection-connection-issue-2/ (Wejście nr 4 unikania negatywnych warunkowe)

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

+0

, ale jaki jest pomysł/cel w przełączaniu wartości zwracanej (nie dostaję go z linku wysłałeś). – mobileDev

+0

zobacz edytowaną odpowiedź –