Discussion :

[JoeChip]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 ?

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

[wax78]

ou bien le code B:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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]

[professeur shadoko]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
private volatile boolean enabled ;

[adiGuba]

Salut,


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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
a.setEnabled( condition );
 
// VS :
 
if (condition)
    enabled();
else
    disabled();

a++

[JoeChip]

sinon ya :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[adiGuba]

sinon ya :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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++

[JoeChip]

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...

[toutgrego]

Moi des fois je fais :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
private boolean enabled=true;
 
    public void setEnabled() {
        enabled = !enabled;
    }

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

[rg77140]

Pourquoi pas mixer A et B :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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.