Pour en avoir le coeur net

Tiens dans le genre où ya moyen de s'étriper un moment ptet aussi :aie: vous préférez quoi ?
Le code A :

Code:

---

1 2 3 4 5 6 7 8 9

[tout ce qu'il faut avant] private boolean enabled = false; public void enable(){ enabled = true; } public void disable(){ enabled = false; } [tout ce qu'il y a après]

---

ou bien le code B:

Code:

---

1 2 3 4 5 6

[tout ce qu'il faut avant] private boolean enabled = false; public void setEnabled(final boolean enabled){ this.enabled = enabled; } [tout ce qu'il y a après]

---

Moi, sans hésiter, A. Alors ? on s'étripe ou bien ? :mrgreen:

Bon, en vérité pour moi le disable et le enable ne seraient pas void, mais renverraient this... Question subsidiaire, quoi.

Citation:

---

Envoyé par JoeChip

ou bien le code B:

Code:

---

1 2 3 4 5 6

[tout ce qu'il faut avant] private boolean enabled = false; public void setEnabled(final boolean enabled){ this.enabled = enabled; } [tout ce qu'il y a après]

---

---

ou bien le code C...

Code:

---

1 2 3 4 5 6

[tout ce qu'il faut avant] private boolean enabled = false; public void setDisabled(final boolean disabled){ this.enabled = (disabled==true)?false:true; } [tout ce qu'il y a après]

---

:aie: ;)

je parie un grain de sel contre un grain de poivre que c'est

Code:

---

1 2	private volatile boolean enabled ;

---

Salut,


Perso je préfère largement le setEnabled() aux enabled()/disabled(), car c'est plus pratique à utiliser :

Code:

---

1 2 3 4 5 6 7 8

a.setEnabled( condition );   // VS :   if (condition) enabled(); else disabled();

---

a++

sinon ya :

Code:

---

condition ? enable() : disable();

---

Je sais, ça peut paraître de mauvaise foi, mais c'est justement un des cas où l'opérateur ternaire se justifie, finalement. Comme quoi je peux évoluer dans mes avis... J'en ai même mis 2 dans mes sources récemment, suite à la discussion précédente :mouarf:

Citation:

---

Envoyé par JoeChip

sinon ya :

Code:

---

condition ? enable() : disable();

---

---

Je préfère quand même le setEnabled(condition), que je trouve bien plus lisible...


Mais surtout l'opérateur ternaire me sort une erreur "not a statement" lorsque je l'utilise comme cela (sans affectation).


a++

Citation:

---

Mais surtout l'opérateur ternaire me sort une erreur "not a statement" lorsque je l'utilise comme cela (sans affectation).

---

Ah oui flûte, décidément j'ai du mal avec l'OT...

Moi des fois je fais :

Code:

---

1 2 3 4 5

private boolean enabled=true;   public void setEnabled() { enabled = !enabled; }

---

Mais je ne sais pas si c'est top x)

Pourquoi pas mixer A et B :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

public void enable(){ setEnabled(true); }   public void disable(){ setEnabled(false); }   public void setEnabled(final boolean enabled){ this.enabled = enabled; }

---

@toutgrego ça ne me parait pas une bonne idée... A la limite faudrait appeler la méthode "switchEnable()" mais je vois pas trop l'intérêt.