Was ist eine bessere Java-Syntax: if (isSomething() == false) { oder if (!isSomething()) {

Question

Wenn du dir ansiehst

if (!x) {
if (x == false) {

Sieht es so aus, als ob !x besser ist, aber

if (!isSomething()) {
if (isSomething() == false) {

kannst du leicht das ! übersehen

Was ist zu tun? Gibt es eine Empfehlung?

Answer 1

Die versteckte dritte Option besteht darin, Ihre Variablen und Methoden ordnungsgemäß zu benennen.

Statt

if (!isDisabled()) {
    ...
}

verwenden Sie

if (isEnabled()) {
    ...
}

oder wenn Sie das Negative überprüfen möchten:

boolean disabled = !isEnabled();
if (disabled) {
    ...
}

oder fügen Sie beide Methoden hinzu:

boolean isDisabled() {
    return !isEnabled();
}

Bearbeitung: Ich habe diese Frage gefunden: Ist es schlecht, explizit gegen boolesche Konstanten zu vergleichen, z. B. if (b == false) in Java?

Answer 2

Ich würde bei der Notation if (!isSomething()) { bleiben. Wenn es für dich oder andere schwer zu lesen ist, kannst du immer ein wenig Leerzeichen um das '!' hinzufügen, um es hervorzuheben:

if ( ! isSomething()) { oder if ( !isSomething()) {

Zusätzlich können mehrere bedingte Aussagen mit der folgenden Notation überwältigend werden

if (isSomething() == false && isSomethingElse() == false && ..),

während die Alternative kurz und prägnant ist. Nach einer Weile wird es natürlich, das '!' zusammen mit den Aussagen als "nicht isSomething() und nicht isSomethingElse()" zu lesen.

Answer 3

Ich glaube nicht, dass es eine Empfehlung gibt, der jeder folgen würde.

Mach es auf deine Art, persönlich würde ich den if (!isSomething()) Stil wählen :)

Vor allem, da ich bereits den if (!x) Stil gewählt habe.

Answer 4

if ( !isSomething() ) {

wäre meiner Meinung nach am besten. Auf diese Weise halten Sie die Zeichenanzahl niedrig, Ihr Code ist lesbar und das ! sticht am Anfang heraus, so dass andere allein beim Überfliegen des Codes dessen Absicht sehen können.