Discussion :

[coco38]

Bonjour,
J'ai chercher sur le net mais aucune information n'a pu m'aider.
Alors j'ai une JCombobox avec des noms de joueurs et chaque joueur à une couleur.
Je voudrais donc afficher le nom de mon joueur avec sa couleur dans un carré à côté dans la combobox.
J'arrive à afficher mes noms de joueurs cela fonctionne mais lorsque je veux mettre la couleur cela ne marche pas comme je le veux (j'obtiens une liste avec uniquement la dernière couleur et le dernier nom insérer).
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
 
jCPlayer.setRenderer(new ListCellRenderer() {
							public Component getListCellRendererComponent(JList list, Object value, int index, boolean isSelected, boolean cellHasFocus) {
								System.out.println(value.toString());
								int w = 20;
								int h = 20;
								BufferedImage img = new BufferedImage(w, h, BufferedImage.TYPE_INT_ARGB);
								Graphics gr = img.getGraphics();
								gr.setColor(g.getPlayerList().get(z).getColor());
								gr.fillRect(0, 0, w, h);
								ImageIcon icon = new ImageIcon(img);
 
								JLabel label = new JLabel( g.getPlayerList().get(z).getName(),icon, JLabel.LEADING);
 
								return label;
								}
						});
						jCPlayer.addItem(g.getPlayerList().get(z).getName());

Merci d'avance

[bouye]

Tout d'abords le principe du renderer c'est d'eviter de recreer un nouveau composant a chaque fois mais plutot de reutiliser le meme. Ensuite DefaultListCellRenderer (un derive de JLabel) est une bonne base de depart.

Il semble de plus normal que ce soit tout le temps le meme joueur qui soit affiche puis que ton z est fixe. Ensuite tu peux directement mettre tes joueurs dans la combo plutot que juste leur noms.

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
 
for (Player player : g.getPlayerList()) {
  jCPlayer.addItem(player);
}
jCPlayer.setRenderer(
   new DefaultListCellRenderer() {
     public Component getListCellRendererComponent(JList list, Object value, int index, boolean isSelected, boolean cellHasFocus) {
       super.getListCellRendererComponent(list, value, index, isSelected, cellHasFocus);
       System.out.println(value.toString());
       ImageIcon icon = null;
       if (value instanceof Player) {
         int w = 20;
         int h = 20;
         BufferedImage img = new BufferedImage(w, h, BufferedImage.TYPE_INT_ARGB);
         Graphics gr = img.getGraphics();
         Player player = (Player)value;
         gr.setColor(player.getColor());
         gr.fillRect(0, 0, w, h);
         ImageIcon icon = new ImageIcon(img);
         setText(player.getName());
      }
      setIcon(icon);
      return this;
   }
});

De la meme maniere, ca peut etre une bonne idee que de cacher les icones qui leur son assocees dans une WeakHashMap<Player, ImageIcon> (par exemple) plutot que de les re-creer a chaque reaffichage.

PS : si ta couleur n'est jamais transparente, utilise plutot BufferedImage.TYPE_INT_RGB a la place.

[coco38]

Merci pour cette réponse.
J'ai testé mais en fait j'ajoute le premier joueur tout vas bien cela s'affiche comme je le souhaite.
Ensuite j'ajoute le deuxième joueur et là j'obtiens dans ma liste le prmier joueur deux fois plus le deuxième joueur.
Si j'ajoute un troisième joueur j'obtiens trois foi le premier joueur pui deux fois le deuxième joueur pui el torisième joueur.
Comment faire???

[bouye]

Code ? Code ? Code ? Code ? Code ? Code ? Code ? Code ? Code ?

[coco38]

Bah le code....j'ai pris celui que tu m'as donné!!!

[bouye]

Si tu n'es pas capable de nous fournir un exemple minimal mais suffisant (et qui soit capable de tourner par lui-meme) qui demontre le probleme ne t'attends pas a avoir rapidement une solution !