Java: NullPointerException of a simple GUI program

Question

Hi I am novice Java learner. I was tring to implement a simple GUI program to change the color of a panel on clicking clicking of a button.

import java.awt.*;
import javax.swing.*;
import java.awt.event.*;

public class Button_lable implements ActionListener {
public JFrame frame;
//JPanel panel;
//JLabel label;

public static void main(String[] args) {

    Button_lable gui = new Button_lable();
    gui.go();
}//end of main
public void go(){
    //System.out.println("Entered Go()");
    JFrame frame = new JFrame();
    frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

    JButton b_frame = new JButton("Click to change the color");
    b_frame.addActionListener(this);
    MyDrawpanel d_panel = new MyDrawpanel();

    frame.getContentPane().add(BorderLayout.SOUTH,b_frame);
    frame.getContentPane().add(BorderLayout.CENTER ,d_panel);

    frame.setSize(300, 300);
    frame.setVisible(true);


}//end of go
public void actionPerformed(ActionEvent e) {
    frame.repaint();
}


}//end of Button_lable


class MyDrawpanel extends JPanel {
    public void paintComponent(Graphics g){
Graphics2D grph = (Graphics2D) g;
int red = (int)(Math.random()* 255);
int green = (int)(Math.random()* 255);
int blue = (int)(Math.random()* 255);
Color strt_clr = new Color(red,green,blue);

red = (int)(Math.random()* 255);
green = (int)(Math.random()* 255);
blue = (int)(Math.random()* 255);
Color end_clr = new Color(red,green,blue);

GradientPaint gradient = new GradientPaint(70,70,strt_clr,150,150,end_clr);  
grph.setPaint(gradient);
grph.fillOval(50,25, 150, 150);
  }
}

I get output window. But when i click on the button, i get the following exception.

 Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
at Button_lable.actionPerformed(Button_lable.java:34)
at javax.swing.AbstractButton.fireActionPerformed(Unknown Source)
at javax.swing.AbstractButton$Handler.actionPerformed(Unknown Source)
at javax.swing.DefaultButtonModel.fireActionPerformed(Unknown Source)
at javax.swing.DefaultButtonModel.setPressed(Unknown Source)
at javax.swing.plaf.basic.BasicButtonListener.mouseReleased(Unknown Source)
at java.awt.Component.processMouseEvent(Unknown Source)
at javax.swing.JComponent.processMouseEvent(Unknown Source)
at java.awt.Component.processEvent(Unknown Source)
at java.awt.Container.processEvent(Unknown Source)
at java.awt.Component.dispatchEventImpl(Unknown Source)
at java.awt.Container.dispatchEventImpl(Unknown Source)
at java.awt.Component.dispatchEvent(Unknown Source)
at java.awt.LightweightDispatcher.retargetMouseEvent(Unknown Source)
at java.awt.LightweightDispatcher.processMouseEvent(Unknown Source)
at java.awt.LightweightDispatcher.dispatchEvent(Unknown Source)
at java.awt.Container.dispatchEventImpl(Unknown Source)
at java.awt.Window.dispatchEventImpl(Unknown Source)
at java.awt.Component.dispatchEvent(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.security.ProtectionDomain$1.doIntersectionPrivilege(Unknown Source)
at java.awt.EventQueue$4.run(Unknown Source)
at java.awt.EventQueue$4.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)

Please advice.

Kind Regards.

Shouldn't the class be named Button_label?

stevenelberger
– stevenelberger

2013-03-31 07:17:25 +00:00
Commented Mar 31, 2013 at 7:17 — stevenelberger
– stevenelberger, Commented Mar 31, 2013 at 7:17

NPE · Accepted Answer · 2013-03-31 07:15:43Z

2

You never initialize this.frame, so it is null.

In go(), you create and initialize a different variable called frame:

JFrame frame = new JFrame();

You might want to remove the first JFrame:

frame = new JFrame();

answered Mar 31, 2013 at 7:15

NPE

503k114 gold badges970 silver badges1k bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

karmanaut Over a year ago

You're the champion for these exceptions, arent you NPE? ;)

karmanaut Over a year ago

Since no PM's around here, how long have you been in the area of programming? Your profile says 2 years in SO but not how long in this field.

user93353 · Accepted Answer · 2013-03-31 07:16:58Z

0

You have declared 2 variables called frame - one class variable and one inside go. Remove the declaration inside go & just initialize the class variable inside go

public void go(){
    //System.out.println("Entered Go()");
    JFrame frame = new JFrame();

Change the above to

public void go(){
    //System.out.println("Entered Go()");
    frame = new JFrame();

Because you are creating a new variable called frame in go, the class variable frame never gets initialized. It remains null and hence the NullPointerException

answered Mar 31, 2013 at 7:16

user93353

14.1k8 gold badges64 silver badges128 bronze badges

Comments

lokoko · Accepted Answer · 2013-03-31 07:17:16Z

0

frame is null. Just use a setter for frame. Something like :

void setFrame(JFrame theFrame) {
    this.frame = theFrame;
}

Also you have declared frame twice. Remove it from go();

answered Mar 31, 2013 at 7:17

lokoko

5,8035 gold badges39 silver badges68 bronze badges

Comments

Miguel Prz · Accepted Answer · 2013-03-31 07:17:32Z

0

Change this line:

JFrame frame = new JFrame();

by this:

frame = new JFrame();

answered Mar 31, 2013 at 7:17

Miguel Prz

13.8k33 silver badges42 bronze badges

Comments

PVR · Accepted Answer · 2013-03-31 07:26:58Z

public class Button_lable implements ActionListener {
public JFrame frame; // This is a class/global variable
//JPanel panel;
//JLabel label;

public static void main(String[] args) {

    Button_lable gui = new Button_lable();
    gui.go();
}//end of main
public void go(){
    //System.out.println("Entered Go()");
    JFrame frame = new JFrame();  // This is local variable.
    frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

    JButton b_frame = new JButton("Click to change the color");
    b_frame.addActionListener(this);
    MyDrawpanel d_panel = new MyDrawpanel();

    frame.getContentPane().add(BorderLayout.SOUTH,b_frame);
    frame.getContentPane().add(BorderLayout.CENTER ,d_panel);

    frame.setSize(300, 300);
    frame.setVisible(true);


}

You have defined class variable as frame. You are instantiating local variable frame. So without assigning it is very obvious that it will throw NPE. Because the scope of local variable resides in particular method only.

You just need to do following.

public class Button_lable implements ActionListener {
public JFrame frame;
//JPanel panel;
//JLabel label;

public static void main(String[] args) {

    Button_lable gui = new Button_lable();
    gui.go();
}//end of main
public void go(){
    //System.out.println("Entered Go()");
    frame = new JFrame(); // here, the class variable is instantiated. So it wont give NPE.
    frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

    JButton b_frame = new JButton("Click to change the color");
    b_frame.addActionListener(this);
    MyDrawpanel d_panel = new MyDrawpanel();

    frame.getContentPane().add(BorderLayout.SOUTH,b_frame);
    frame.getContentPane().add(BorderLayout.CENTER ,d_panel);

    frame.setSize(300, 300);
    frame.setVisible(true);


}

kenju · Accepted Answer · 2015-09-03 07:30:42Z

0

Let me explain the reason of Exception.

the first public JFrame frame; you declare under first braces has global scope. And the second JFrame frame = new JFrame(); you initialize creates a new instance of it. the frame you declare in go() method has local scope and it initialize it not the global one. That's why global scope instance of frame is uninitialized. it is causing NullPointerException.

edited Sep 3, 2015 at 7:30

kenju

5,9741 gold badge44 silver badges41 bronze badges

answered Jul 23, 2015 at 7:21

Zar E Ahmer

34.5k20 gold badges244 silver badges306 bronze badges

Collectives™ on Stack Overflow

Java: NullPointerException of a simple GUI program

6 Answers 6

2 Comments

Comments

Comments

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

6 Answers 6

2 Comments

Comments

Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related