[java] JComboBox object toevoegen

[Tjeerd]

Ik heb een klasse genaamd Tumor, met daarin de methode toString()-geplaatst die de naam van de tumor teruggeeft. En verder nog wat andere methoden die info over de tumor teruggeven.

Ik had begrepen dat je gewoon een object in een JComboBox kunt toevoegen m.b.v. addItem(...). Nu probeer ik dat te doen, alleen worden mijn tumor-objecten geweigerd / niet toegevoegt.

Ook had ik begrepen dat je object verplicht een toString-methode moet hebben voordat je m als object kunt toevoegen aan de combobox. Misschien dat ik toch wat over het hoofd heb gezien, want ik kom er zo 123 niet uit waarom ik mijn objecten niet kan toevoegen.

[drm]

Nu probeer ik dat te doen, alleen worden mijn tumor-objecten geweigerd / niet toegevoegt.

Krijg je foutmeldingen? Zo ja, wat zijn die dan? Zo nee, laat even wat code zien waarmee je de combobox instantieert, de Tumor objecten instantieert en toevoegt aan de combobox.

Overigens is het voor deze zaken het handigst om een ComboboxModel te implementeren, waarmee je ook zorg draagt voor het firen van de juiste events zodra er iets wijzigt in de inhoud. Zorg daarmee dat je de listeners die (onder water) aan de het model toegevoegd worden allemaal op de hoogte gesteld worden zodra er iets wijzigt in de inhoud.

offtopic:
luguber, trouwens

[koli-man]

Voor een voorbeeldje:
http://java.sun.com/j2se/...addItem(java.lang.Object)
Dit is ongeveer wat je wil hebben lees ik zo.

[Tjeerd]

Ok, voor de volledigheid zal ik even alles neerdumpen wat relevant is:

Java:

public class Tumor
{
private String tumorname;
private Color tumorcolor;

    public Tumor(String theName)
    {  
        tumorname = theName;
    }
    
    public String toString()
    {
        return tumorname;
    }   
}

Vervolgens ga ik via een ListIterator door een ArrayList heen:

Java:

ListIterator t = heatmap.tumortypes.listIterator();
Tumor temp;

while ( t.hasNext() )
                 {
        temp = (Tumor)t.next();
        tumorlist.addItem( temp );
         }

Foutmelding:

Exception in thread "main" java.lang.ClassCastException
at HeatmapPanel.actionPerformed(TestSwing.java:358)
at javax.swing.JComboBox.fireActionEvent(Unknown Source)
at javax.swing.JComboBox.contentsChanged(Unknown Source)
at javax.swing.AbstractListModel.fireContentsChanged(Unknown Source)
at javax.swing.DefaultComboBoxModel.setSelectedItem(Unknown Source)
at javax.swing.DefaultComboBoxModel.addElement(Unknown Source)
at javax.swing.JComboBox.addItem(Unknown Source)
at HeatmapPanel.buildTumor_List(TestSwing.java:169)
at HeatmapPanel.Setup_Heatmap(TestSwing.java:153)
at HeatmapFrame.<init>(TestSwing.java:56)
at TestSwing.main(TestSwing.java:16)

Geweigerd in de zin van, hij wil het object van type Tumor niet toevoegen
Ik doe iets simpels fout, maar ik kom er gewoon zelf niet uit waarom object niet kan worden toegevoegd.

Koli-man: dat stukje heb ik ook al gade geslagen, maar ik zie even niet hoe ik die in mijn Tumor-klasse versie moet verwerken.

[ Voor 10% gewijzigd door Tjeerd op 10-05-2004 15:49 ]

[bigbeng]

Is je combobox wel Mutable?

Kijk anders even hier voor wat pointers.

[drm]

Die actionPerformed in de HeatmapPanel, waar luistert die naar? En hoe ziet die eruit?

Nogmaals: een ComboBoxModel implementeren is beter / makkelijker onderhoudbaar voor dit soort dingen.

[bigbeng]

drm schreef op 10 mei 2004 @ 15:53:
Die actionPerformed in de HeatmapPanel, waar luistert die naar? En hoe ziet die eruit?

Nogmaals: een ComboBoxModel implementeren is beter / makkelijker onderhoudbaar voor dit soort dingen.

Zoals ik het zie triggert de additem die Exception, dus volgens mij heeft Nestor vergeten om zijn combobox datamodel op MutableComboBoxModel in te stellen.

[Tjeerd]

Ik had gekeken naar deze oplossing die Alarmnummer 2 jaar geleden ergens al had gegeven, maar dan kan ik nog steeds niet bij de waarden binnen de klasse komen. Logisch agz hij alleen maar strings toevoegt aan de combobox.

drm: wat betreft ActionPerformed:

Java:

public void actionPerformed(ActionEvent e) 
    {
         JComboBox cb = (JComboBox)e.getSource();
         String tumor = (String)cb.getSelectedItem();
         System.out.println( tumor );
         }

[ Voor 15% gewijzigd door Tjeerd op 10-05-2004 16:15 ]

[bigbeng]

Nestor schreef op 10 mei 2004 @ 16:13:
Ik had gekeken naar deze oplossing die Alarmnummer 2 jaar geleden ergens al had gegeven, maar dan kan ik nog steeds niet bij de waarden binnen de klasse komen. Logisch agz hij alleen maar strings toevoegt aan de combobox.

En dat is dus niet waar. Als je objecten toevoegt aan een combobox dan blijven die bestaan. Voor het renderen gebruikt het default combobox model de toString methode van je object.
Als je dus een getitem of een getselecteditem doet, dan krijg je gewoon weer het object terug zoals je het erin gestopt hebt.

Vraag van de source van je event eens de classname op:
Dus e.getSource().getClass().getName();

Misschien dat die class cast exception daarvandaan komt.

[Tjeerd]

bigbeng schreef op 10 mei 2004 @ 16:27:
[...]

En dat is dus niet waar. Als je objecten toevoegt aan een combobox dan blijven die bestaan. Voor het renderen gebruikt het default combobox model de toString methode van je object.
Als je dus een getitem of een getselecteditem doet, dan krijg je gewoon weer het object terug zoals je het erin gestopt hebt.

Vraag van de source van je event eens de classname op:
Dus e.getSource().getClass().getName();

Misschien dat die class cast exception daarvandaan komt.

Heb ik het dus verkeerd begrepen, dacht dat alleen de toString-methode werd gepakt en dat ie puur die string in de combobox zette -- daar zag ik trouwens al het nut niet van in, dus wat je zegt is wel zo logisch.

Ga zo ff proberen, kom hier nog op terug.

[drm]

Allereerst: Als het een Tumor is wat je toevoegt, is het object dat geselecteerd wordt ook de Tumor zelf. De string die in de combobox gebruikt wordt om het object te representeren wordt gewoon genomen door de toString() method aan te roepen. Het object dat je dus terugkrijgt als je de getSelectedItem () aanroept van de combobox, is (een referentie aan) hetzelfde object als dat je met addItem toegevoegd hebt. Casten naar een String is dan een beetje zinloos, lijkt mij

Verder kijk je wat de source is van het ActionEvent, maar dit kan volgens mij helemaal niet de combobox zijn, want je moet een ItemListener aan een combobox hangen, niet een actionlistener. De getSource() van het ActionEvent object dat meegegeven wordt aan de actionPerformed is (meestal) een button waarop geklikt wordt, en waar je een ActionListener aan gehangen hebt. Ken dah?

Dus: waarschijnlijk komt die ClassCastException door de cast op regel 3 (naar een combobox).

edit:
dit is bij nader inzien niet waar

edit:
laat

[ Voor 5% gewijzigd door drm op 10-05-2004 16:37 ]

[Martin Sturm]

In het kader van topic-kaping heb ik ook nog ff een vraagje:
Ik wil dus in een combobox een id en een string opslaan, waarbij ik op basis van het id een bepaalde waarde wil selecteren. Daarnaast wil ik dan weer op basis van de geselecteerde string de bijbehorende id ophalen. Ik heb het nu wel werkend, maar imho niet echt super netjes.
Het ophalen van een id voor de geselecteerde string is simpel: ik vul de jcombobox met objecten met daarin een naam en id en een toString() method die de string teruggeeft en een getId() method die de id teruggeeft.
Ik selecteer nu bij initialisatie van het formulier de gewenste waarde door een linear search toe te passen op de arraylist die ik in de combobox stop en de index van de juiste waarde als parameter mee te geven aan setSelectedIndex(). Dit werkt, maar kan dit nou niet mooier?

[drm]

Ik snap niet helemaal wat je bedoelt, maar is setSelectedItem niet gewoon wat je bedoelt? Sowieso lijkt het mij het handigst om de List (waarin de items zitten) te wrappen in een ComboBoxModel, eventueel door gewoon een anonymous afgeleide van DefaultComboBox te maken.

[Martin Sturm]

drm schreef op 10 mei 2004 @ 20:16:
Ik snap niet helemaal wat je bedoelt, maar is setSelectedItem niet gewoon wat je bedoelt? Sowieso lijkt het mij het handigst om de List (waarin de items zitten) te wrappen in een ComboBoxModel, eventueel door gewoon een anonymous afgeleide van DefaultComboBox te maken.

Nee, setSelectedItem is volgens mij niet wat ik bedoel. Het is als volgt: ik heb dus een combobox waarin een (mogelijk) groot aantal Strings staan met aan elke string een bepaald ID gekoppeld (dit id is niet ascending en niet opeenvolgend, het kan dus zijn dat in de combobox 1 - bla, 5 - blabla en 6 - bliep) staat, waarbij dus het getal het 'id'). Bij initialisatie van het venster moet er een bepaald veld in die combobox worden geselecteerd op basis van de gegeven id. Bij afsluiting van het venster moet de huidig geselecteerde id worden opgehaald.

[Tjeerd]

Even een update, blijkbaar is er niets aan de hand, want door alleen de toString()-methode toe te voegen aan de Tumor-klasse dacht ik dat ie de objecten zelf verder niet zou toevoegen. Maar, wat blijkt, als ik nu toch uit de JComboBox een selectie maak en m.b.v. de actionlistener kijk wat er geselecteerd is, kan ik m.b.v. de volgende code toch netjes de kleur opvragen:

Java:

JComboBox cb = (JComboBox)e.getSource();
Tumor tumor = (Tumor)cb.getSelectedItem();
System.out.println( "Kleur: " + tumor.getColor() );

Zou bijna zweren dat ik dit ook eerder had geprobeerd, maar toen een foutmelding kreeg. Iig het is opgelost, nou geef ik verder wel toestemming om dit topic te kapen

[ Voor 6% gewijzigd door Tjeerd op 11-05-2004 10:12 ]

[Gert]

setSelectedItem is wel wat je bedoelt maar waarschijnlijk moet je de .equals() method overriden om daarin op id te vergelijken.

[Robtimus]

Nestor schreef op 10 mei 2004 @ 16:13:

Java:

JComboBox cb = (JComboBox)e.getSource();
String tumor = (String)cb.getSelectedItem();
System.out.println( tumor );

Nestor schreef op 11 mei 2004 @ 10:11:

Java:

JComboBox cb = (JComboBox)e.getSource();
Tumor tumor = (Tumor)cb.getSelectedItem();
System.out.println( "Kleur: " + tumor.getColor() );

Zie je het verschil?
Precies wat drm al zei.

[drm]

Martin Sturm:
Nee, setSelectedItem is volgens mij niet wat ik bedoel. Het is als volgt: ik heb dus een combobox waarin een (mogelijk) groot aantal Strings staan met aan elke string een bepaald ID gekoppeld (dit id is niet ascending en niet opeenvolgend, het kan dus zijn dat in de combobox 1 - bla, 5 - blabla en 6 - bliep) staat, waarbij dus het getal het 'id'). Bij initialisatie van het venster moet er een bepaald veld in die combobox worden geselecteerd op basis van de gegeven id. Bij afsluiting van het venster moet de huidig geselecteerde id worden opgehaald.

Ah, ok, dus het is alleen bij initialisatie van belang dat er op id gezocht kan worden? Zo ja, dan zou ik het ook gewoon met een search doen, zo nee, dan zou ik ze (in eerste instantie al, voordat je met de combobox aan de slag gaat) in een Map stoppen (dat is met id/value paren sowieso niet onhandig om te doen)

[ Voor 4% gewijzigd door drm op 11-05-2004 11:03 ]

[Tjeerd]

IceManX schreef op 11 mei 2004 @ 10:40:
[...]

[...]
Zie je het verschil?
Precies wat drm al zei.

Hij heeft ook gelijk, nogmaals bedankt voor de snelle hulp

[ Voor 10% gewijzigd door Tjeerd op 11-05-2004 10:56 ]