Klasse Graphics Nullpointerexception

[Jaensk]

Hallo,

versuche gerade so ne Art Spaceballs zu programmieren. Scheitere jedoch schon am Zeichnen des Balls. Ich versuche die Bewegung mit Threads zu realisieren bekomme jedoch andauernd eine nullpointerexception. Diese Exception wird geworfen sobald versucht wird den Kreis zu zeichnen. Was habe ich falsch gemacht?

package spaceball;

import java.awt.Color;
import java.awt.Component;
import java.awt.Dimension;
import java.awt.Graphics;
import java.awt.Graphics2D;
import javax.swing.BorderFactory;
import javax.swing.JComponent;
import javax.swing.JFrame;
import javax.swing.JPanel;

/**
 *
 * @author 
 */
public class Spaceball extends JFrame{

    
    
    public Spaceball(){
        super("Spaceball");
        setDefaultCloseOperation(EXIT_ON_CLOSE);
        setBounds(450,300,450,450);
        final JPanel space = new JPanel(); 
        space.setBackground(Color.ORANGE);
        space.setBorder( BorderFactory.createLoweredBevelBorder());
        Ball ball = new Ball(space);
        add(space);
        ball.start();
        setVisible(true);
             
   
    }
    

    static class Ball extends Thread{
     
        JComponent comp;   
        
        int radius1 = 20;
        int radius2 = 20;
        int startx = 200;
        int starty = 400;
        int velocityx = 2;
        int velocityy = 2;
        
        public Ball(JComponent comp){
            this.comp = comp;
        }
        
        public void run(){
            
            Graphics graphic = comp.getGraphics();
            graphic.setColor(Color.BLACK);
            graphic.drawOval(startx, starty, radius1, radius2);
            
        }
     
    }
 
}

 

[Wildcard]

So geht das nicht.
http://www.java-forum.org/de/topic46550_zeichnen-swing-tutorial.html

 

[Marco13]

Und nochmal mein Lieblingssatz: Auf Components 'getGraphics' aufzurufen ist fast IMMER falsch. ALLES, was gezeichnet wird, muss von der paint/paintComponent-Methode aus gezeichnet werden - und zwar in das Graphics, das dort übergeben wurde. Pflichtlektüre: http://java.sun.com/products/jfc/tsc/articles/painting/

 

[Jaensk]

Mit der paint Methode habe ich es auch schon probiert, dort es auch funktioniert. aber ich kann die paint-Methode ja nicht in die run-Methode implementieren oder geht das irgendwie? Wie kann ich aus der Run-Methode heraus die Paint-Methode aufrufen? Bin Anfänger :roll: .

 

[Niki]

mit repaint auf die Komponente die du neu zeichnen möchtest, also einfach comp.repaint();

 

[Jaensk]

Irgendwie hat das mit dem repaitn auch nicht funktioniert. Gibt es keine Möglichkeit den oberen Quelltext so umzuschreiben das es funktioniert?

 

[Marco13]

Ja, du mußt nur zwischen die erste { und die letzte } anderen Code einfügen :bae:
Mal im ernst: Das oben verlinkte Tutorial ist schon sehr ausführlich. Man kann eben nicht einfach munter runterschreiben, und dann erwarten, dass alles funktioniert. Selbst wenn jetzt jemand den geposteten Code ändern würde, wüßtest du nicht, was die Änderungen bedeuten, warum sie so sind, wie sie sind, und wie man was anders machen könnte, damit es so wird, wie du willst...

 

[Jaensk]

package spaceball;

import java.awt.Color;
import java.awt.Dimension;
import java.awt.Graphics;
import java.awt.GridLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import javax.swing.BorderFactory;
import javax.swing.JButton;
import javax.swing.JComponent;
import javax.swing.JFrame;
import javax.swing.JPanel;

/**
 *
 * @author Janosch
 */
public class Spaceball extends JFrame{
    
    public Spaceball(){
   
        super("Spaceball");
        setDefaultCloseOperation(EXIT_ON_CLOSE);
        JPanel contentPane1 = new JPanel();
        final JPanel space = new JPanel();
        space.setPreferredSize( new Dimension(640, 480));
        space.setBackground( Color.white);
        space.setBorder( BorderFactory.createLoweredBevelBorder());
        contentPane1.add(space);
        JButton startButton = new JButton("Start");
        startButton.addActionListener(new ActionListener(){
            public void actionPerformed(ActionEvent e){
              Ball b = new Ball(space);
              b.start();
            }
        });
              
         contentPane1.add(startButton);
         add(contentPane1);
         pack();
    }
   
    static class Ball extends Thread{
        
        JComponent comp;
        
        int radius1 = 20;
        int radius2 = 20;
        int startx = 240;
        int starty = 400;
        
        public Ball(JComponent comp) {
            this.comp = comp;
        }
        
        
        public void run(){
        Graphics gc = comp.getGraphics();
        
        gc.setColor(Color.BLACK);
        gc.drawOval(startx, starty, radius1, radius2);
   
        }
        }
    public static void main(String[] args){
        new Spaceball().setVisible(true);
    }
    
}

So jetzt geht es. Das einzige was ich gemacht habe ich habe die Objekterzeugung der Klasse Ball und die Methode start() in den Actionlistener von einem Button gepackt. Jetzt wird der Kreis gezeichnet sobald ich den Button drücke. Aber du hast recht Marco ich hab keine Ahnung warum es jetzt funktioniert . Warum Funktioniert es ohne Button nicht?

 

[Wildcard]

Es funktioniert sowieso nicht richtig, weil du immer noch getGraphics verwendet.
Lies dir doch bitte den Link durch.