Probleme SWT et fontdialog

Version imprimable

Bonjour,

Voici mon problème, je suis en train de travailler sur un changement de font dans une application en SWT. Voici mon code :

Code:

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
 
myLayout = new RowLayout(SWT.VERTICAL);
myLayout.wrap = true;
parent.setLayout(myLayout);
 
label = new Label(parent, SWT.NONE);
label.setText("Je suis un label");
 
Button bouton = new Button(parent, SWT.PUSH);
bouton.setText("Je suis le bouton");
bouton.addListener(SWT.MouseDown, new Listener() {
   public void handleEvent(Event event) {
      FontDialog ftd = new FontDialog(parent.getShell());
      ftd.setText("choisir une police");
 
      ftd.setFontList(label.getFont().getFontData());
      ftd.setRGB(label.getForeground().getRGB());
 
      ftd.open();
 
      //On récupère la couleur et on créé l'objet pour l'attribuer au label
      RGB colorRGB = ftd.getRGB();
      Color colorText = new Color(parent.getShell().getDisplay(), colorRGB);
      label.setForeground(colorText);
 
      //On récupère la font
      FontData [] fdata = ftd.getFontList();
      Font fontText = new Font(parent.getShell().getDisplay(), fdata);
      label.setFont(fontText);				
 
      colorText.dispose();
      //fontText.dispose();  //-> LA PROBLEME
   }
});

Mon problème est que lorsque je "dispose" ma variable fontText et que je clique à nouveau sur mon bouton, j'ai perdu tout sauf la couleur. Mais si je ne "dispose" pas tout marche bien.

Savez vous ou est le problème??

Merci d'avance de votre aide.

03/04/2008, 23h21
soft0613

oui tu ne dois pas libérer les ressources tant que l'objet est utilisé, tu ne dois appeller la méthode dispose() que lorsque la police n'est plus nécessaire.

Oui il faut faire le dispose quand tu n'as plus besoin de ta couleur ou de ta fonte, comme ici :

Code:

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
36
37
38
39
40
 
myLayout = new RowLayout(SWT.VERTICAL);
myLayout.wrap = true;
parent.setLayout(myLayout);
 
label = new Label(parent, SWT.NONE);
label.setText("Je suis un label");
 
final Button bouton = new Button(parent, SWT.PUSH);
bouton.setText("Je suis le bouton");
bouton.addListener(SWT.MouseDown, new Listener() {
   public void handleEvent(Event event) {
      FontDialog ftd = new FontDialog(parent.getShell());
      ftd.setText("choisir une police");
 
      ftd.setFontList(label.getFont().getFontData());
      ftd.setRGB(label.getForeground().getRGB());
 
      ftd.open();
 
      //On récupère la couleur et on créé l'objet pour l'attribuer au label
      RGB colorRGB = ftd.getRGB();
      final Color colorText = new Color(parent.getShell().getDisplay(), colorRGB);
      label.setForeground(colorText);
 
      //On récupère la font
      FontData [] fdata = ftd.getFontList();
      final Font fontText = new Font(parent.getShell().getDisplay(), fdata);
      label.setFont(fontText);                
 
      // Nettoyage
      bouton.addDisposeListener(new DisposeListener() {
 
            public void widgetDisposed(DisposeEvent arg0) {
                colorText.dispose();
                fontText.dispose();
            }
         }));
   }
});

08/04/2008, 09h16
toinekikil

Ok merci beaucoup. Mais je ne comprends toujours pas pourquoi lors de l'attribution il n'y a pas de recopie afin que l'on puisse libérer les ressources aisément??
08/04/2008, 20h26
soft0613

je me suis posé la même question, à mon avis c'est pour éviter de créer trop d'objets, tu peux créer une seule instance of Font et l'utiliser pour tous les controles de ton programme plutôt que d'avoir une instance of Font pour chaque élément.
10/04/2008, 14h36
Jense

Le mieux est d'utiliser la classe org.eclipse.jface.resource.JFaceResources
10/04/2008, 16h37
soft0613

Oui sauf si on n'utilise que la librairie SWT...