NullPointerException wird geworfen, bitte um Hilfe!

[Kampfschnitzel]

Hallo zusammen,
ich code zurzeit ein "Programm", in dem man Bälle mit den Pfeiltasten beschleunigen kann, und diese dann annähernd physikalisch gebremst werden, sobald sie nicht mehr beschleunigt werden.

Das Programm lief auch immer, ohne das etwas gethrowt wurde, jedoch wird manchmal die NullPointerException geworfen, und manchmal nicht. Ich weiß aber nicht, warum dies so ist. Die Kugeln werden aber trotzdem gezeichnet.

Die NullPointerException wird geworfen, sobald er versucht meine Ovale/Bälle zu zeichnen.

Bitte um Hilfe, da ich noch ein fortgeschrittener Neuling bin.

Hier der Konsoleneintrag:

Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
	at SpieleEntwicklungVersuch1.Frame$Screen.paintComponent(Frame.java:153)
	at javax.swing.JComponent.paint(Unknown Source)
	at javax.swing.JComponent.paintChildren(Unknown Source)
	at javax.swing.JComponent.paint(Unknown Source)
	at javax.swing.JComponent.paintChildren(Unknown Source)
	at javax.swing.JComponent.paint(Unknown Source)
	at javax.swing.JLayeredPane.paint(Unknown Source)
	at javax.swing.JComponent.paintChildren(Unknown Source)
	at javax.swing.JComponent.paintToOffscreen(Unknown Source)
	at javax.swing.RepaintManager$PaintManager.paintDoubleBuffered(Unknown Source)
	at javax.swing.RepaintManager$PaintManager.paint(Unknown Source)
	at javax.swing.RepaintManager.paint(Unknown Source)
	at javax.swing.JComponent.paint(Unknown Source)
	at java.awt.GraphicsCallback$PaintCallback.run(Unknown Source)
	at sun.awt.SunGraphicsCallback.runOneComponent(Unknown Source)
	at sun.awt.SunGraphicsCallback.runComponents(Unknown Source)
	at java.awt.Container.paint(Unknown Source)
	at java.awt.Window.paint(Unknown Source)
	at javax.swing.RepaintManager$3.run(Unknown Source)
	at javax.swing.RepaintManager$3.run(Unknown Source)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.security.ProtectionDomain$1.doIntersectionPrivilege(Unknown Source)
	at javax.swing.RepaintManager.paintDirtyRegions(Unknown Source)
	at javax.swing.RepaintManager.paintDirtyRegions(Unknown Source)
	at javax.swing.RepaintManager.prePaintDirtyRegions(Unknown Source)
	at javax.swing.RepaintManager.access$1100(Unknown Source)
	at javax.swing.RepaintManager$ProcessingRunnable.run(Unknown Source)
	at java.awt.event.InvocationEvent.dispatch(Unknown Source)
	at java.awt.EventQueue.dispatchEventImpl(Unknown Source)
	at java.awt.EventQueue.access$200(Unknown Source)
	at java.awt.EventQueue$3.run(Unknown Source)
	at java.awt.EventQueue$3.run(Unknown Source)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.security.ProtectionDomain$1.doIntersectionPrivilege(Unknown Source)
	at java.awt.EventQueue.dispatchEvent(Unknown Source)
	at java.awt.EventDispatchThread.pumpOneEventForFilters(Unknown Source)
	at java.awt.EventDispatchThread.pumpEventsForFilter(Unknown Source)
	at java.awt.EventDispatchThread.pumpEventsForHierarchy(Unknown Source)
	at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
	at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
	at java.awt.EventDispatchThread.run(Unknown Source)

Klasse Screen mit der gezeichnet wird (interne Klasse, falls dies relevant sein sollte):

private class Screen extends JLabel {
		@Override
		protected void paintComponent(Graphics g) {
			super.paintComponent(g);
			
			
			g.setColor(Color.RED);
			g.fillOval(player1.getBounding().x, player1.getBounding().y,
					player1.getBounding().width, player1.getBounding().height);

			g.setColor(Color.CYAN);
			
			g.fillOval(player2.getBounding().x, player2.getBounding().y,
					player2.getBounding().width, player2.getBounding().height);
		}
}

Durch getBounding().... bekomme ich jeweils die Größen, wie das Oval gezeichnet wird.

Danke im Voraus!ueh:



EDIT: Habe das ganze nun in eine try-catch Anweisung gepackt, Problem besteht aber immernoch.

private class Screen extends JLabel {
		@Override
		protected void paintComponent(Graphics g) {
			super.paintComponent(g);
			
			// hier wird gemalt
			
			// erstellt oval mit den groessen, welche beim erstellen des players
			// deklariert werden
			try {
				g.setColor(Color.RED);
				g.fillOval(player1.getBounding().x, player1.getBounding().y,
					player1.getBounding().width, player1.getBounding().height);

				g.setColor(Color.CYAN);
			
				g.fillOval(player2.getBounding().x, player2.getBounding().y,
					player2.getBounding().width, player2.getBounding().height);
			} catch(NullPointerException e) {
				System.out.println("Fehler beim Zeichnen.");
			}
			
		}
}

 

[Gelöschtes Mitglied 48276]

Welche von den Zeilen ist denn Zeile 153? Da entsteht schließlich der Fehler (siehe Fehlermeldung). Irgendwo in paintComponent() kracht es.

 

[Kampfschnitzel]

Ooh Entschuldigung, dies habe ich vergessen zu sagen.
Die Zeile 153 ist im JavaCode diese hier:

g.fillOval(player1.getBounding().x, player1.getBounding().y,
					player1.getBounding().width, player1.getBounding().height);

 

[Gelöschtes Mitglied 48276]

Dann überprüfe mal, ob entweder player1, oder player1.getBounding() null zurückgiebt. g sollte nicht null sein, denn das wird in der Zeile vorher schon benutzt.

 

[Kampfschnitzel]

ok ich probier mal eben, bin in spätestens 5 minuten zurück!

komisch ist nur, dass er die exception manchmal wirft, und manchmal nicht.

 

[Kampfschnitzel]

Ich werde mal den Code meines Programms in gekürzter Version veröffentlichen.
Die getter liefern immer Werte !=0 raus.

Main:

public class Main {

	public static void main(String[] args) {
		
		//player wird erstellt
		Player1 player1 = new Player1(200, 200, 20, 0.17, 0.05);
		Player1 player2 = new Player1(300, 200, 20, 0.15, 0.05);  //letzten zwei: acceleration, brake
	
		
		Frame f = new Frame(player1, player2);
		
		
		int zeit = 0;
		
		while(true) {
			//berechnen
			player1.update(f.getUp(), f.getDown(), f.getLeft(), f.getRight());
			player2.update(f.getUp(), f.getDown(), f.getLeft(), f.getRight());
			
			f.repaintScreen();
			try {
				Thread.sleep(15);
			} catch (InterruptedException e) {
				e.printStackTrace();
			}
		}
	}

}

Frame:

public class Frame extends JFrame {

	private Screen screen;

	private final Player1 player1;
	private final Player1 player2;

        public Frame(Player1 player1, Player1 player2) { 

		super("SpieleEntwicklungVersuch1");
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		setSize(1500, 600);
		getContentPane().setBackground(Color.DARK_GRAY);
		setVisible(true);
		setResizable(false);
		setLayout(null);
		// Panel
		screen = new Screen();
		screen.setBounds(0, 0, 1000, 600);
		screen.setBackground(Color.BLACK);
		screen.setOpaque(true);
		add(screen);

		// KeyListener
		addKeyListener(new KeyHandler());

		//Anfang Komponenten
                .....
		// Ende Komponenten

		// Player
		this.player1 = player1;
		this.player2 = player2;

	}

         private class Screen extends JLabel {
		@Override
		protected void paintComponent(Graphics g) {
			super.paintComponent(g);
			
			
			try {
				g.setColor(Color.RED);
				g.fillOval(player1.getBounding().x, player1.getBounding().y,
					player1.getBounding().width, player1.getBounding().height);

				g.setColor(Color.CYAN);
			
				g.fillOval(player2.getBounding().x, player2.getBounding().y,
					player2.getBounding().width, player2.getBounding().height);
			} catch(NullPointerException e) {
				System.out.println("Fehler beim Zeichnen.");
				System.out.println(e);
			}
			
		}
	}

Player:

public Player1(int x, int y, int size, double acceleration, double brake) {
		bounding = new Rectangle(x, y, size, size);
		
		
		f_posx = x;
		f_posy = y;
}


public void update(boolean up, boolean down, boolean left, boolean right) {  //booleans für keypressevents

		//berechnungen -> f_posx und f_posy werden geändert

		bounding.x = (int) f_posx;
		bounding.y = (int) f_posy;
		
		
		
}

public int getBounding_x() {
		return bounding.x;
	}

	public int getBounding_y() {
		return bounding.y;
	}

Ich hoffe die Übersicht bleibt bestehen, da ich viele Sachen rausgelassen habe.





Zu erwähnen wäre noch meine Vorgehensweise:
In der Endlosschleife wird immer berechnet und wieder repaintet.

Der Player ist ein Oval, das gezeichnet wird. Um dieses Oval wird diese Rectangle gelegt, um nachher Kollisionen zu berechnen. Bei jedem Berechnen in der Player-Klasse werden die x bzw y koordinaten des rectangles mit den x und y koordinaten des ovals gleichgesetzt.



Das Problem ist halt, dass er manchmal diese Exception wirft, aber manchmal auch nicht.

 

[Gelöschtes Mitglied 48276]

Okay, also meine Vermutung ist, dass das Problem hierin besteht: Im Konstruktor von Frame rufst du add(screen) auf (Container (Java Platform SE 7 )). Dies könnte u.U. dazu führen, dass die Komponente neu gezeichnet werden möchte und ruft Screen.paintComponent() auf. Dieses wiederum möchte auf player1 aus der Frame-Klasse zugreifen, welches noch nicht zugewiesen wurde.

Weise mal im Konstruktor von Frame player1 und player2 zuerst zu, und dann erstelle die Komponenten.

Immer wenn du mit GUI arbeitest, hast du inherente Parallelität, welche dir manchmal ärger einhandeln kann.

 

[Kampfschnitzel]

ok ich danke dir so vielmals!
ich sag gleich bescheid obs klappt!



EDIT:

Und es klappt.! Ich danke dir wirklich von Herzen

Ich staune immer wieder, wie Menschen sich in diesen Codeschnipseln zurecht finden können, und dazu noch Lösungen finden, auf die ich selber niemals kommen würde!