Solved

NullPointerException

Posted on 1998-11-18
2
186 Views
Last Modified: 2010-03-30
This seems like a really dumb question
(but ill as it anyway)
The easiest way to describe it would just be to give you the
code:

import java.awt.*;
import java.awt.event.*;

public class WandGui extends Frame  {
     
     String wands[];
     Checkbox checkbox[];
     
     WandGui(String filename)  {

          super("Wand Selection");
         
          wands= getWandFile(filename);
         
         
          this.setLayout(new BorderLayout());
          this.setBackground(Color.lightGray);
          Font labelFont= new Font("SansSerif",Font.PLAIN,14);
         
          Panel p1= new Panel();
          p1.setLayout(new GridLayout(0,1,10,10));
         
          CheckboxGroup checkbox_group= new CheckboxGroup();
          Checkbox[] checkbox= new Checkbox[wands.length];            
         
          Label label = new Label("Choose a wand from the List below:");
          label.setFont(labelFont);
          p1.add(label);
         
          for (int i=0;i<wands.length;i++)  {
               
               checkbox[i]= new Checkbox("Wand " + wands[i],checkbox_group,false);
               checkbox[i].setFont(labelFont);
               p1.add(checkbox[i]);
          }
         
          Panel p2= new Panel();
          p2.setLayout(new FlowLayout(FlowLayout.CENTER,10,10));
         
          ButtonListener buttonlistener= new ButtonListener();
         
          Button okbutton= new Button("OK");
          okbutton.addActionListener(buttonlistener);
          okbutton.setActionCommand("ok");
          p2.add(okbutton);
          Button cancelbutton= new Button("Cancel");
          cancelbutton.addActionListener(buttonlistener);
          cancelbutton.setActionCommand("cancel");
          p2.add(cancelbutton);
         
         
          this.add(BorderLayout.WEST,new Label("  "));
          this.add(BorderLayout.EAST,new Label("  "));
          this.add(BorderLayout.CENTER,p1);
          this.add(BorderLayout.SOUTH,p2);
         
         
     }
     
     
     public static void main(String args[])  {
         
          WandGui wg= new WandGui("file.txt");
          wg.pack();
          wg.setVisible(true);
     }
     
     
     private String[] getWandFile(String filename)  {
         
          String wands[]={"AAA","BBB","CCC","DDD","EEE","FFF"};
         
          return wands;
     }
     
     class ButtonListener implements ActionListener  {
          public void actionPerformed(ActionEvent e)  {
               if (e.getActionCommand().equals("ok"))  {  
                    System.out.println(checkbox);    //comes up as null here
                    for (int i=0;i<checkbox.length;i++)  {
                         System.out.println("STATE: " + checkbox[i].getState());
                         if (checkbox[i].getState())
                              System.out.println("Picking " + wands[i]);
                    }
               }
               System.exit(0);
          }
     }
}

Notice in the event code I try to access the checkboxes that I initialized in the constructor...
why doesnt it work? and what do I need to do differently?

0
Comment
Question by:mbunkows
2 Comments
 
LVL 5

Accepted Solution

by:
msmolyak earned 50 total points
ID: 1227717
Look at the first and the last lines of the code below. The first line declares a instance variable while the last declares the local to constructor variable with the same name. The former one is never initialize. Remove the initializer from the last line.

Checkbox checkbox[];
         
         WandGui(String filename)  {

              super("Wand Selection");
               
              wands= getWandFile(filename);
               
               
              this.setLayout(new BorderLayout());
              this.setBackground(Color.lightGray);
              Font labelFont= new Font("SansSerif",Font.PLAIN,14);
               
              Panel p1= new Panel();
              p1.setLayout(new GridLayout(0,1,10,10));
               
              CheckboxGroup checkbox_group= new CheckboxGroup();
              Checkbox[] checkbox= new Checkbox[wands.length];      
0
 

Author Comment

by:mbunkows
ID: 1227718
ARRGGGHHHHH
someone slap me  :)

Thanks
0

Featured Post

How to run any project with ease

Manage projects of all sizes how you want. Great for personal to-do lists, project milestones, team priorities and launch plans.
- Combine task lists, docs, spreadsheets, and chat in one
- View and edit from mobile/offline
- Cut down on emails

Join & Write a Comment

Suggested Solutions

By the end of 1980s, object oriented programming using languages like C++, Simula69 and ObjectPascal gained momentum. It looked like programmers finally found the perfect language. C++ successfully combined the object oriented principles of Simula w…
Introduction This article is the last of three articles that explain why and how the Experts Exchange QA Team does test automation for our web site. This article covers our test design approach and then goes through a simple test case example, how …
Viewers will learn about arithmetic and Boolean expressions in Java and the logical operators used to create Boolean expressions. We will cover the symbols used for arithmetic expressions and define each logical operator and how to use them in Boole…
Viewers will learn about basic arrays, how to declare them, and how to use them. Introduction and definition: Declare an array and cover the syntax of declaring them: Initialize every index in the created array: Example/Features of a basic arr…

743 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question

Need Help in Real-Time?

Connect with top rated Experts

14 Experts available now in Live!

Get 1:1 Help Now