JComboBox Renderer Problem

[Colt45]

Moin, ich versuch nun schon seit ner Weile, in einer Combobox Icons (und später noch zusätzlich text) anzeigen zu lassen, aber bis jetzt klappts nich so wirklich. Ich bekomme jedesmal eine ClassCastException "String cannot be cast to Integer"

package unit;

import java.awt.Component;
import java.util.Vector;

import javax.swing.Icon;
import javax.swing.ImageIcon;
import javax.swing.JLabel;
import javax.swing.JList;
import javax.swing.ListCellRenderer;

public class IconCellRenderer extends JLabel implements ListCellRenderer 
{

	private Vector<Icon> icons;

	public IconCellRenderer()
	{
		this.setOpaque(true);
		this.setHorizontalAlignment(CENTER);
        this.setVerticalAlignment(CENTER);
	}
	
	public IconCellRenderer(Vector<Icon> icons) 
	{
		this.icons = icons;	
		this.setOpaque(true);
		this.setHorizontalAlignment(CENTER);
		this.setVerticalAlignment(CENTER);
	}
	
	@Override
	public Component getListCellRendererComponent(
			JList list, Object value, int index,
			boolean isSelected, boolean cellHasFocus) 
	{
        int selectedIndex = ((Integer)value).intValue();
        ImageIcon icon = (ImageIcon) icons.elementAt(selectedIndex);
        this.setIcon(icon);
        
        return this;
	}
}

und hier setze ich den Renderer:

Vector<Icon> icons = new Vector<Icon>();
icons.add(new ImageIcon("icons\\arzttrupp.jpg"));
icons.add(new ImageIcon("icons\\aeskulap.jpg"));
combobox.setRenderer(new IconCellRenderer(icons));

Ich verstehe ja, dass value eben ein String ist und kein Integer, aber wenn ich icons.elementAt(index); aufrufe, kommt eine ArrayIndexOutOfBoundsException "-1".

Ich kapier auch nocht so ganz, wie das überhaupt mit so einem Renderer funktioniert, wann die getListCellRendererComponent aufgerufen wird etc.

 

[hdi]

Hey,

value enthält den internen Datensatz der zu dem jeweiligen Eintrag in der Combobox gehört. Was das genau für ein Wert und Datentyp ist bestimmt dein Model. Was für ein Model verwendest du für die Combobox? D.h. was sind die internen Daten dafür? Wenn ich dich richtig verstehe sind das einfach Ziffern als String?

Das würde keinen Sinn machen. Den index hast du ja direkt in der Methode, unter dem Namen "index". Wenn der den Wert "-1" hat dann heißt das einfach nur, dass nix ausgewählt wurde in der Box. In dem Fall kannst du das Label "leeren" (setText und setIcon auf null setzen) und returnen. Nur wenn index != -1 ist, ist etwas aus der Box ausgewählt.

if(index == -1){
   setText(null);
   setIcon(null);
}
else{
  setText(value == null ? null : value.toString());
  setIcon(icons.elementAt(index));
}
return this;

 

[Colt45]

So gesehen gibt es kein Model, die Daten sind lediglich ImageIcons, die in einen Vector gepackt werden, der dem Renderer übergeben wird, der wiederum der combobox übergeben wird.
Da sind eigentlich keine Strings oder Ziffern meines Erachtens. Interessanterweise, wenn ich mir "value" ausgeben lasse, erscheinen in der Konsole ein paar Leerzeichen untereinander. "value" sollte doch aber eigentlich ein Icon aus dem Vector sein, oder irre ich mich da?

 

[hdi]

Der Vector der ImageIcons sollte dem Model der ComboBox übergeben werden, nicht dem Renderer. Die Combobox muss ja erstmal Daten enthalten, um diese anzeigen zu können. Ich nehme an du erzeugst ein leeres DefaultComboBoxModel. Demnach hast du auch keine Daten da drin. Was du tun musst:

JComboBox<Icon> myIconBox = new JComboBox<Icon>(icons);

Dem Renderer musst du nix mehr übergeben, und der muss sich die Icons auch nicht separat abspeichern. Du brauchst nur deine vom Interface definierte Methode implementieren, und einfach:

myIconBox.setRenderer(new IconCellRenderer());

"value" enthält dann jeweils die Referenz auf eines deiner ImageIcons. Du kannst in deiner Renderer-Methode also einfach sagen setIcon(value), und setText(value.toString()), oder was auch immer du als Text anzeigen möchtest.

 

[Colt45]

Es funktioniert immer noch nicht, ich habe zwar keine Exceptions mehr, aber die Combobox bleibt leer...
und wenn ich mir value auf die Konsole ausgeben lassen will, erscheint nichts

JComboBox<Icon> cb = new JComboBox<Icon>(icons);
cb.setRenderer(new IconCellRenderer());

package unit;

import java.awt.Component;
import java.util.Vector;

import javax.swing.Icon;
import javax.swing.ImageIcon;
import javax.swing.JLabel;
import javax.swing.JList;
import javax.swing.ListCellRenderer;

public class IconCellRenderer extends JLabel implements ListCellRenderer 
{

	private Vector<Icon> icons;

	public IconCellRenderer()
	{
		this.setOpaque(true);
	}
	

	@Override
	public Component getListCellRendererComponent(
			JList list, Object value, int index,
			boolean isSelected, boolean cellHasFocus) 
	{
		System.out.println(value);
		
		this.setIcon((Icon) value);
		this.setText("test");
		
		return this;
	}
}

 

[hdi]

Die Instanz-Variable für den IconVector kannst du aus der Renderer-Klasse schmeißen, die brauchst du ja eh nicht mehr. Zudem könntest du das generische Verhalten von ListCellRenderer nutzen, dann kannst du dir auch das Casten von value sparen:

public class IconCellRenderer extends JLabel implements ListCellRenderer<Icon>

In dem Fall musst du auch die Signatur der Methode anpassen: Satt

Object value

muss das dann

Icon value

heißen.

Aber: Das sind nur Aufhübschungen. An sich müsste es jetzt schon funktionieren. Du musst irgendwo einen Fehler machen. Entweder das "icons", was du dem Konstruktor der Combobox übergibst, enthält gar nicht das, was du denkt (Lass dir mal unmittelbar bevor du das der ComboBox übergibst ausgeben was da genau drin ist), oder du überschreibst das Model der ComboBox nach der Erzeugung noch irgendwie (setModel()??), oder du zeigst in deiner GUI gar nicht diese Combobox an sondern ne andere.. Respektive die Sache mit dem Model überschreiben gilt ja auch für Box selbst: Vllt hast du ja irgendwo anders ne weitere Erzeugung der Box..

An (mindestens) einem dieser Dinge muss es liegen. Denn wenn es so aussehen würde wie du es hier schreibst, dann würde es funktionieren.

 

[Colt45]

Danke, ich werds umsetzen...
Du hattest Recht, da ich in meinem Frame das GroupLayout verwende, darf keines der Komponenten null sein, somit wurde die Box hier schon instanziiert. in meiner Controller-Klasse hab ich dann erst die Box erstellt mit dem Vector und diese mittels setIconComboBox(JComboBox box) an das Frame übergeben. ich hab nun den Vector direkt im Frame erstellt, nun gehts.
Danke dir...

 

[hdi]

Gerne. Bitte markier das Thema als erledigt (Knopf oben).