getter/setter bei JTextField ?

[Meteora]

Hallo,

Ich habe ca. 50 JTextFields und benötige dafür 50 getXX/setXX Methoden

Lasse ich diese automatisch erstellen:

private JTextField aSTF = new JTextField();

public void setASTF(JTextField aSTF) {
		this.aSTF = aSTF;
	}

bekomme ich als return wert das JTextField selbst... doch ich will den Text daraus als return wert sprich ich muss von Hand alle 100 get/set Methoden mit getText()/setText() nachrüsten was zeitraubend ist und nervig dazu... geht das nicht geschickter?

 

[0x7F800000]

1) Frage: Bist du dir sicher, dass ale diese 50 Felder sich voneinander so wesentlich unterscheiden, dass du für jedes einzelne eine extra variable anlegen musst? Könntest du nicht den ganzen kram evtl in ein array packen, und alle 50 gleichzeitig verwalten?

2) Frage: wo ist das Problem mit dem herausholen von Text, die stellen doch alle getText() zur verfügung?
edit: achso, ja, du hast nur keinen nerv dafür das alles 100 mal einzutippen, verständlich...

 

[Meteora]

1) Frage: Bist du dir sicher, dass ale diese 50 Felder sich voneinander so wesentlich unterscheiden, dass du für jedes einzelne eine extra variable anlegen musst? Könntest du nicht den ganzen kram evtl in ein array packen, und alle 50 gleichzeitig verwalten?

2) Frage: wo ist das Problem mit dem herausholen von Text, die stellen doch alle getText() zur verfügung?
edit: achso, ja, du hast nur keinen nerv dafür das alles 100 mal einzutippen, verständlich...

1. die ca. 50 Textfelder enthalten alle unterschiedliche Werte die gleich sein können aber nicht müssen sprich ich brauche eine individuelle abfrage mit get und ebenso eine mit set wenn ich die werte aus der xml datei lese in das jeweilige textfeld hinein zurück.

2.)war das jetzt ironisch? ich meine es ist doch ziemlich blöd von 50 gettern den rückgabewert auf String zu ändern, an jeden textfeld eine .getText() zu hängen etc...

eclipse könnte die get/set Kreationen ruhig individueller einstellbar machen von mir aus...

 

[Verjigorm]

Textfelder in ein Array packen und dann eine Get-Methode bauen:

So in etwa

public String getText(int n)
{
     return textfields[n].getText();
}

edit: Mist zu langsam
edit2: Wozu gibts find/replace?

 

[0x7F800000]

1. die ca. 50 Textfelder enthalten alle unterschiedliche Werte die gleich sein können aber nicht müssen sprich ich brauche eine individuelle abfrage mit get und ebenso eine mit set wenn ich die werte aus der xml datei lese in das jeweilige textfeld hinein zurück.

das heisst schonmal, dass all diese werte in etwa gleich aussehen... Bist du dir sicher dass du diese 50 textfelder nicht in ein einziges array mit einem einzigen setter und einem einzigen getter (siehe beispiel von Verjigorm) quätschen kannst?

2.)war das jetzt ironisch?

nein, nein, wars nicht, ich depp hab einfach deinen post nicht zu ende gelesen, sry^^

@ Verjigorm: ein getter mit "void" als rückgabetyp ist eine tolle sache

edit:

Wozu gibts find/replace?

na, wenn man schon code mit einer for()-schleife erzeugen muss, da muss doch an dem konzept irgendetwas mächtig faul sein... :roll:

 

[Verjigorm]

hups, das kommt davon, wenn man sich nicht entscheiden kann ob man einen Getter oder einen Setter schreiben will, weil man zu faul ist beiden zu tippseln
da sind eh noch mehr Rechtschreibfehler drinne

wobei dein "quätschen" auch sehr cool ist, kommt das von Quatsch? oder meinst du quetschen?

 

[Meteora]

Textfelder in ein Array packen und dann eine Get-methode bauen:

So in etwa

public void gettext(int n)
{
     return textfields[n].getText();
}

edit: Mist zu langsam
edit2: Wozu gibts find/replace?

das würde aber bedeuten, dass ich als Coder genau wissen muss an welcher Stelle im Array welche JTextField instanz sich befindet, das ist ja mal assi sorry ... verstehe ich das wirklich richtig dass alle 50 JTFields in eine ArrayList eingelesen werden sollen ? Und dann greife ich über den index darauf zu?? :noe:

 

[0x7F800000]

wobei dein "quätschen" auch sehr cool ist, kommt das von Quatsch? oder meinst du quetschen?

erwischt^^ ja, ich nix deutsch da stand ja auch anfangs "quatschen" (tippfehler, der mir aufgefallen ist) dann hab ich noch zwei minuten lang überlegt wie ich das nachbessere ???:L , und dann ist da halt so ein müll rausgekommen :lol: ich meinte "quetschen" :toll:

 

[Verjigorm]

Textfelder in ein Array packen und dann eine Get-methode bauen:

So in etwa

public void gettext(int n)
{
     return textfields[n].getText();
}

edit: Mist zu langsam
edit2: Wozu gibts find/replace?

das würde aber bedeuten, dass ich als Coder genau wissen muss an welcher Stelle im Array welche JTextField instanz sich befindet, das ist ja mal assi sorry ... verstehe ich das wirklich richtig dass alle 50 JTFields in eine ArrayList eingelesen werden sollen ? Und dann greife ich über den index darauf zu?? :noe:

ähm du musst dir doch auch die Namen von 50 Gettern/Settern merken bzw. deren Textfelder oO

ansonsten benutz halt Find/Replace von Eclipse, sollte keine 2Min dauern die getter/Setter zu bearbeiten...

 

[Meteora]

ähm du musst dir doch auch die Namen von 50 Gettern/Settern merken bzw. deren Textfelder oO

Namen sind semantisch, zahlen nicht in dem Fall. Greife ich direkt auf ein JTF zu sehe ich auch den Namen. Greife ich auf eine Nummer zu frage ich mich was dahinter ist...

 

[ARadauer]

Meteora hat recht ArrayList is da nicht so optimal, ich würd eine HasMap nehmen.. code folgt: will nicht das mir wer zuvorkommt mit der hasmap idee ;-)

 

[Meteora]

Meteora hat recht ArrayList is da nicht so optimal, ich würd eine HasMap nehmen.. code folgt: will nicht das mir wer zuvorkommt mit der hasmap idee ;-)

au ja auf die idee bin ich gespannt supi!! :wink:

 

[ARadauer]

hier mein Code:

public class Test{
HashMap<String, JTextField> textFelder;
public void initFelder(){
		textFelder = new HashMap<String, JTextField>();
		textFelder.put("NAME", new JTextField());
		textFelder.put("ALTER", new JTextField());
		textFelder.put("DURST", new JTextField());
		usw...
	}
	
	public String getValue(String key){
		return textFelder.get(key).getText();
	}
	
	public void setValue(String key, String text){
		textFelder.get(key).setText(text);
	}
}

Elemente können anhand eines eindeutigne Schlüssels, (hier ein String) eingefügt und wieder heraus gehohlt werden. Würde einem assiozativen Array in PHP entsprechen....

 

[0x7F800000]

das würde aber bedeuten, dass ich als Coder genau wissen muss an welcher Stelle im Array welche JTextField instanz sich befindet, das ist ja mal assi sorry ... verstehe ich das wirklich richtig dass alle 50 JTFields in eine ArrayList eingelesen werden sollen ? Und dann greife ich über den index darauf zu?? :noe:

sind alle diese felder in deinem programm also doch extrem individuell, haben jeweils eine vollkommen andere bedeutung, die dort eingegebenen daten werden alle völlig unterschiedlich behandelt und so weiter? Dann verstehe ich gar nicht, warum du dir um die 50 getter setter sorgen machst: Wenn alle diese daten auf völlig unterschiedliche art und weise verarbeitet werden müssen, kommt auf dich ein recht monströses stück arbeit zu, da machen solche getter setter doch gar nichts mehr aus...

Wir könnten das natürlich wesentlich besser beurteilen, wenn du uns sagen würdest, was da in diese ganzen felder eigentlich reinkommt...

ansonsten könnte ich noch anbieten, alle felder in ein array zu packen, aber statt sich indizes zu merken, einfach eine liste mit strings mitspeichern, jedem string so ein Textfeld zuweisen, und dann über diese schlüsselwörter auf die einzelnen Textfelder zugreifen... Ist zwar unüblich, und etwas lahm, aber besser als 50 getter zu schreiben oder indizes zu merken... Oder kA... erzeug dir für jeden index eine spezielle konstante mit einem vernünftigen namen etwa:

//alles in array packen
JtextField[] tf=new JtextField[50];
//einen einzigen getter/setter schreiben wie Verjigorm vorgeschlagen hat
...
//50 konstanten erstellen, die diesen indizes irgendeinen halbwegs sinnvollen namen geben...
public final int TF_ERSTES_TOTAL_WICHTIGES_TEXTFELD=0;

:autsch:

edit: okay, ich bin ein krüppel, hört lieber auf ARadauer^^ hashmap heissen solche sachen, okay^^ auch ma wida was dazugelernt^^

 

[ARadauer]

ansonsten könnte ich noch anbieten, alle felder in ein array zu packen, aber statt sich indizes zu merken, einfach eine liste mit strings mitspeichern, jedem string so ein Textfeld zuweisen, und dann über diese schlüsselwörter auf die einzelnen Textfelder zugreifen... Ist zwar unüblich, und etwas lahm, aber besser als 50 getter zu schreiben oder indizes zu merken... Oder kA... erzeug dir für jeden index eine spezielle konstante mit einem vernünftigen namen etwa:

ich sag nur hashmap

 

[0x7F800000]

ich sag nur hashmap

iss ja gut, ich bin ein noob, steinigt mich bitte
:bae: :toll:

 

[ARadauer]

nein nein, niemand steinigt dich! ist doch gut, dass du dich mit dem thema auseinandersetzt.
Ich hab mir sicher auch 1-2 mal ein zusätzliches String Array angelegt um mir über einen String den index für das eigentliche Array mit den Daten zu suchen. - bis ich die HashMap kennen gelernt habe.

 

[Meteora]

hier mein Code:

public class Test{
HashMap<String, JTextField> textFelder;
public void initFelder(){
		textFelder = new HashMap<String, JTextField>();
		textFelder.put("NAME", new JTextField());
		textFelder.put("ALTER", new JTextField());
		textFelder.put("DURST", new JTextField());
		usw...
	}
	
	public String getValue(String key){
		return textFelder.get(key).getText();
	}
	
	public void setValue(String key, String text){
		textFelder.get(key).setText(text);
	}
}

Elemente können anhand eines eindeutigne Schlüssels, (hier ein String) eingefügt und wieder heraus gehohlt werden. Würde einem assiozativen Array in PHP entsprechen....

Danke dir, das probiere ich aus!

 

[Meteora]

hier mein Code:

public class Test{
HashMap<String, JTextField> textFelder;
public void initFelder(){
		textFelder = new HashMap<String, JTextField>();
		textFelder.put("NAME", new JTextField());
		textFelder.put("ALTER", new JTextField());
		textFelder.put("DURST", new JTextField());
		usw...
	}
	
	public String getValue(String key){
		return textFelder.get(key).getText();
	}
	
	public void setValue(String key, String text){
		textFelder.get(key).setText(text);
	}
}

Elemente können anhand eines eindeutigne Schlüssels, (hier ein String) eingefügt und wieder heraus gehohlt werden. Würde einem assiozativen Array in PHP entsprechen....

Hm jetzt habe ich aber das problem, dass die JTFields beim "putten" erzeugt werden, wobei ich die JTFields ja schon vorher erzeugte in meiner Klasse mit

JTextField bla = new JTextField();
add(bla);
bla.setBounds(0,0,100,25);

wie stelle ich denn das an, dass ich nicht doppelt die JTextfields erstelle oder denke ich falsch?

 

[Tobias]

Indem du an deinen Code ein

textFelder.put(blah);

anhängst.

 

[Murray]

Indem du an deinen Code ein

textFelder.put(blah);

anhängst.

Da fehlt dann allerdings noch der Key:

textFelder.put( "BLAH", blah);

Ich würde auf jeden Fall für die Namen Konstanten definieren - oder sogar einen enum-Typ dafür verwenden. Und wenn man vorher weiß (oder auch per Reflection ermitteln kann), wieviele TextFields es sind, dann ist m.E. ein Array doch besser als eine HashMap.

 

[Murray]

Ein Beispiel, wie man das relativ redundanzfrei mit Annotations und einem Aufzählungstyp für die typsichere Adressierung machen könnte:

import java.lang.annotation.Annotation;
import java.lang.annotation.ElementType;
import java.lang.annotation.Retention;
import java.lang.annotation.RetentionPolicy;
import java.lang.reflect.Field;
import java.util.ArrayList;

import javax.swing.JFrame;
import javax.swing.JTextField;

public class TFList extends JFrame {
	
	@Retention( RetentionPolicy.RUNTIME)
	@java.lang.annotation.Target( ElementType.FIELD)
	public @interface RegisteredField {
		TFKey key();
	}

	public enum TFKey{ name, alter, durst};

	private JTextField[] textFields;

	@RegisteredField( key=TFKey.name)
	private JTextField tfName;
	
	@RegisteredField( key=TFKey.alter)
	private JTextField tfAlter;

	@RegisteredField( key=TFKey.durst)
	private JTextField tfDurst;
	
	private void initList() {
		
		try {
			ArrayList<JTextField> list = new ArrayList<JTextField>(); 
			Field[] flds = getClass().getDeclaredFields();
			for ( Field fld : flds) {
				
				if ( !JTextField.class.isAssignableFrom( fld.getType())) continue;
				JTextField theTextField;
				try {
					theTextField = (JTextField)fld.get( this);
				} catch ( Exception e) {
					if ( e instanceof RuntimeException) throw (RuntimeException)e;
					throw new RuntimeException( e);
				}
				for ( Annotation annotation : fld.getAnnotations()) {
					if( annotation instanceof RegisteredField) {
						RegisteredField registeredField = (RegisteredField)annotation;
						list.add( registeredField.key().ordinal(), (JTextField)fld.get( this));
						break;
					}
				}
			}
			
			textFields = list.toArray( new JTextField[ list.size()]);
		} catch ( Exception e) {
			if ( e instanceof RuntimeException) throw (RuntimeException)e;
			throw new RuntimeException( e);
		}
	}
	
	public TFList() {
		
		super( TFList.class.getName());
		
		setSize( 200,200);
		setVisible( true);
		
		tfAlter = new JTextField();
		add( tfAlter);
		tfAlter.setBounds( 0, 0, 100, 50);

		tfName = new JTextField();
		add( tfName);
		tfName.setBounds( 0, 50, 100, 50);

		tfDurst = new JTextField();
		add( tfDurst);
		tfDurst.setBounds( 0, 100, 100, 50);
		
		setDefaultCloseOperation( JFrame.EXIT_ON_CLOSE);
		
		initList();
	}

	public String getText( TFKey key) {
		return textFields[ key.ordinal()].getText();
	}

	public void setText( TFKey key, String txt) {
		textFields[ key.ordinal()].setText( txt);
	}
	
	public static void main( String[] args) {
		TFList tfl = new TFList();
		
		tfl.setText( TFKey.alter, "ALTER");
		tfl.setText( TFKey.durst, "DURST");
		tfl.setText( TFKey.name,  "NAME");
	}
	
}