Klassen Problem

[Gurkus]

Hi
habe Probleme bei der folgenden "Teil" Aufgabe:

Sie besitzen einen Garten mit kleinen Gartenzwergen und möchten die Zwergengruppe
möglichst kostengünstig umzäunen, da der Nachbarshund Ihre Zwerge immer umstößt.
Schreiben Sie die Klasse GardenGnome, welche die Position sowie den Namen eines Zwerges
speichert. Schreiben Sie anschließend eine Klasse Garden, die genug Platz für die Speicherung
von 256 Zwergen bietet.
Entwickeln Sie eine Methode void readGnomes(String filename, Garden garden), die
aus einer vom Benutzer auf der Kommandozeile angegebenen Datei filename im Format
<Name des Zwerges> <X-Koordinate> <Y-Koordinate>
... ... ...
die Namen und Positionen der Zwerge einliest und in garden speichert.



1.Klasse

public class GardenGnome 
{
	String name;
	double[] position;
}

2.Klasse

public class Garden 
{
	GardenGnome[] gnomes;
	
}

3.Klasse

public class GardenPlanner 
{
	static void readGnomes(String filename, Garden garden)
	{
		
	}
	
	public static void main(String args[])
	{
		//VARIABLEDECLARATION
		Garden garden = new Garden();
		garden.gnomes = new GardenGnome[256];
		garden.gnomes.position = new int [2];      //Hier dürfte das Problem liegen ...
		
		
		//INPUT
		In.open(args[0]);
		{
			int j = 0;
			while(In.done())
			{
				garden.gnomes[j].name = new String();             //FEHLER auch ab hier ...
				garden.gnomes[j].name = In.readWord();
				garden.gnomes[j].position[j] = In.readInt();
				garden.gnomes[j].position[j+1] = In.readInt();
				j++;
				
			}
		}
		In.close();
	}
}

Datei zum einlesen

Chef 0.5 1
Hatschi 2 2
Brummbaer 2 4.5
Happy 4 1
Seppl 3.5 2.5

Vielleicht weiss einer wieso dort ein Fehler ist !
Bin einfach am Ende mit meinem Latein ...



Also danke im vorraus !
bis denne
greetz nils

 

[Gurkus]

---push---

 

[anfänger15]

In der Klasse GardenGnome verwendest du einen double-Array und in GardenPlaner versuchst du ein Int-Array zu iniziallisieren.

 

[SlaterB]

was ist denn das Problem, eine NullPointerException?
oder ein Compiler-Mecker-Gewitter?
dann poste das doch auch alles

> garden.gnomes.position = new int [2];

garden.gnomes ist ein Array, ein Array kennt keine Position,
meinst du vielleicht ein Element aus dem Array?
das wäre

garden.gnomes[87].position = new int [2];


später schreibst du doch auch richtiger
> garden.gnomes[j].position[j] = In.readInt();
..

------

zu bedenken ist noch:

das 256er Array ist anfangs leer, überall steht null drin,
du musst die Objekte erst noch einzeln erzeugen in einer Schleife
und dann kannst du auch das int[] erzeugen, dass soll sicherlich auch für jedes der 256 Gnome passieren,

------

generell kannst du noch überlegen, Arbeit in die Konstruktoren zu verschieben,
warum legt nicht der Konstruktor von Garden das gnomes-Array an?
falls nur main() die Anzahl kennt (256), kann man diese als Parameter mit übergeben




edit:
double statt int, auch das noch



edit
> garden.gnomes[j].position[j] = In.readInt();

wenn j 12 ist und das PositionArray nur Indexe 0 und 1 erlaubt, auweia

 

[Gurkus]

HEy danke erstmal für die Antworten !
Aber ich komme einfach irgendwie nicht weiter stehet jetzt hier :

Klasse 1

public class GardenGnome 
{
	String name;
	double[] position = new double[2];
	
}

Klasse 2

public class Garden 
{
	GardenGnome[] gnomes = new GardenGnome[256];
	
}

Klasse 3

public class GardenPlanner 
{

	public static void main(String args[])
	{
		
		//VARIABLEDECLARATION
		Garden garden = new Garden();
		
		
		for(int i = 0;i < 256;i++)
		{
			garden.gnomes[i] = null;
		}
		

		//INPUT
		In.open(args[0]);
		{
			int j = 0;
			while(In.done())
			{
				System.out.println("testing");
				garden.gnomes[j].name = new String();
				garden.gnomes[j].name = In.readWord();
				garden.gnomes[j].position[0] = In.readDouble();
				garden.gnomes[j].position[1] = In.readDouble();
				
				System.out.println(garden.gnomes[j].name);
				System.out.println(garden.gnomes[j].position[0]);
				System.out.println(garden.gnomes[j].position[1]);
				System.out.println("next one");
				
				j++;
				
			}
		}
		In.close();
	}
	
}

Fehler wäre dieser :

Exception in thread "main" java.lang.NullPointerException
	at GardenPlanner.main(GardenPlanner.java:13)

Weiss nicht mehr weiter meine Lektüre hilft mir irgendwie nicht nicht mehr ...
Habe Sprechen sie Java vom Mössenböck
und dürfen nur die behandelten Themen bis einschließlich Kapitel 10 nehmen.
also haben sozusagen nur Klassen und keine Objektorientierung zur Verfügung.
danke nochmal !
greetz nils

 

[SlaterB]

im Konstruktor erzeugst du nur ein Array in einer lokalen Variablen

schreibe
gnomes = new GardenGnome[256];
statt
GardenGnome[] gnomes = new GardenGnome[256];

 

[Gurkus]

im Konstruktor erzeugst du nur ein Array in einer lokalen Variablen

schreibe
gnomes = new GardenGnome[256];
statt
GardenGnome[] gnomes = new GardenGnome[256];

Hm
das verstehe ich nun nicht.
Und so wie du es schreibst klappt es auch nicht bei mir in der Klasse Garden.
Ich verzweifle an der Aufgabe probiere nun schon den ganzen tag daran herum ...
greetz nils

 

[SlaterB]

public class Test {

	String x = "a";

	public Test() {
		x = "b";
		System.out.println("1: " + x);

		String x = "c";
		System.out.println("2: " + x);
		System.out.println("3: " + this.x);
	}

	public static void main(String[] args) throws Exception {
		new Test();
	}
}

Ausgabe:
1: b
2: c
3: b

wenn was nicht geht, dann Fehlermeldung und Code posten

 

[Gurkus]

Fehlermeldung :

Exception in thread "main" java.lang.Error: Unresolved compilation problem: 
	Syntax error on token "gnomes", VariableDeclaratorId expected after this token

	at Garden.<init>(Garden.java:4)
	at GardenPlanner.main(GardenPlanner.java:9)

Klasse 1

public class GardenGnome 
{
	String name;
	double[] position = new double[2];
	
}

Klasse 2

public class Garden 
{
	gnomes = new GardenGnome[256]; 
}

Klasse 3

public class GardenPlanner 
{

	public static void main(String args[])
	{
		
		//VARIABLEDECLARATION
		Garden garden = new Garden();
		
		/*
		for(int i = 0;i < 256;i++)
		{
			garden.gnomes[i].name =;
		}
		

		//INPUT
		In.open(args[0]);
		{
			int j = 0;
			while(In.done())
			{
				System.out.println("testing");
				garden.gnomes[j].name = new String();
				garden.gnomes[j].name = In.readWord();
				garden.gnomes[j].position = new double[2];
				garden.gnomes[j].position[0] = In.readDouble();
				garden.gnomes[j].position[1] = In.readDouble();
				
				System.out.println(garden.gnomes[j].name);
				System.out.println(garden.gnomes[j].position[0]);
				System.out.println(garden.gnomes[j].position[1]);
				System.out.println("next one");
				
				j++;
				
			}
		}
		In.close();
		
		*/
		
	}
	
}

Bin so langsam total verwirrt :shock:

 

[SlaterB]

ok, da habe ich wirlich was ganz anderes gelesen,

schreibe also wieder

> public class Garden
> {
> GardenGnome[] gnomes = new GardenGnome[256];
>
> }

das ist recht gut

allerdings kommt bei mir dann keine

> Exception in thread "main" java.lang.NullPointerException
> at GardenPlanner.main(GardenPlanner.java:13)

für die damalige Zeile 13

> garden.gnomes = null;

deshalb habe ich da was anderes gelesen, du hast auch nichts zwischendurch editiert?

-----

aktuell würde

> garden.gnomes.name =;

allerdings zu einer solchen Exception führen, mal abgesehen von dem fehlenden Namen in diesem Code,

denn garden.gnomes ist ein leeres Array,
nirgendwo wurden neu erzeugte Gnome-Objekte hineingeschrieben

 

[Gurkus]

So Perfekt funtzt soweit alles !!!

Klasse 1

public class GardenGnome 
{
	String name;
	double[] position = new double[2];
	
}

Klasse 2

public class Garden 
{
	GardenGnome[] gnomes = new GardenGnome[256]; 
}

Klasse 3

public class GardenPlanner 
{

	public static void main(String args[])
	{
		
		//VARIABLEDECLARATION
		Garden garden = new Garden();
		
		for(int i = 0;i < 256;i++)
		{
			garden.gnomes[i] = new GardenGnome();
			garden.gnomes[i].name = new String();
		}
		

		//INPUT
		In.open(args[0]);
		{
			int j = 0;
			while(In.done())
			{
				System.out.println("testing");
				garden.gnomes[j].name = In.readWord();
				garden.gnomes[j].position[0] = In.readDouble();
				garden.gnomes[j].position[1] = In.readDouble();
				
				System.out.println(garden.gnomes[j].name);
				System.out.println(garden.gnomes[j].position[0]);
				System.out.println(garden.gnomes[j].position[1]);
				System.out.println("next one");
				
				j++;
				
			}
		}
		In.close();
		
		
	}
	
}

Datendatei mit Daten zum einlesen

Chef 0.5 1
Hatschi 2 2
Brummbaer 2 4.5
Happy 4 1
Seppl 3.5 2.5

Output

testing
Chef
0.5
1.0
next one
testing
Hatschi
2.0
2.0
next one
testing
Brummbaer
2.0
4.5
next one
testing
Happy
4.0
1.0
next one
testing
Seppl
3.5
2.5
next one
testing

0.0
0.0
next one

Danke an alle nochmal !
und noch nen schönen abend + rest we !
greetz nils

 

[SlaterB]

> garden.gnomes.name = new String();

macht keinen Sinn, name kann null bleiben,
später wird ein komplettes String-Objekt zugewiesen,


"new String(" sollte nie in einem Java-Programm auftauchen,
es sei denn man ist absoluter Experte und weiß tatsächlich eine Anwendung dafür
(mir ist noch keine untergekommen)

 

[Gurkus]

> garden.gnomes.name = new String();

macht keinen Sinn, name kann null bleiben,
später wird ein komplettes String-Objekt zugewiesen,


"new String(" sollte nie in einem Java-Programm auftauchen,
es sei denn man ist absoluter Experte und weiß tatsächlich eine Anwendung dafür
(mir ist noch keine untergekommen)

Hey danke
habe es soweit bei mirverbessert nur keine Lust oben alles nochmal umzuschreiben und hier her zu kopieren !
schönen abend noch !
greetz nils