NullPointerExeption in Array

[Pauli85]

Hallo,
ich habe die Aufgabe aus einer Datei Koordinaten einzulesen (sind ca. 100.000 Stück) und diese dann zeichnen zu lassen. Dazu soll ich beim Einlesen die Koordinaten zuerst in einem Point-Objekt speichern. Die Datei sieht in etwa so aus:

66
11
467	191
467	191
467	191
467	191
467	191
467	191
467	191
467	191
467	191
467	191
467	191
212
547	152
547	152
547	152
547	152
547	152
547	152
......

Die erste Zahl gibt die Anzahl der Polygonzüge an, also 66, die zweite die Anzahl der Punkte beim ersten Zug, dann kommen die 11 Punkte, die nächste Zahl sind die Gesamtpunkte beim nächsten Poygonzug, dann kommen die 212 Punkte usw. Diese Koordinaten habe ich in ein pointArray[][] gespeichert und dort ist auch alles richtig (bin das mal mit ner Schleife durchgegangen).

Wenn ich nun zeichnen lassen will, hole ich mir einzeln die Point-Werte und speichere sie in int[] x, int[] y Arrays, um sie der Methode drawPoygon(int[] x, int[]y, int n) zu übergeben. Das sieht dann wie folgt aus:

public void paint(Graphics g) {
		super.paint(g);
		// Schreibt die Werte aus dem Point-Array in int-Arrays und zeichnet dann nacheinander die Polygonzüge
		for(int l = 0; l < pointArray.length; l++) {
			int[] x = new int[pointArray[l].length];
			int[] y = new int[pointArray[l].length];
			for(int i = 0; i < pointArray[l].length; i++) {
				x[i] = (int) pointArray[l][i].getX();
				y[i] = (int) pointArray[l][i].getY();
				//System.out.println(x[i] + " / " + y[i]);
			}
			g.drawPolygon(x,y,pointArray[l].length);
		}
	}

Jeds mal wenn ich jetzt die Klasse ausführe erhalte ich bei [JAVA=5]int[] x = new int[pointArray[l].length];[/code] einen NullPointerExeption, weiß aber nicht wieso, da das Bild ja auch korrekt gezeichnet wird. Wenn ich die Kommentare von [JAVA=10]//System.out.println(x + " / " + y);[/code] wegnehme, funktioniert alles wunderbar. Dies kann ich mir nicht erklären.

Hat jemand eine Ahnung woran es liegen könnte?

Grüße

 

[SlaterB]

die Exception kommt genau deswegen, weil pointArray[l] für irgendein l null ist, einfacher gehts nicht,

deine Ausgaben usw. sind skeptisch zu betrachten, könnten sich auf ein kleines l oder einen vorherigen Durchlauf
der Methode beziehen, wo zufällig manches klappt,

oder du bekommst so viele Ausgaben, dass du die Exceptions gar nicht siehst,
wenn ein paint abgebrochen wird, ist das Programm nicht gleich zu Ende,
vielleicht funktionieren spätere Mal-Vorgänge und ersetzen die alte Ausgabe der Exception

------

gib einfach zu Beginn der Methode die Länge des pointArray aus und in einer Schleife alle Objekte untereinander,
schon siehst du glasklar, ob ein Element im Array null ist oder nicht,
NullPointerExeption sind so leicht zu behandeln..

wenn es natürlich wieder viele paint-Aufrufe gibt, droht auch diese Information verdrängt zu werden


warum im Array nicht alles gesetzt ist kann man bisher nicht erkennen, hängt von anderen Code ab

 

[bERt0r]

Gibst du den Arrays der 2. Dimension beim Einlesen denn immer die richtige Größe oder ist die fix?

 

[Pauli85]

die Exception kommt genau deswegen, weil pointArray[l] für irgendein l null ist, einfacher gehts nicht,
....
gib einfach zu Beginn der Methode die Länge des pointArray aus und in einer Schleife alle Objekte untereinander,
schon siehst du glasklar, ob ein Element im Array null ist oder nicht,
NullPointerExeption sind so leicht zu behandeln..
...

Ich hab alle Einträge schon mal auf null überprüft und auch pointArray[l], aber alles war so wie es sein soll. Dies habe ich allerdings außerhalb meiner paint(Graphics g) Methode gemacht (pointArray[][] ist Objektvariable). Wie gesagt, alles ohne Fehler.
Wenn ich aber die selbe Überprüfung in der Methode paint(Graphics g) mache, bekomme ich die Exeption. Wie kann das sein?

Hier mal mein Code:

import java.awt.Graphics;
import java.awt.Point;
import java.io.BufferedReader;
import java.io.File;
import java.io.FileInputStream;
import java.io.FileNotFoundException;
import java.io.IOException;
import java.io.InputStreamReader;

import javax.swing.JFrame;
import javax.swing.SwingUtilities;

public class Drawing extends JFrame {
	
	private Point[][] pointArray; // Array in dem alle Punkte gespeichert werden

	Drawing() { // Erzeugung des Fensters
		setSize(500,500);
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		setVisible(true);
	}
		
	public void load(File datei) {
		try {
			// input ließt alle Werte aus dem Dokument
			BufferedReader input = new BufferedReader(new InputStreamReader(new FileInputStream(datei)));
			int numberOfPoly = Integer.parseInt(input.readLine());		
			pointArray = new Point[numberOfPoly][]; 
			// Schreibt alle Werte aus der Datei in das Array als x,y-Koordinaten (Point)
			for(int l = 0; l < numberOfPoly; l++) {	
				int numberOfPoints = Integer.parseInt(input.readLine());	
				Point[] points = new Point[numberOfPoints];
				for(int i = 0; i < numberOfPoints; i++) {
					String line = input.readLine();
					String[] lineArray = line.split("\t");
					points[i] = new Point(Integer.parseInt(lineArray[0]),Integer.parseInt(lineArray[1]));
				}
				pointArray[l] = points;
			}			

//		//Zum testen
//		for(int i = 0; i < pointArray.length; i++) {
//				for(int l = 0; l < pointArray[0].length; l++) {
//					System.out.println(pointArray[i][l].getX() +" / " + pointArray[i][l].getY());	
//				}
//			}
			
		} catch (FileNotFoundException e) {
			e.printStackTrace();
		}  catch (IOException e) {
			e.printStackTrace();
		}
	}
	
	public void paint(Graphics g) {
		super.paint(g);
		g.translate(50,50);
		// Schreibt die Werte aus dem Point-Array in int-Arrays und zeichnet dann nacheinander die Polygonzüge
		for(int l = 0; l < pointArray.length; l++) {
			//System.out.println(l +": " +pointArray[l].length); //Test
			int[] x = new int[pointArray[l].length];
			int[] y = new int[pointArray[l].length];
			for(int i = 0; i < pointArray[l].length; i++) {
				x[i] = (int) pointArray[l][i].getX();
				y[i] = (int) pointArray[l][i].getY();
				//System.out.println(x[i] + " / " + y[i]);  //Test
			}
			g.drawPolygon(x,y,pointArray[l].length);
		}
	}
}

Hier ist auch noch die Datei mit den Koordinaten falls es jemanden interessiert:
https://www.dropbox.com/s/col58k5rm3kish3/de0.pol

 

[SlaterB]

eine einzelne Datei kann keine Aussage über ein Programm treffen,
vielleicht gibt es zwei Drawing-Objekte, in einem wird das Array initialisiert, im anderen paint() ausgeführt..

füge in den Konstruktor eine Ausgabe ein um zu erkennen, ob mehrere erstellt werden

 

[Pauli85]

Ne, wird nur ein Drawing Objekt erstellt.
Ich habe gerade gemerkt, dass wenn ich das Fenster vergrößere oder verkleinere die Zeichnung neu angefertigt wird. Wenn ich die jeweilige Größe des Arrays, also rowArray.length mitausgeben lasse, dann bekomm ich meistens beim 28. Eintrag die Exeption. Wenn ich aber nun die Größe des Fensters verändere und das Bild neu gezeichnet wird, dann läuft meine Ausgabe ohne Fehler bis zum Schluss durch. Also nur beim ersten Zeichenvorgang wird irgendwo eine null erkannt, danach ist dieses null plötzlich weg.

Grüße

 

[bERt0r]

Wann rufst du denn deine load Methode auf? Dein pointArray wird nicht initialisiert. Nachdem du in deinem Konstruktor ein setVisible(true) drin hast, versucht dein Drawing Objekt gleich mal ein paintComponent, obwohl pointArray noch nicht initialisiert ist. Entweder du gibst das load in den konstruktor oder löschst das setVisible raus und machst es nachdem erfolgreich geladen wurde.
Eine andere möglichkeit wäre im paintComponent pointArray==null abzufragen...

 

[Pauli85]

Genau daran habe ich nicht gedacht, dass paint() ja schon automatisch zu beginn ausgeführt wird. Jetzt habe ich das setVisible(true) in die load-Methode gepackt, danke!

Ich habe noch eine kleine Frage. Kannst du mir vielleicht sagen wie ich meinem Fenster bzw. dem Zeichenbereich noch einen JScrollPane hinzufüge? Hier noch mal mein Code:

public class Drawing extends JFrame {

	private Point[][] pointArray; // Array in dem alle Punkte gespeichert werden
	
	public Drawing() { // Erzeugung des Fensters
		setSize(600,600);
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		setVisible(false);
	}
	
	public void load(File datei) {
		try {
			// input ließt alle Werte aus dem Dokument
			BufferedReader input = new BufferedReader(new InputStreamReader(new FileInputStream(datei)));
			int numberOfPoly = Integer.parseInt(input.readLine());		
			pointArray = new Point[numberOfPoly][]; 
			// Schreibt alle Werte aus der Datei in das Array als x,y-Koordinaten (Point)
			for(int l = 0; l < numberOfPoly; l++) {	
				int numberOfPoints = Integer.parseInt(input.readLine());	
				Point[] points = new Point[numberOfPoints];
				for(int i = 0; i < numberOfPoints; i++) {
					String line = input.readLine();
					String[] lineArray = line.split("\t");
					points[i] = new Point(Integer.parseInt(lineArray[0]),Integer.parseInt(lineArray[1]));
				}
				pointArray[l] = points;
			}						
		} catch (FileNotFoundException e) {
			e.printStackTrace();
		}  catch (IOException e) {
			e.printStackTrace();
		}
		this.setVisible(true);
	}
	
	public void paint(Graphics g) {		
		super.paint(g);
		g.translate(30,30);
		for(int i = 0; i < pointArray.length; i++) { 
			for(int l = 0; l < pointArray[i].length-1; l++) {
				g.fillRect((int)pointArray[i][l].getX(), (int)pointArray[i][l].getY(), 3, 3);
				g.drawLine((int)pointArray[i][l].getX(), (int)pointArray[i][l].getY(), (int)pointArray[i][l+1].getX(), (int)pointArray[i][l+1].getY());
			}
		}
	}
}

Grüße

 

[bERt0r]

Das geht mit deiner Struktur nicht so leicht. Ich mache z.B immer den Frame in einer eigenen "MainFrame" Klasse, welche nur die Komponenten anordnet. Wenn du jetzt spezielle Komponenten mit eigenen Funktionalitäten hast, wie dein Drawing teil, realisiere ich das als JPanel oder JComponent. Das kannst du dann bequem in ein Scrollpane packen.

 

[California]

Leite Drawing statt von JFrame von JPanel ab.
Mach eine MainFrame Klasse extends JFrame.
Setze dort die ContentPane auf eineJScrollPane, die wiederum deine Drawing- Klasse enthält:

public class MainFrame extends JFrame {

main () {
   new MainFrame.setVisible( true );
}

public MainFrame() {
      JscrollPane scrollPane = new JScrollPane( new Drawing() );
      setContentPane( scrollPane );
... // Scroll- Policy setzen...
// sonstiger FrameSchmus mit defaultCloseOperation und LookAndFeel usw...
}

Code ist symbolisch und dient nur zur Erläuterung des Prinzips.

 

[Pauli85]

Danke für eure Hilfe!