override de setText (JLabel)

**Sharcoux** · 21/02/2012, 15h04

Bonjour,

Je cherche à permettre l'affichage du point décimal sous sa forme française ( , ) dans un JLabel. Je pensais donc surcharger les méthodes getText et setText de mon JLabel de cette façon :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
        public static String MARQUEUR_DECIMAL = ",";
 
        @Override
            public String getText() {
                return super.getText().replace(MARQUEUR_DECIMAL, ".");
            }
 
        @Override
            public void setText(String texte) {
                System.out.println(texte.replace(".", MARQUEUR_DECIMAL));
                super.setText(texte.replace(".", MARQUEUR_DECIMAL));
            }

Problème :
Lorsque je test mon objet, j'obtiens un résultat assez déroutant :

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
27
28
29
30
31
32
33
34
35
public class Main {
 
    public static void main(String[] args) {
        JFrame a=new JFrame();
        EcranDAffichage ecran = new EcranDAffichage("");
        a.getContentPane().add(ecran);
        a.setDefaultCloseOperation(JFrame.DISPOSE_ON_CLOSE);
        a.setSize(220,80);
        a.setVisible(true);
        ecran.setText("0,1");
    }
 
    public static String MARQUEUR_DECIMAL = ",";
 
        public static class EcranDAffichage extends JLabel {
            public EcranDAffichage(String label) {
                super(label);
                setHorizontalAlignment(JLabel.RIGHT);
                setPreferredSize(new Dimension(220, 40));
            }
        @Override
            public String getText() {
                String texte = super.getText();
                return texte.replace(MARQUEUR_DECIMAL, ".");
            }
 
        @Override
            public void setText(String texte) {
                System.out.println(texte.replace(".", MARQUEUR_DECIMAL));
                super.setText(texte.replace(".", MARQUEUR_DECIMAL));
                repaint();
            }
        }
 
}

Le println me renvoie "0,1"
Le JLabel m'affiche "0.1"...

Merci de votre aide

**Lady** · 21/02/2012, 15h14

le paint de JLabel n'utiliserait t'il pas le getText ??? Si tu a les .java du jdk c'est à aller vérifier.

**oneagaindoguys** · 21/02/2012, 15h30

Ca c'est fort possible. Mais si tu n'as pas les sources .java comme tu redéfinies la méthode getText(), tu peux mettre une trace ou un point d'arrêt en déboggant

**Sharcoux** · 21/02/2012, 15h34

Ben... Probablement, puisqu'en mettant mon override de getText en commentaire, ça marche...

Est-ce que je suis le seul à trouver aberrant le fait que paint utilise la méthode getText qui est publique, non final, et surtout, getteur ! C'est à dire qu'il est justement susceptible d'être modifié pour s'adapter aux besoins d'entrée/sortie de l'utilisateur !

Enfin bon... Bref. Quels sont mes moyens de contrer ça? Laisser tomber la surcharge de getText et remplacer tous les getText de mon code par un getTextFormaté(), un truc du genre? Une autre idée?

**oneagaindoguys** · 21/02/2012, 16h03

Perso, je ne vois pas ce qui te dérange :

Dans getText(), tu transformes les ',' en '.'
Dans setText(), tu fais les l'inverse : tu transformes les '.' en ','

JLabel étant un objet, sans creuser plus, à mon avis il fonctionne comme tout objet respectant la norme de l'encapsulation. A savoir, a un attribut de type String, et lorsques tu fais un setText(), il écrit dans cette variable et getText() lit la variable. Donc si tu transformes les points en virgule, il n'y a pas de raison que de faire le chemin inverse dans la méthode getText().

**Lady** · 21/02/2012, 16h09

Sinon tu peut pas transformer le String en float ? tu aura alors le . ou la , en fonction de paramètre local de ton appli ...

**Sharcoux** · 21/02/2012, 16h09

Dans getText(), tu transformes les ',' en '.'
Dans setText(), tu fais les l'inverse : tu transformes les '.' en ','

Je ne suis pas d'accord :

Comme précisé précédemment, paint utilise getText. Au final, si j'essaye d'écrire "0,1", avec ta méthode, il va faire :

setText("0.1")
- transforme "0.1" en "0,1"
- appelle paint() -> getText() -> transforme "0,1" en "0.1" -> affiche "0.1" dans le JLabel

C'est ce qu'il se passe dans mon exemple (Cf premier post)

**oneagaindoguys** · 21/02/2012, 16h16

Envoyé par Sharcoux

Je ne suis pas d'accord :

Comme précisé précédemment, paint utilise getText. Au final, si j'essaye d'écrire "0,1", avec ta méthode, il va faire :

setText("0.1")
- transforme "0.1" en "0,1"
- appelle paint() -> getText() -> transforme "0,1" en "0.1" -> affiche "0.1" dans le JLabel

C'est ce qu'il se passe dans mon exemple (Cf premier post)

Donc setText() appelle paint ?
Pour quelle raison getText() transforme les ',' en '.' ?

**Sharcoux** · 21/02/2012, 16h26

Donc setText() appelle paint ?

Pas directement, mais de toute façon, on ne voit pas le résultat de setText avant l'appel de paint().

Pour quelle raison getText() transforme les ',' en '.' ?

Pour garder la compatibilité avec les commandes du type

Code :

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

Double.valueOf(ecran.getText()).doubleValue()

qui jonchent le code

**oneagaindoguys** · 21/02/2012, 16h33

Alors, tu devrais faire une autre méthode getContain par exemple :

Code :

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

Double.valueOf(ecran.getTContain()).doubleValue()

Ne pas redéfinir getText()

Code :

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

1
2
3
4
@Override
        public String getContain() {
            return super.getText().replace(MARQUEUR_DECIMAL, ".");
        }

**Sharcoux** · 21/02/2012, 16h35

Sinon tu peut pas transformer le String en float ? tu aura alors le . ou la , en fonction de paramètre local de ton appli ...

J'ai essayé de mettre ça en place, mais le marqueur utilisé est toujours le "."
Comment puis-je changer cela?

(très rapidement)

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
        public static class EcranDAffichage extends JLabel {
            public EcranDAffichage(String label) {
                super(label);
                setHorizontalAlignment(JLabel.RIGHT);
                setPreferredSize(new Dimension(220, 40));
            }
 
        @Override
            public void setText(String texte) {
                System.out.println(texte);
                super.setText(texte.equals("") ? "" : ""+Float.valueOf(texte).floatValue());
                repaint();
            }
        }

**Sharcoux** · 21/02/2012, 16h53

Alors, tu devrais faire une autre méthode getContain par exemple :

Oui, c'est ce que j'avais pensé aussi :

Quels sont mes moyens de contrer ça? Laisser tomber la surcharge de getText et remplacer tous les getText de mon code par un getTextFormaté(), un truc du genre?

Du coup, je me suis rabattu sur cette méthode.

Merci

**thelvin** · 21/02/2012, 19h35

Enfin bon, quelque part, il suffit d'ajouter un nouveau set() à côté du nouveau get().
'Faut être logique, les méthodes existantes sont là pour qu'on s'en serve, pas pour qu'elles fassent des trucs qui n'ont rien à voir avec leur fonction d'origine.

Idéalement je me créerais un nouveau composant FilteredJLabel qui masque le JLabel originel au lieu d'en hériter, mais c'est plus facile à dire qu'à faire. Les deux nouvelles méthodes, c'est plus réaliste.

override de setText (JLabel)

avec Java

Discussions similaires

Partager

Partager