NullPointerException in erweitertem for

[hyperion]

Hallo,

in meinem Programm bekomme ich eine NullPoitnerException in einem erweitertem for. Ich binder festen Überzeugung, dass die Profis den Fehler sofort sehen

getStartPos(laenge);

public List<Point> getStartPos(int laengeSnake)
{	
	int j = hoeheLab / 2;

	List<Point> positions = new LinkedList<Point>();
		
	// Länge Schlange = laengeLab / 3, Schlange soll i nder Mitte starten.
	for( int i = laengeLab / 3; i < laengeSnake + laengeLab / 3; i++)
	{	
	        positions.add(new Point(i, j));
	}
		
		return positions;
}

public void paint(Graphics g)
{
	Point oldPoint = null, newPoint;
		
	[b]for(Point p: positions)[b]
	{			
		if(!(oldPoint == null))
		{	
			newPoint = p;
			g.drawLine(oldPoint.x * 20, oldPoint.y * 20, newPoint.x * 20, newPoint.y * 20);	
		}
		oldPoint = p;
	}
		
}

Im Prinzip habe ich eine Liste mit Punkten über die ich eine Schleife mache. Ziel des ganzen ist die Punkte durch Linien zu verbinden. Ich verstehe nur nicht wie da eine NullPointerException ausgelöst werden kann. Eignetlich sollte die Schleife ja nur solange wiederholen bis kein Element mehr in der Liste ist und prüfen ob der alte Punkt gesetzt ist tue ich ja auch. Übrig bleiben würde dann doch nur noch, dass ich beim Befüllen der Liste einen Punkt = null hinzugefügt hätte oder? Wenn ic haber meine Schleife zum Füllen ansehe, dann sehe ich da nicht die Möglichkeit, dass ein Punkt = null reinrutscht.

mfg hyperion

 

[Wildcard]

Kann ja nur hier sein:

g.drawLine(oldPoint.x * 20, oldPoint.y * 20, newPoint.x * 20, newPoint.y * 20);

Kann nur dann auftreten wenn newPoint==null.

Übrigens:

!(oldPoint == null)

Finde ich schwer lesbar.
Lieber

oldPoint!=null

oder

null!=oldPoint

 

[hyperion]

Kann ja nur hier sein:
Java Code:

1



g.drawLine(oldPoint.x * 20, oldPoint.y * 20, newPoint.x * 20, newPoint.y * 20);

Kann nur dann auftreten wenn newPoint==null.

Genau. Und das beduetet ich habe einen Punkt = null in der Liste hinzugefügt. Wenn ich mir aber meine Methdoe anschaue, welche die Liste füllt:

public List<Point> getStartPos(int laengeSnake)
{	
	int j = hoeheLab / 2;

	List<Point> positions = new LinkedList<Point>();
		
	// Länge Schlange = laengeLab / 3, Schlange soll i nder Mitte starten.
	for( int i = laengeLab / 3; i < laengeSnake + laengeLab / 3; i++)
	{	
	       positions.add(new Point(i, j));
	}
		
	return positions;
}

sehe ich keine Möglichkeit wie hier ein Punkt = null reinrutscht. Eingangsparameter sind: laengeSnake = 7. hoeheLab = 13 und laengeLab = 23.

mfg hyperion

Übrigens:

!(oldPoint == null)

Finde ich schwer lesbar.
Lieber

oldPoint!=null

oder

null!=oldPoint

Danke für den Tipp. Habs korrigiert

 

[SlaterB]

positions selber kann doch null sein, wann wird das gesetzt?

übrigens: wieso raten wir hier, ist doch alles kinderleicht zu prüfen, entweder mit Debugger oder
System.out.println("positions ist: "+positions);

 

[hyperion]

Ja Entschuldigung. Ja du hast recht postions ist null.

In der Methode getStartPos(laenge) wird aber positions richtig gefüllt.

Der Debugger stürzt bei einem ab welchen ich hier nicht gepostet hab. Die Meldung ist "Source not found.".

Hier der Code der gebraucht wird:

public class SnakeGame 
{	
	JLabel labyrinth;
	int laengeLab, hoeheLab;
	int laengeSnake;
	
	SnakeGame()
	{
		laengeLab = 23;
		hoeheLab = 13;
		laengeSnake = 7;
                buildGUI();
	}

        private void buildGUI()
	{
		labyrinth = new Labyrinth(laengeLab, hoeheLab, laengeSnake);
	}
	
	public static void main(String[] input)
	{
		new SnakeGame();
	}
}

class Labyrinth extends JLabel
{
	Snake schlange;
	int laengeLab, hoeheLab;
	List<Point> positions;
	
	
	Labyrinth(int laengeLab, int hoeheLab, int laengeSnake)
	{
		this.laengeLab = laengeLab;
		this.hoeheLab = hoeheLab;
		
		schlange = new Snake(getStartPos(laengeSnake), this);
	}	
	
	public List<Point> getStartPos(int laengeSnake)
	{	
		int j = hoeheLab / 2;

		List<Point> positions = new LinkedList<Point>();
		
		// Länge Schlange = laengeLab / 3, Schlange soll i nder Mitte starten.
		for( int i = laengeLab / 3; i < laengeSnake + laengeLab / 3; i++)
		{	
		   positions.add(new Point(i, j));
		}
		
		return positions;
	}

	public void paint(Graphics g)
	{
		Point oldPoint = null, newPoint;
		
		for(Point p: positions)
		{			
			if(oldPoint != null)
			{	
				newPoint = p;
				g.drawLine(oldPoint.x * 20, oldPoint.y * 20, newPoint.x * 20, newPoint.y * 20);	
			}
			oldPoint = p;
		}
		
	}	
}

public class Snake implements KeyListener
{
	List<Point> positions;
	Labyrinth lab;
	
	
	[b]Snake(List<Point> positions, Labyrinth lab)[/b]
	{
		this.positions = positions;
		this.lab = lab;
	}	
	
	
	
	public void keyReleased(KeyEvent e)
	{
		  
	}
	
	public void keyPressed(KeyEvent e)
	{
		
	}
	
	public void keyTyped(KeyEvent e)
	{
		  
	}
}

Habe den Debugger jetzt zum ersten mal verwendet In der Fett markierten Zeile stürzt er ab.

mfg hyperion

 

[SlaterB]

was hat denn jetzt der Konstruktor von Snake mit der Sache zu tun und wie kann dort etwas abstürzen?

paint von Labyrinth interessiert sich dafür jedenfalls derzeit nicht, Labyrinth selber hat auch eine Liste positions, und die wird nie gesetzt

 

[hyperion]

Ahh, genau des wars.
Hab ich gar nicht gesehen da ich in der Klasse 2 mal positions hatte einmal in der Methode und einmal als Instanzvariable. Die Methode bezieht sich nun auch auf die Instanzvariable und aus

schlange = new Snake(getStartPos(laengeSnake), this);

habe ich

positions = getStartPos(laengeSnake);
schlange = new Snake(positions, this);

gemacht.

mfg hyperion

 

[Lexi]

Point oldPoint = null, newPoint;

Was ist das denn für eine Zuweisung ?
Oder ist das garnicht als soeine gedacht ? Habe sowas noch nie gesehen oO ...

 

[SlaterB]

> Point oldPoint = null, newPoint;

=
Point oldPoint = null;
Point newPoint;