Wieder mal ne NullPointer Exception.

[Revenant]

Hi,

zum 2. mal jetzt hab ich dieses Problem. Kann den Fehler einfach nicht finden.. kompilieren geht, aber ausführen nicht. Der Fehler tritt in Zeile 117 auf in der paintComponent Methode des Panels, in das ich zeichnen möchte.

Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
at CJPanel.paintComponent(CGrafik.java:117)
.......

//dieses Programm zeigt unter Verwendung der Panel Komponente das Zeichnen von
//verschiedenen Figuren mit Hilfe von Optionsfeldern


import javax.swing.*;    //Swing-Komponenten
import java.awt.*;       //der Layout-Manager
import java.awt.event.*; //die Ereignisbehandlung


//die Hauptklasse
public class CGrafik {
    public static void main(String[] args){
	CFenster fenster = new CFenster("Zeichnen");
	fenster.pack();
	fenster.setSize(450,350);
	fenster.setResizable(true);
	fenster.setVisible(true);
    }
}


//Die Klasse des Fensters
class CFenster extends JFrame {
    ButtonGroup knopfgruppe;
    
    //der Konstruktor
    CFenster (String titel) {
	//ruft zunächst den Konstruktor der Basisklasse auf
	super(titel);
	//ein Layout-Manager für das Fenster wird angelegt
	setLayout(new FlowLayout());
	setDefaultCloseOperation(WindowConstants.EXIT_ON_CLOSE);
	
	//ein neues Panel für die Zeichenfläche
	CJPanel panel = new CJPanel(knopfgruppe);
	add(panel);
    
	//ein Panel für die RadioButtons
	JPanel knopfpanel = new JPanel();
	add(knopfpanel);

	//Die Buttons werden erstellt
	JRadioButton knopf1 = new JRadioButton("Kreis",false);
	JRadioButton knopf2 = new JRadioButton("Scheibe",false);
	JRadioButton knopf3 = new JRadioButton("Rechteck",false);

	//Befehlsnamen für Ereignisbehandlung festelegen
	//der Einfachheit halber = dem Buttonnamen
	knopf1.setActionCommand("Kreis");
	knopf2.setActionCommand("Scheibe");
	knopf3.setActionCommand("Rechteck");

	//erstellen einer ButtonGroup um mit den RadioButtons umgehen zu können
	//liefert Methoden die die selektierten Buttons zurückgeben, bewirkt 
	//ausserdem, dass nur 1 Knopf der Gruppe selektiert werden kann
	knopfgruppe = new ButtonGroup();
	
	//dem Panel hinzugefügt
	knopfpanel.add(knopf1);
	knopfpanel.add(knopf2);
	knopfpanel.add(knopf3);

	//die Buttons der ButtonGruppe hinzufügen
	knopfgruppe.add(knopf1);
	knopfgruppe.add(knopf2);
	knopfgruppe.add(knopf3);
	
	knopfpanel.setLayout(new GridLayout(3,1,20,20));
	knopfpanel.setBorder(BorderFactory.createEtchedBorder(1));
	knopfpanel.setBorder(BorderFactory.createTitledBorder("Form"));
    }

}


class CJPanel extends JPanel{
    int mx, my; //Mauskoordinaten
    ButtonGroup knopfgruppe;

    CJPanel (ButtonGroup kgruppe) {
	knopfgruppe = kgruppe;
	//dem Panel wird ein Listener zur Ereignisbehandlung zugewiesen
	addMouseListener(new CMausListener());
    }
    
    public Dimension getMinimumSize() {
	return new Dimension(300,300);
    }
    
    public Dimension getPreferredSize() {
	return getMinimumSize();
    }
    
    //der MausListener zur Ereignisbehandlung des Panels
    //wird natürlich im Panel an sich definiert, da er ja auch zu diesem gehört
    class CMausListener extends MouseAdapter {
	public void mousePressed(MouseEvent e) {
	    //die aktuelle Position der Maus merken
	    mx = e.getX();
	    my = e.getY();
	}
    }
    
    //in dieser Methode wird gezeichnet!
    public void paintComponent(Graphics g) {
	//Aufruf des Super-Konstruktors bei paintComponent ein MUSS
	super.paintComponent(g);
	//Festlegen der Hintergrundfarbe
	setBackground(Color.black);

	String befehlsname; //String zum Überprüfen des ActionCommands
	ButtonModel aktuell = null; //ButtonModel des ausgewählten Knopfes
	
	//welcher Knopf ist ausgewählt? -> gibt das ButtonModel
	aktuell = knopfgruppe.getSelection(); // HIER TRITT DER FEHLER AUF

	//durch das ButtonModel lässt sich der Befehlsname des Knopfes erfragen
	//und somit seine Identität feststellen
	befehlsname = aktuell.getActionCommand();
	
	//entsprechend reagieren
	if (aktuell == null)
	    return;

	//grüne Farbe zum zeichnen
	g.setColor(Color.green);

	if(befehlsname.equals("Kreis"))
	    g.drawOval(mx,my,40,40);

	if(befehlsname.equals("Scheibe"))
	    g.fillOval(mx,my,40,40);

	if(befehlsname.equals("Rechteck"))
	    g.drawRect(mx,my,40,40);
    }
}

Hoffe es kann mir jemand helfen...

 

[muckelzwerg]

Ich kann keine Stelle finden, an der eine ButtonGroup erzeugt wird.
So wie ich das sehe reichst Du die ganze Zeit ein leeres "knoppfgruppe" durch die Gegend.

-- -- muckelzwerg

 

[Murray]

Offenbar wird paintComponent aufgerufen, bevor Du knopfgruppe initialisiert hast. Das kann man aber leicht abfangen:

public void paintComponent(Graphics g) {

   super.paintComponent(g);

   //Festlegen der Hintergrundfarbe
   setBackground(Color.black);

   String befehlsname; //String zum Überprüfen des ActionCommands
   ButtonModel aktuell; //ButtonModel des ausgewählten Knopfes
   
   //welcher Knopf ist ausgewählt? -> gibt das ButtonModel
   if ( kopfgruppe != null) {
      aktuell = knopfgruppe.getSelection();
   }

   if ( aktuell == null) {
       return;
   }

   //durch das ButtonModel lässt sich der Befehlsname des Knopfes erfragen
   //und somit seine Identität feststellen
   befehlsname = aktuell.getActionCommand();
   
   //entsprechend reagieren
  
   //grüne Farbe zum zeichnen
   g.setColor(Color.green);

   if ( "Kreis".equals( befehlsname) ) {
       g.drawOval(mx,my,40,40);
   } else if ( "Scheibe".equals( befehlsname) ) {
       g.fillOval(mx,my,40,40);
   } else if ( "Rechteck".equals( befehlsname) ) {
       g.drawRect(mx,my,40,40);
   }
}

 

[Roar]

knopfgruppe ist in der klasse aber immer null, weil null im konstruktor übergeben wird.

 

[Murray]

knopfgruppe ist in der klasse aber immer null, weil null im konstruktor übergeben wird.

Stimmt natürlich. Also die Zeile

knopfgruppe = new ButtonGroup();

weiter nach oben (vor den Aufruf des CJPanel-Konstruktors) schieben.

 

[Revenant]

Ich kann keine Stelle finden, an der eine ButtonGroup erzeugt wird.
So wie ich das sehe reichst Du die ganze Zeit ein leeres "knoppfgruppe" durch die Gegend.

In Zeile 57 wurde eine Instanz der Klasse ButtonGroup erzeugt.

Roar hat folgendes geschrieben::
knopfgruppe ist in der klasse aber immer null, weil null im konstruktor übergeben wird.


Stimmt natürlich. Also die Zeile
Code:

1


knopfgruppe = new ButtonGroup();

weiter nach oben (vor den Aufruf des CJPanel-Konstruktors) schieben.

Jo das wars. Es is dann noch so ne NullPointerException aufgetreten, aber im Endeffekt lag auch die wieder daran, dass ich da mit was gearbeitet habe, "das noch nicht initialisiert war". Der Vollständigkeit halber:

die Zeile

befehlsname = aktuell.getActionCommand()

in der paintComponent Methode muss nach der If-Anweisung stehen, da ja am Anfang kein RadioButton selektiert ist und aktuelle = null ist.

Vielen Dank für eure Hilfe, bin jetzt wieder um ne Erfahrung reicher :wink:


Gruß
Revenant

 

[Murray]

die Zeile

befehlsname = aktuell.getActionCommand()

in der paintComponent Methode muss nach der If-Anweisung stehen, da ja am Anfang kein RadioButton selektiert ist und aktuelle = null ist.

Hatte ich in meinem Post bereits umgestellt :wink:

Außerdem habe ich die Fallunterscheidung noch umgestellt: erstens kann man die Vergleiche umdrehen ( Stringliteral.equals( Variable) anstelle von Variable.equals( Stringliteral) ist sicherer, weil keine NullpointerException auftreten kann); zweitens sollten man in solchen Fällen mit else if arbeiten, um unnötige Vergleiche zu vermeiden (wenn der erste Vergleich bereits stimmt, dann braucht man alle anderen nicht mehr zu prüfen).

 

[muckelzwerg]

In Zeile 57 wurde eine Instanz der Klasse ButtonGroup erzeugt.

Erwischt. So weit hatte ich gar nicht mehr gesucht, nachdem das CJPanel erzeugt wurde.

-- -- muckelzwerg

 

[Revenant]

die Zeile

befehlsname = aktuell.getActionCommand()

in der paintComponent Methode muss nach der If-Anweisung stehen, da ja am Anfang kein RadioButton selektiert ist und aktuelle = null ist.

Hatte ich in meinem Post bereits umgestellt :wink:

Außerdem habe ich die Fallunterscheidung noch umgestellt: erstens kann man die Vergleiche umdrehen ( Stringliteral.equals( Variable) anstelle von Variable.equals( Stringliteral) ist sicherer, weil keine NullpointerException auftreten kann); zweitens sollten man in solchen Fällen mit else if arbeiten, um unnötige Vergleiche zu vermeiden (wenn der erste Vergleich bereits stimmt, dann braucht man alle anderen nicht mehr zu prüfen).

du verdammter gosu ! da sieht man ja mal echt was du aufm kasten hast - genial!