Discussion :

[laurentapologic]

Bonjour

J'utilise toujours les mêmes icones dans une application. Pour les charger j'utilise cette méthode :

Code :

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

jMenuItem1.setIcon(new javax.swing.ImageIcon(getClass().getResource("/com/test/rcp/icon/exit.png")));

Je pense que du coup pour une image identique chargée x fois, elle est montée en mémoire à chaque fois.

Que pensez-vous de l'idée de faire un depository où je chargerais toutes mes icones en "public final static". Là je pense que même si on les applique à x boutons, l'image n'est montée qu'une seule fois en mémoire ?

Je ne connais pas le comportement des composants Swing sur ce point.

Merci de vos réponses.
Laurent

[natha]

Personnellement j'utilise une classe Factory utilisant un système de cache.
Ce qui m'évite de charger en static toutes les icônes (j'en ai une pété, ça ralentirait mon programme), et m'évite de charger 2x la même si je l'ai déjà en cache.

Aucun problème pour utiliser la même instance d'icône à 20 endroits différents dans l'application, pas de soucis sur ce point (tant que tu ne la modifies pas bien sûr).

[laurentapologic]

J'ai utilisé ton principe Factory + Depository. J'ai diminuer d'environ 20mo la taille de l'application.
Cool
Merci

[jojodu31]

@natha et/ou @laurentapologic:
est-ce que l'on pourrait en savoir un peu plus sur ta Factory (éventuellement un bout de code) ça m'intéresse bien

merci d'avance

[laurentapologic]

Ca manque surement de commentaire mais c'est pas très compliqué

Pour la factory (un singleton):

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
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
 
package com.test.rcp.util;
 
import java.lang.reflect.Field;
import java.util.ArrayList;
import java.util.Hashtable;
import java.util.List;
import javax.swing.ImageIcon;
 
/**
 *
 * @author laurent
 */
public final class IconFactory
{
    //instance du factory (qui est en fait un singleton)
    private static IconFactory instanceIconFactory;
 
    //map des icones
    private Hashtable<String,ImageIcon> iconCache;
    private List<String> iconLst;
 
    // objet utilisé uniquement pour la synchronisation du singleton
    private static Object objetSynchrone = new Object();
 
 
    /** Creates a new instance of IconFactory */
    private IconFactory()
    {
        this.iconCache = new Hashtable<String,ImageIcon>();
        this.iconLst = new ArrayList<String>();
 
        Field[] champs = IconDepository.class.getFields();
        IconDepository iconDepository = new IconDepository();
 
        if(champs!=null)
        {
            for(Field field : champs)
            {
                if(field.getName().startsWith("ICON_"))
                {
                    try
                    {
                        this.iconLst.add((String)field.get(iconDepository));
                    } catch (IllegalArgumentException ex)
                    {
                        ex.printStackTrace();
                    } catch (IllegalAccessException ex)
                    {
                        ex.printStackTrace();
                    }
                }
            }
        }
    }
 
    public static IconFactory getInstance() {
        if (instanceIconFactory == null) 
        { 
            // on s'assure de synchroniser la creation du singleton
            synchronized(objetSynchrone) {
                    instanceIconFactory = new IconFactory();
            }
        }
        return instanceIconFactory;
    }
 
    public ImageIcon getIcon(String cle)
    {
        ImageIcon img = null;
        if(isIconInDepository(cle))
        {
            if(iconCache.contains(cle))
            {
                img = iconCache.get(cle);
            }
            else
            {
                img = new javax.swing.ImageIcon(getClass().getResource(cle));
                iconCache.put(cle,img);
            }
        }
        return img;
    }
 
    private boolean isIconInDepository(String cle)
    {
        boolean retour = false;
 
        if(this.iconLst.contains(cle))
        {
            retour=true;
        }    
        return retour;
    }
}

Pour le Depository:

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
package com.test.rcp.util;
 
 
/**
 *
 * @author laurent
 */
public class IconDepository
{           
    /** 
     * Facturation : état de la facture calculée
     * /com/test/rcp/icon/calculator.png
     */
    public static final String ICON_SMALL_CALCULATOR = "/com/test/rcp/icon/calculator.png";
    /** 
     * Bouton valider dans les fiches de gestion, les listes de sélection, ...
     * /com/test/rcp/icon/check2.png
     */
    public static final String ICON_SMALL_CHECK2 = "/com/test/rcp/icon/check2.png";
}

Pour appeler une icone:

Code :

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

com.test.rcp.util.IconFactory.getInstance().getIcon(com.test.rcp.util.IconDepository.ICON_CONTACT_MAIL)

[jojodu31]

ok merci je garde ça dans un coin on en apprend tous les jours

[bouye]

Et plutot que definir les chemins des icones dans le Repository sous forme de constantes autant utiliser un fichier .properties et un ResourceBundle (ce qui permet bien sur de faire varier l'icone, nottament celles ayant du texte sur le dessin, en fonction de la Locale).

[natha]

+1 pour bouye
Je travaille également avec un properties.