java: boucle sur les deux valeurs booléennes (false, true)

Question

Ceci est une question de style. Je veux boucler deux fois avec une variable on qui est définie sur false, puis sur true. Lequel de ces est plus clair:

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) quelque chose d'autre

---

modifier: la loi de Murphy d'interprétations inattendues entre en jeu. .. le cas d'utilisation que j'ai à l'origine ressemblé à quelque chose comme ceci au lieu 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); 
     } 
    } 
} 

Mais je pense que j'aime la réponse de Brendan, je vais juste Refactor le contenu de la boucle dans une méthode distincte:

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

Depuis ses deux lignes, je venais de sauter la boucle et faire:

doStuffBasedOnABooleanFlag(false); 
doStuffBasedOnABooleanFlag(true); 

Moins de code, plus évident, plus efficace.

Answer 2

Si vous voulez vraiment utiliser une boucle, j'irais avec (a). Bien que ce soit nouveau, c'est aussi clair et efficace. Je pourrais déplacer le tableau booléen vers un static privé, pour éviter de recréer le tableau à chaque fois.

Mais j'aime mieux la réponse de Brendan.

Answer 3

Une autre option serait d'éviter le booléen et utiliser un ENUM:

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

Vous pouvez alors soit itérer:

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

Ou les appels directement:

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

L'avantage de ceci serait que l'API indique plus clairement ce qui se passe.

Answer 4

Ce n'est pas seulement la boucle, je suis également très mal à l'aise avec l'utilisation de booléens de cette façon.

Qu'en est-il quelque chose comme:

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

Cela laisse encore le booléen au sein de l'auditeur, mais sans savoir ce que la méthode doStuffLater fait c'est aussi loin que nous pouvons aller.