[java] naamloos JTextField uitlezen?

[Wbdsgnr]

Als stageopdracht moet ik een programma afmaken van mijn voorganger, deze maakt aan de hand van enkele hashtables een layout aan in swing. Erg makkelijk met het opbouwen van een layout, maar uitlezen is een ander verhaal. Is het uberhaupt mogelijk om bijvoorbeeld een tfield uit te lezen? Aangezien er een stuk of 10 JTextFields worden aangemaakt met de variable naam tfield, heb ik

tfield.setName(...);

toegevoegd, maar ik kom er verder niet uit... Ironisch genoeg heb ik nog niks kunnen vinden op google noch sun.com...

Ik dacht iets in de richting van

waarde = tfield.getName().equals("portname").getText();

maar niet al te gek werkt dit niet aangeziend equals een boolean terug geeft...

Java:

        while(groupkeys.hasMoreElements())
        {
            String groupname=groupkeys.nextElement();
            SettingsGroup group=groups.get(groupname);
            
            JPanel panel=new JPanel();

            panel.setLayout(null);
            
            int ypos=5;
            
            //get all settings contained in this group, create a label and textfield
            Hashtable<String,Setting> settings=group.getSettings();
            Enumeration<String> settingkeys=settings.keys();
            
            while(settingkeys.hasMoreElements())
            {
                String settingname=settingkeys.nextElement();
                Setting setting=settings.get(settingname);
                
                JLabel label=new JLabel(setting.getDescription());
                label.setBounds(5,ypos,200,20);
                
                panel.add(label);                
                
                
                //check if the contained value is a string/integer or boolean and match the way of editting
                Object val=setting.getValue();

                if (val instanceof Boolean)
                {
                    cfield=new JCheckBox();
                    cfield.setName(setting.getValue().toString());  // om te onderscheiden
                    cfield.setSelected((Boolean)val);
                    cfield.setBounds(205, ypos, 200, 20);
                    panel.add(cfield);
                }
                else
                {                
                    //JTextField 
                    tfield=new JTextField(setting.getValue().toString());
                    tfield.setBounds(205, ypos, 200, 20);
                    tfield.setName(setting.getValue().toString());  // om te onderscheiden
                    //field.addFocusListener(this);
                    panel.add(tfield);
                }
                
                ypos+=25;
            }

            tabpane.addTab(groupname,panel);        
        }

[Janoz]

Die tfield en cfield hebben alleen betekenis binnen die while. Het is niet een of andere wrapper of collection waarin vervolgens alle waarden opgeslagen worden?

Hoe kom je trouwens aan deze opdracht? De vraag die je hier stelt doet mij vermoeden dat je nog nauwelijks ervaring hebt met programmeren/java.

[Wbdsgnr]

Ik heb tfield en cfield wel public gedeclareerd in het begin van de class, ze ontstaan niet in de whileloop... Ik heb inderdaad niet heel veel ervaring met java, maar het reikt wel verder dan scopes ^^ Dit is trouwens alleen een gedeelte van de code, het hele programma is wel iets groter... Voorzover ik weet zijn het ordinaire textfields die ik probeer uit te lezen...

[Standeman]

Java:

waarde = tfield.getName().equals("portname").getText();

Wat je hier doet is een getText() method afroepen op een boolean. Aangezien primitives geen methods hebben en dus ook een boolean niet, gaat dit natuurlijk fout..

Wat wel kan is:

Java:

if (tfield.getName().equals("portname")) {
  waarde = tfield.getText();
}

nofi overgens, maar dit is een vrij eenvoudige (OO-)programmeer vaardigheid (zoals Janoz al opmerkte). Misschien moet je 's goed de basis java tutorial eens lezen, dat zal je een heel eind op weg helpen met het aanpassen van andersman code.

[ Voor 37% gewijzigd door Standeman op 06-10-2005 14:10 ]

[Wbdsgnr]

Wbdsgnr schreef op donderdag 06 oktober 2005 @ 13:18:...maar niet al te gek werkt dit niet aangeziend equals een boolean terug geeft...

Heb ik ook al geprobeerd, maar hij geeft altijd false terug, dus er moet een andere manier zijn...

[momania]

Wbdsgnr schreef op donderdag 06 oktober 2005 @ 13:57:
Ik heb tfield en cfield wel public gedeclareerd in het begin van de class, ze ontstaan niet in de whileloop...

Dat is iedere keer maar een referentie naar een JTextField die je in je while steeds vult met een nieuwe JTextField en dan toevoegd aan je panel.

De referentie die je overhoud in de variable 'tfield' is dan naar de laatst aangemaakte JTextField. De rest ben je gewoon kwijt.

[Standeman]

momania schreef op donderdag 06 oktober 2005 @ 14:10:
[...]

Dat is iedere keer maar een referentie naar een JTextField die je in je while steeds vult met een nieuwe JTextField en dan toevoegd aan je panel.

De referentie die je overhoud in de variable 'tfield' is dan naar de laatst aangemaakte JTextField. De rest ben je gewoon kwijt.

Niet helemaal. Deze zijn vast nog wel ergens op het panel te vinden waar ze op geplaatst worden.

Ik heb even snel in API gekeken en daar kan ik geen methods vinden om de children van een Container uit te vragen.. Nooit verwacht dat dat niet kon.

enfin, het is nog simpel op te lossen door ze in een Set / Hashtable te zetten en hierover te itereren wanneer je ze wilt uitlezen

edit:

(leuk... 3x dezelfde oplossing.. en de mijn mijn was al laatste )

[ Voor 30% gewijzigd door Standeman op 06-10-2005 14:16 ]

[momania]

Standeman schreef op donderdag 06 oktober 2005 @ 14:11:
[...]


Niet helemaal. Deze zijn vast nog wel ergens op het panel te vinden waar ze op geplaatst worden.

dat wel, maar in eerste instantie worden ze nergens bewaard


Je kan ze het best, zoals Janoz al aangaf, in een collection bewaren.. simpel voorbeeld:

Java:

Map fields = new HashMap();
while (iets) {
  JTextField field = new JTextField();
  field.setName(name);
  fields.put(name, field);
}

Dan kan je later via een naam een veld weer ophalen:

Java:

JTextField field = (JTextField)fields.get(name);

[Nick_S]

^^^ Met hem, dus.

Sla al je textfields op in een hashmap met als key de naam. Dan kun je ze later ook weer terug vinden.

Grumbl, weer te laat...

[ Voor 32% gewijzigd door Nick_S op 06-10-2005 14:18 ]

[Robtimus]

Standeman schreef op donderdag 06 oktober 2005 @ 14:11:
Ik heb even snel in API gekeken en daar kan ik geen methods vinden om de children van een Container uit te vragen.. Nooit verwacht dat dat niet kon.

Eh:
public int getComponentCount()
public Component getComponent(int i)
public Component[] getComponents()
public Component getComponentAt(int x, int y)
public Component getComponentAt(Point p)

Die eerste drie zelfs zonder API kunnen opsommen

[Robtimus]

momania schreef op donderdag 06 oktober 2005 @ 14:15:
Je kan ze het best, zoals Janoz al aangaf, in een collection bewaren..

offtopic:
Technisch gezien is een Map geen Collection
Maar wel de beste oplossing ja

[momania]

IceManX schreef op donderdag 06 oktober 2005 @ 14:41:
[...]

offtopic:
Technisch gezien is een Map geen Collection
Maar wel de beste oplossing ja

offtopic:
true, alleen een collection interface, maar dat mag de pret niet drukken

[Standeman]

IceManX schreef op donderdag 06 oktober 2005 @ 14:39:
[...]
Eh:
public int getComponentCount()
public Component getComponent(int i)
public Component[] getComponents()
public Component getComponentAt(int x, int y)
public Component getComponentAt(Point p)

Die eerste drie zelfs zonder API kunnen opsommen

ahhaa.. ik was even snel op zoek naar een collection die getChildren() ofzo zou heten..

offtopic:
.NET maakt meer Java kennis stuk dan je lief is
Ben weer lekker bezig vandaag...

[netvor]

Wat je ook kan doen, is je Setting class aanpassen door er een field

Java:

private JComponent component;

in te zetten, die je met een mooie method

Java:

public Object getComponentValue( ) { ... }

uit kan lezen. Dat maakt het allemaal een stuk transparanter voor je. Je hoeft in je applicatie alleen maar de hashmap met alle settinggroups (en daarin alle settings) bij de hand te houden, en vanuit die hashmap kan je automagisch de UI uitlezen.

[Wbdsgnr]

Hmm.. Inderdaad, bedankt voor de reacties allemaal, hier kom ik weer een stuk verder mee!