java: bucle en los dos valores booleanos (falso, verdadero)

Question

Esta es una pregunta estilística. Quiero hacer un bucle dos veces con una variable on que se establece en falso, luego en verdadero. ¿Cuál de estos es más claro:

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) algo más

---

edición: ley de interpretaciones no deseadas de Murphy entra en juego. .. el caso de uso que originalmente me pareció algo así en lugar de 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); 
     } 
    } 
} 

Pero creo que me gusta la respuesta de Brendan, sólo voy a refactorizar el contenido de bucle en un método separado:

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

Dado que se trata de dos líneas, que acababa de saltar el lazo y hacer:

doStuffBasedOnABooleanFlag(false); 
doStuffBasedOnABooleanFlag(true); 

Menos código, más obvio, más eficiente.

Answer 2

Si realmente desea utilizar un bucle, yo iría con (a). Si bien es novedoso, también es claro y eficiente. Podría mover la matriz booleana a una estática privada, para evitar volver a crear la matriz cada vez.

Pero me gusta más la respuesta de Brendan.

Answer 3

Otra opción sería evitar el booleano y el uso de una enumeración:

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

Se podría entonces o iterar:

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

O hacer las llamadas directamente:

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

La ventaja de esto sería que la API indica más claramente lo que está sucediendo.

Answer 4

No es solo el ciclo, también estoy muy incómodo con el uso de booleanos de esta manera.

¿Qué pasa algo como:

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

Eso todavía deja el booleano dentro del oyente, pero sin saber lo que hace el método doStuffLater eso es por lo que podemos llegar.