Discussion :

[juninho269]

Bonjour,
Mon problème est je pense assez simple. J'ai une classe startStopButton qui étend Button. Cette classe possède une donnée membre etatOn qui marche de cette manière :
- etatOn = false -> stop
- etatOn = true -> start
Je défini un constructeur par defaut qui ne reçoit aucun paramètre. Avec ce constructeur le bouton sera à "stop".
Par contre je veux un deuxième constructeur qui attend un booléen en paramètre. Si ce paramètre est à false, j'appelle le constructeur par defaut pour ne pas réécrire de code. Par contre si le paramètre est à true je veux que le bouton se crée directement à "start".
J'appelle donc "super()" après avoir testé la valeur du paramètre et mon erreur est la suivante : L'appel de super() doit être la première chose à faire.

Voici ma classe :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
 
public class startStopButton extends Button {
 
    // Données membres
    private boolean etatOn;
 
    // Constructeur par defaut : Button stop
    public startStopButton() {
        super("Off");
        setBackground(Color.red);
        this.etatOn = false;
    }
 
    // Constructeur à un paramètre
    public startStopButton(boolean etatOn) {
        if(etatOn) {
            super("On");
            setBackground(Color.GREEN);
            this.etatOn = etatOn;
        } else {
            new startStopButton();
        }
    }
 
}

Que dois-je faire pour utiliser super après le test de mon paramètre ?

[the-gtm]

tu ne peux pas, par contre tu peux faire une méthode init que tu appele depuis le constructeur avec en paramètre on/off

[juninho269]

ok j'essayerais. Mais que se passe t-il si je fait sa :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
 
public class startStopButton extends Button implements ActionListener {
 
    // Données membres
    private boolean etatOn;
 
    // Constructeur par defaut : Button stop
    public startStopButton() {
        super("Start");
        setBackground(Color.GREEN);
        this.etatOn = false;
        addActionListener(this);
    }
 
    // Constructeur à un paramètre
    public startStopButton(boolean etatOn) {
        if(etatOn) {
            new Button("Stop");
            setBackground(Color.RED);
            this.etatOn = etatOn;
            addActionListener(this);
        } else {
            new startStopButton();
        }
    }
}

[natha]

Bien sûr que si il peut (et heureusement) ! mais autrement =>

Code :

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

1
2
3
4
5
6
7
8
9
10
11
    // Constructeur par defaut : Button stop
    public startStopButton() {
        this(false);
    }
 
    // Constructeur à un paramètre
    public startStopButton(boolean etatOn) {
            super(etatOn ? "On" : "Off");
            setBackground(etatOn ? Color.GREEN : Color.red);
            this.etatOn = etatOn;
    }

[natha]

ok j'essayerais. Mais que se passe t-il si je fait sa :
[...]

Ca peut pas marcher ton truc. Si tu fais "new" tu appelles un autre constructeur, tu ne construit pas l'instance courante.

[juninho269]

Merci natha sa marche parfaitement bien ! J'avoue que le problème était bête mais bon je n'en voyais pas la solution

[natha]

Par contre tu devrais essayer de respecter les conventions de nommage de Sun !

Une classe commence par une majuscule, les méthodes commencent par des minuscules (hormis les constructeurs mais ce ne sont pas des "méthodes"), etc.

Ca rendra ton code plus lisible pour les habitués.