Java: Schleife auf den zwei booleschen Werten (falsch, wahr)

Question

Dies ist eine stilistische Frage. Ich möchte zweimal mit einer Variablen on, die auf false gesetzt ist, und dann auf true. Welche von diesen ist klarer:

A)

for (final boolean on : new boolean[] { false, true}) 
{ 
    doStuffBasedOnABooleanFlag(on); 
} 

B)

for (int i = 0; i < 2; ++i) 
{ 
    final boolean on = (i == 1); 
    doStuffBasedOnABooleanFlag(on); 
} 

C) etwas anderes

---

edit: Murphys Gesetz der unbeabsichtigten Interpretationen ins Spiel kommt. .. der Use Case, den ich ursprünglich so ähnlich sah statt doStuffBasedOnABooleanFlag:

for (final boolean on : new boolean[] { false, true}) 
{ 
    JButton button = on ? onButton : offButton; 
    button.addActionListener(new ActionListener() { 
     @Override public void actionPerformed(ActionEvent event) { 
     doStuffLaterBasedOnABooleanFlag(on); 
     } 
    } 
} 

Aber ich denke, ich Brendan Antwort mag, ich werde nur die Loop-Inhalt in eine separate Methode Refactoring:

doStuffBasedOnABooleanFlag(false); 
doStuffBasedOnABooleanFlag(true); 

    ... 

private void doStuffBasedOnABooleanFlag(final boolean on) 
{ 
    JButton button = on ? onButton : offButton; 
    button.addActionListener(new ActionListener() { 
     @Override public void actionPerformed(ActionEvent event) { 
     doStuffLaterBasedOnABooleanFlag(on); 
     } 
    } 
} 

Answer 1

Da es zwei Linien ist, würde ich nur die Schleife überspringen und zu tun:

doStuffBasedOnABooleanFlag(false); 
doStuffBasedOnABooleanFlag(true); 

Weniger Code, offensichtlicher, effizienter.

Answer 2

Wenn Sie wirklich eine Schleife verwenden möchten, würde ich mit (a) gehen. Obwohl es neuartig ist, ist es auch klar und effizient. Ich könnte das boolesche Array in eine private Static verschieben, um das Array nicht jedes Mal neu zu erstellen.

Aber ich mag Brendan's Antwort besser.

Answer 3

wäre eine weitere Option, die boolean zu vermeiden und eine ENUM verwenden:

for(Mode m : Mode.values()) doStuff(m); 

Oder die Anrufe direkt tun:

enum Mode { APPEND, REPLACE } // or whatever your boolean indicated 

Sie dann iterieren könnte entweder

doStuff(Mode.APPEND); 
doStuff(Mode.REPLACE); 

Der Vorteil davon wäre, dass die API deutlicher anzeigt, was passiert.

Answer 4

Es ist nicht nur die Schleife, ich bin auch wirklich unwohl mit der Verwendung von Booleans auf diese Weise.

Was ist so etwas wie:

ActionListener myListener = new ActionListener() { 
    @Override 
    public void actionPerformed(ActionEvent event) { 
     doStuffLaterBasedOnABooleanFlag(event.getSource() == onButton); 
    } 
    }; 
    onButton.addActionListener(myListener); 
    offButton.addActionListener(myListener); 

, die noch die boolean innerhalb des Hörers lässt, aber ohne zu wissen, was die doStuffLater Methode tut, das ist so weit wir gehen können.