JPanel & getGraphics() : NullPointerException

Version imprimable

Bonsoir,

lors d'un appel à un getGraphics() sur un JPanel qui vient d'être instancié, je reçois une variable null. Comment est-ce possible, et surtout, comment y remédier ?

Voici mon code :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 public class GameScreen extends JPanel { public GameScreen(Image img) { super(); setLayout(null); setPreferredSize(SCREEN_SIZE); setBackground(Color.BLACK); Graphics g = this.getGraphics(); g.drawImage(img, 0, 0, Color.BLACK, this); } }
Un grand merci à ceux qui pourront m'aider à résoudre ce souci ;)

c'est très simple:

lorsque tu crée ton JPanel, il est réservé en mémoire mais pas encore ajouté à la fenêtre. Tu ne peux donc pas dessiner dedans !

Il faut attendre de l'afficher réellement avant de pouvoir dessiner !
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 public class MaClasse extends JFrame public MaClasse(){ /* un peu de code ...*/ show(); JPanel pan = new JPanel(); pan.getGraphics.drawString("blabla"); // <- NullPointerException this.add(pan); pan.getGraphics.drawString("blabla"); // <- pas d'exception }
essaie en faisant comme ça, je pense que ça va marcher

21/05/2004, 00h39
dingoth
Ca a l'air de marcher, mais seulement si je mets mes méthodes repaint et update en commentaire ... (dans la classe étendant le JPanel)
Code:

1 2 3 4 5 6 7 8 9 public void repaint() { paint(getGraphics()); } public void update(Graphics g) { paint(g); }
Est-ce normal ? Je suis en train de développer un petit jeu, et si je n'utilise pas ces méthodes, j'ai des problèmes de scintillement ...

que se passe-t-il lorsque tu ne les mets pas en commentaires ?

si c'est de nouveau un NullPointerException, c'est sans doute parce que tu appelles ces méthodes avant d'afficher ton JPanel (par exemple dans le constructeur de ta classe qui étend JPanel).

Tu aurais meilleur temps de l'appeler dans le constructeur de la fenêtre
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 class MonJPanel extends JPanel{ public MonJPanel (){ setDoubleBuffered(true); // <- pr limiter le scintillement } public void paintComponent(Graphics g){ super.paintComponent(g); // à faire lors de la MàJ du panel } } class MaJFrame extends JFrame{ public MaJFrame(){ MonJPanel pan = new MonJPanel(); this.add(pan); pan.paintComponent(pan.getGraphics()); pan.update(); ........ } }

Je n'y comprends plus rien ... 8O

Voici mon code. J'ai tout mis en commentaire sauf ce qui est visible ici :

MonPanel.java

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
public class MonPanel extends JPanel
{
    public MaFrame()
    {
        super();
    }
 
    synchronized protected void paintComponent(Graphics g)
    {
        g.drawImage(
            Toolkit.getDefaultToolkit().getImage(".data/img/image.png"),
            0,
            0,
            Color.BLACK,
            this);
    }
 
    public void repaint()
    {
        paintComponent(getGraphics());
    }
 
    public void update(Graphics g)
    {
        paintComponent(g);
    }
 
}

MaFrame.java

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
public class MaFrame extends JFrame implements GameConstants
{
    private JPanel mainPanel;
 
    public MaFrame()
    {
        super();
        this.setSize(SCREEN_SIZE);
        this.addWindowListener(new WindowAdapter()
        {
            public void windowClosing(WindowEvent we)
            {
                System.exit(1);
            }
        });
 
        this.show();
        mainPanel = new MonPanel();
        this.add(mainPanel);
        mainPanel.paintComponents(mainPanel.getGraphics());
    }
}

Je fais ça de manière très simple pourtant ... Rien de bien complexe ...

Le panel doit être ajouté après l'appel à show() ? Je dois le redesssiner en permanence pourtant, d'où les surcharges des méthodes paintComponent(), repaint() et update() ...

Apparemment, en faisant super() dans le constructeur de MonPanel(), il y a un appel à repaint() (vu la stacktrace). Normal ?

Je rajoute quand même le stacktrace ici, histoire que vous y voyiez plus clair ...

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
java.lang.NullPointerException
	at pack.gui.MonPanel.paintComponent(MonPanel.java:80)
	at pack.gui.MonPanel.repaint(MonPanel.java:96)
	at javax.swing.JComponent.setFont(Unknown Source)
	at javax.swing.LookAndFeel.installColorsAndFont(Unknown Source)
	at javax.swing.plaf.basic.BasicPanelUI.installDefaults(Unknown Source)
	at javax.swing.plaf.basic.BasicPanelUI.installUI(Unknown Source)
	at javax.swing.JComponent.setUI(Unknown Source)
	at javax.swing.JPanel.setUI(Unknown Source)
	at javax.swing.JPanel.updateUI(Unknown Source)
	at javax.swing.JPanel.<init>(Unknown Source)
	at javax.swing.JPanel.<init>(Unknown Source)
	at javax.swing.JPanel.<init>(Unknown Source)
	at pack.gui.MaFrame.<init>(MonPanel.java:48)
	at pack.gui.MonPanel.<init>(MaFrame.java:56)
	at pack.Application.main(Application.java:33)
Exception in thread "main"

21/05/2004, 11h28
Pill_S

oui, c'est mieux d'ajouter le panel après le show(), pour des raisons de NullPointerException.

Concernant ton code, il me semble tout à fait correct. J'ai juste un peu de peine à comprendre à quoi te servent les redéfinitions de repaint() et update()... puisqu'elles appelent toutes les deux paintComponent() ! (autant appeler directement paintComponent() )

Mais sinon, c'est exactement ce que je te disais plus haut :wink:

[EDIT]
essaie d'enlever le super() du constructeur de ton JPanel....
[/EDIT]
21/05/2004, 14h04
dingoth

J'ai repris ton squelette, et ça marche nickel. J'ai beau comparer mon code et le tien, je ne vois rien de dissemblable ...

Enfin, le principal est que ça marche maintenant ;)

merci beaucoup
21/05/2004, 15h56
Pill_S

De rien !

A une prochaine :P