Aus eingelesener Datei korrekt Objekte erzeugen

[fehrenbach1119]

Hallo,
ich hoffe sehr auf eure Hilfe.
Ich habe diese Übungsaufgabe bekommen:
Wir haben einen Parser bekommen der eine Datei einliest und auswertet. In diesem ist eine Methode, die wir überschreiben sollen. Sodass die erstellten Instanzen in anderen Klassen genutzt werden können.
[CODE lang="java" title="Parser"]public static void Reader(String[] args) {
if (args.length > 0) {
File pnmlDatei = new File(args[0]);
if (pnmlDatei.exists()) {
MyPNMLParser mypnmlParser = new MyPNMLParser(pnmlDatei);
mypnmlParser.initParser();
mypnmlParser.parse();

} else {
System.err.println("Die Datei " + pnmlDatei.getAbsolutePath()
+ " wurde nicht gefunden!");
}
} else {
System.out.println("Bitte eine Datei als Parameter angeben!");
}

}

/**
* Diese Methode wird aufgerufen um geladene Transitionen zu
* erstellen.
*
* @param id Identifikationstext der Transition
*/
@Override
public void newTransition(String id) {
Transition t = new Transition(id);

}[/CODE]

Transition t = new Transition(id); habe ich eingefügt aber das Problem wie mir dann klar wurde ist, dass ich Instanzen in einer void Methode erstelle und diese natürlich nicht in anderen Klassen nutzen kann, da kein Rückgabewert.
Ich komme aber nur durch diese Methode an die "id" die ich benötige zur Instanzierung meiner Klasse Transition. Kann mir jemand den Konten lösen, ich befürchte ich denke einfach total falsch.
Vielen Dank

 

[Mart]

du machst es erstmal nicht static

du machst dir eine Klasse mit dem reader zeug

du machst ein Klassen attribut, wenn du die ID bekommst setzt du die ID
von außerhalb kannst du dann an die ID über einen getter ran kommen

 

[fehrenbach1119]

Hallo,
danke für den Tipp.
[CODE lang="java" title="Klasse Parser"]public class MyPNMLParser extends PNMLWopedParser {

public String id;

public MyPNMLParser(File pnml) {
super(pnml);
this.id=id;

}

public void Reader(String[] args) {
if (args.length > 0) {
File pnmlDatei = new File(args[0]);
if (pnmlDatei.exists()) {
MyPNMLParser mypnmlParser = new MyPNMLParser(pnmlDatei);
mypnmlParser.initParser();
mypnmlParser.parse();
mypnmlParser.getID();
System.out.println(getID());

} else {
System.err.println("Die Datei " + pnmlDatei.getAbsolutePath()
+ " wurde nicht gefunden!");
}
} else {
System.out.println("Bitte eine Datei als Parameter angeben!");
}
}

/**
* Diese Methode wird aufgerufen um geladene Transitionen zu
* erstellen.
*
* @param id Identifikationstext der Transition
*/
@Override
public void newTransition(String id) {
setID(id);
}

private void setID(String id) {
this.id =id;
}

public String getID() {
return id; [/CODE]

Leider ergibt die Ausgabe immer null.

 

[Mart]

this.id=id;

im konstruktor bewirkt gar nichts, da du die id setzt mit dem wert der in id steht im moment, die zeile ist unwichtig

mypnmlParser.getID();

falls das java konform ist, heist das du bekommst da die id als return wert

DIESEN musst du in id speichern, dann kannst du id abrufen und dann wirds auchn icht null sein

im moment ist dein Status
public String id; -> steht gar nichts drin, nicht initialisiert
this.id=id; -> du setzt den wert auf sich selbst, ich weis nicht mal warum das geht um ehrlich zu sein thhis.id = id wäre wenn du einen id parameter hättest ( den du aber nicht brauhcst)
System.out.println(getID()); -> hier rufst du ID ab, aber die anderen zwei sachen vorher setzen es ja nicht!

du musst shcon das ergebnis vom parser in die id sepichern sonst wird das nix

 

[fehrenbach1119]

mypnmlParser.getID(); ist leider nicht Java konform.

 

[Mart]

java konform = es heißt get dh ich bekomm was

es hießt getID
dh ich bekomme irgendwas was einer ID entspricht
und das musst du abspeichern
und dann in dem objekt setzen
was für ein datentyp es hat steht halt nicht drin blöderweise