Wie Objektparamter auslesen?

[MaikHSW]

Hey zusammen.
Ich habe ein Programm gelesen, was xml-Dateien einliest.
Das einlesen bzw. vorbereiten für die Objektanlage inkl. Anlage sieht folgendermaßen aus:

....

for(int i=Lizenzanzahl-1; i!=-1;i--){
		Element Lizenz=(Element) Lizenzen.get(i);
		Element id=Lizenz.getChild("id");
		Element eval=Lizenz.getChild("eval");
		Element von=Lizenz.getChild("von");
		Element bis=Lizenz.getChild("bis");
		Element system=Lizenz.getChild("isSystem");
		Element maxuser=Lizenz.getChild("maxusers");
		Lic License =new Lic(id.getText(),eval.getText(),von.getText(),bis.getText(),system.getText(),maxuser.getText());
		Lizenzliste.add(License);
		}

Nun bin ich aber zu doof, mit den Objekten (testweise erstmal 2) weiterzuarbeiten.
Die Klasse, in der die Objekte erzeugt werden sieht mit Konstruktor so aus:

public class Lic {
	String eval;
	String von;
	String bis;
	String system;
	String maxuser;
	
public Lic(String Lizenz, String eval, String von, String bis, String system, String maxuser){	
this.eval=eval;

}
public String geteval (){
	return eval;
	
}}

Nun beginne ich danach mit einem Iterator die Liste zu durchwandern und hole mir eine testweise Ausgabe:

Iterator Lizenze=Lizenzliste.iterator();
		Lic Testfall=(Lic) Lizenze.next();
		String Prüfung=Testfall.geteval();
		Lic Testfall2=(Lic) Lizenze.next();
		String Prüfung2=Testfall.geteval();
	    System.out.println("Iterator"+Prüfung);
	    System.out.println("IT2"+Prüfung2);

Nun könnt ihr euch denken was passiert.
Ich bekomme leider nicht wie gewünscht verschiedene Parameter (in den Objekten sind sie verschieden) sondern immer schön den gleichen, nämlich den bei der Erzeugung letzten Parameterwert.
Das ist natürlich nicht gewollt.
Ich habe da nun einige Tage dran probiert und verschiedenste Vorgehensweisen getestet, ich komm nicht drauf.

Kann mir da einer mal aufs Pferd helfen?

Danke und Gruß


Maik

 

[Civilazi]

Ich sehe da jetzt nicht gleich durch, aber sollte hier

String Prüfung2=Testfall.geteval();

nicht sowas stehen?

String Prüfung2=Testfall2.geteval();

 

[Andreas29]

Hi,

wenn du dir auch zweimal den gleichen Wert holst, wird die Ausgabe auch die gleiche sein...
Versuch es mal mit:

Iterator Lizenze=Lizenzliste.iterator();
Lic Testfall=(Lic) Lizenze.next();
String Prüfung=Testfall.geteval();
Lic Testfall2=(Lic) Lizenze.next();
String Prüfung2=Testfall2.geteval();
System.out.println("Iterator"+Prüfung);
System.out.println("IT2"+Prüfung2);

PS:
Variablen schreibt man klein. Also testfall anstatt Testfall und üblich sind englische Namen.

 

[MaikHSW]

Hey zusammen.
Oh man, wie doof von mir.
Aber nun wirklich,
der Fehler ist mehr als laienhaft und doof.
Das war es aber in der Tat,
ich danke euch.
Somit ist das Thema erledigt.

Ja zum Thema Variablen:
Da muß ich noch einiges lernen und mich an einige Konventionen gewöhnen um mal wirklich was besseres auf die Beine zu stellen.

Danke euch allen.


Gruß

 

[Gastredner]

for(int i=Lizenzanzahl-1; i!=-1;i--){
		//...
}

Erlaube mir noch eine kurze Anmerkung zu deiner Schleife bzw. ihrer Abbruchbedingung:
Es ist ein wenig unglücklich, Bedingungen wie x != -1 zu verwenden. Bei Integern mag das prinzipiell kein Problem sein, wenn du aber irgendwann mal ein Float oder Double dort haben solltest, könntest du ein Problem kriegen. Es kann nämlich vorkommen, dass es auch mal zu gewissen Ungenauigkeiten kommt, weshalb dein Float dann den Wert -1,0001 (oder so) annimmt und die Bedingung daher weiterhin als erfüllt gilt. Außerdem könnte es passieren, dass du die Zahlvariable im Schleifenrumpf änderst. Wenn sie dabei versehentlich unter die -1 rutscht, geht der Vorgang dennoch weiter und resultiert folglich in einer Exception.
Besser - oder sicherer - sind daher Bedingungen, die den inakzeptablen Wertebereich komplett ausschließen, nicht nur seinen Anfang. Hier also z. B.:

for(int i=Lizenzanzahl-1; i>-1;i--){
		//...
}
//Oder:
for(int i=Lizenzanzahl-1; i>=0;i--){
		//...
}

Und wenn man schon mit Collections arbeitet (und mit Java >v1.5), dann kann man auch gleich die foreach-Schleife verwenden (es sei denn, du musst die Liste zwingend von Hinten durchlaufen):

for(Element e: Lizenzen){
        Element id = e.getChild("id");
        Element eval = e.getChild("eval");
	//...
}