Discussion :

[toinekikil]

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 :

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

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

[meddle]

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

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
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();
            }
         }));
   }
});

[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??

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

[Jense]

Le mieux est d'utiliser la classe org.eclipse.jface.resource.JFaceResources