Go4Expert

Go4Expert (http://www.go4expert.com/)
-   Java (http://www.go4expert.com/forums/java/)
-   -   Complete Java Newbie (http://www.go4expert.com/forums/complete-java-newbie-t26960/)

Taharka 19Oct2011 03:42

Complete Java Newbie
 
No matter how many times I check it seems OK, but the compiler keeps screaming that there are errors, very frustrated any help will be greatly appreciated. The two errors are at the bottom of the program I will also be able to get a good nights sleep not thinking about the code and were I went wrong cheers..




Code:

// java tutorial 68 JRadioButton
// Java tutorial 67 final JRadioButton
// Java tutorial 69 Drop down list program

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

public class BT68Gui extends JFrame{
   
    private JComboBox box;
    private JLabel picture;
   
    private static String[] filename = {"b.png, x.png"};// and Array with two elements this Array stores the filename
    private Icon[] pics = {new ImageIcon(getClass().getResource(filename[0])), new ImageIcon(getClass().getResource(filename[1]))}; // This Array stores the pictures filename 0 is the first element of the Aray
   
    public BT68Gui(){
        super("The Title");
        setLayout(new FlowLayout());
       
           
        box = new JComboBox(filename);       
        box.addItemListener(
                new ItemListener(){
                    public void itemStateChanged(ItemEvent event){ od
                    if(event.getStateChange()==ItemEvent.SELECTED)
    picture.setIcon(pics[box.getSelectedIndex()]);/
                    }
              }
      };                      //compiler shows error
      add(box);      // compiler shows error
      picture = new JLabel(pics[0]);
      add(picture);
  }
}


ewaldhorn 25Oct2011 15:19

Re: Complete Java Newbie
 
Very interesting.

Might I suggest the following?

While you are learning Java, try to lign up your code logically, like the following example:

Code:


if (condition)
{
      for (something)
      {
      }
}


Notice how the brackets line up? In your code, you have brackets that EXCLUDE portions of the code "add(box)" from the body of the method they belong to. When you align the brackets, things start to look a lot better all of a sudden and it's easy to spot the problem areas.

Formatting code properly is going to help you a lot, especially as your programs grow in size and complexity.

I hope this helps.

virxen 11Nov2011 03:34

Re: Complete Java Newbie
 
see the errors in your code in order to compile

Code:

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

public class BT68Gui extends JFrame{
   
    private JComboBox box;
    private JLabel picture;
   
    private static String[] filename = {"b.png, x.png"};// and Array with two elements this Array stores the filename
    private Icon[] pics = {new ImageIcon(getClass().getResource(filename[0])), new ImageIcon(getClass().getResource(filename[1]))}; // This Array stores the pictures filename 0 is the first element of the Aray
   
    public BT68Gui(){
        super("The Title");
        setLayout(new FlowLayout());
       
           
        box = new JComboBox(filename);       
        box.addItemListener(
                new ItemListener(){
                    public void itemStateChanged(ItemEvent event){ //od <--error
                    if(event.getStateChange()==ItemEvent.SELECTED)
                        picture.setIcon(pics[box.getSelectedIndex()]);// error
                    }
              }
      );  //error                  //compiler shows error
      add(box);      // compiler shows error
      picture = new JLabel(pics[0]);
      add(picture);
  }
}



All times are GMT +5.5. The time now is 01:01.