"Zeichnen" funktioniert nicht...

[Zettelkasten]

Hallo!

Ich habe ein Problem mit der Klasse

Graphics

.
Hier mein Code:
[JAVA=4]import java.awt.*;

import javax.swing.*;


public class game extends JFrame
{
public void drawImage()
{
Graphics graphics = getGraphics();
graphics.setColor(new Color(100,20,70));
graphics.fillRect(20,40,40,60);
graphics.drawOval(30,50,60,80);
graphics.drawLine(2,40,2,20);
graphics.drawLine(70,30,50,80);
}

public game()
{
super("Malen!");
JPanel panel = new JPanel();
panel.setLayout(new FlowLayout());
setContentPane(panel);
drawImage();
}

public static void main(String[] args)
{
game panel = new game();
panel.setSize (450,290);
panel.setDefaultCloseOperation (JFrame.EXIT_ON_CLOSE);
panel.setVisible(true);
}
}
[/code]
Ich will, dass diese Linien angezeigt werden.
Doch wenn ich das mit Eclipse ausführen will, kommt folgende Fehlermeldung:

Exception in thread "main" java.lang.NullPointerException
	at was_ist_die_richtige_farbe.game.drawImage(game.java:14)
	at was_ist_die_richtige_farbe.game.<init>(game.java:27)
	at was_ist_die_richtige_farbe.game.main(game.java:32)

Ich weiß nicht, aber laut meiner Sicht ist an der Zeile alles richtig:
[JAVA=14]graphics.setColor(new Color(100,20,70));[/code]

Ich hoffe jemand kennt die Lösung!

Danke schon mal im Voraus,
Zettelkasten.

 

[eRaaaa]

Du rufst das drawImage in deinem Konstruktor auf, zu dem Zeitpunkt ist der Frame noch nicht sichtbar, daher wirst du wohl ein null erhalten bei getGraphics. Das passiert eben wenn man es nicht so macht wie man machen sollte (klugscheiss )

getGraphics aufrufen ist böse. Überschreibe paintComponent von dem von dir erstelltem Panel, dort bekommst du ein Graphics-Objekt übergeben mit dem du zeichnen kannst.
Siehe: Galileo Computing :: Java ist auch eine Insel (8. Auflage) – 17 Grafikprogrammierung oder auch Malen in AWT und Swing - Byte-Welt Wiki

z.B.:

import java.awt.*;
 
 
public class Game extends JFrame  //Klassen groß !
{
    public void drawImage( Graphics graphics )
    {
        graphics.setColor(new Color(100,20,70));
        graphics.fillRect(20,40,40,60);
        graphics.drawOval(30,50,60,80);
        graphics.drawLine(2,40,2,20);
        graphics.drawLine(70,30,50,80);
    }
 
    public Game()
    {
        super("Malen!");
        JPanel panel = new JPanel(){
        	@Override
        	protected void paintComponent(Graphics g) {
        		super.paintComponent(g);
        		drawImage(g);
        	}
        };
        panel.setLayout(new FlowLayout()); //hat eh schon ein flowlayout :D
        setContentPane(panel);
    }
    
    public static void main(String[] args)
    {
        Game panel = new Game();
        panel.setSize (450,290);
        panel.setDefaultCloseOperation (JFrame.EXIT_ON_CLOSE);
        panel.setVisible(true);
    }
}

 

[Zettelkasten]

Ich werde mir die zwei Seiten mal genauer ansehen. Dein Code-Beispiel funktioniert einwandfrei

Danke!