JFrame issues

K

K

Here is my script does anyone know why this isn't working

import javax.swing.JFrame;
import javax.swing.ImageIcon;
import javax.swing.JLabel;
import javax.swing.Continer;

class Jframe {

public static void main(String[] args) {
JFrame frame = new JFrame();
ImageIcon icon = new ImageIcon(puffin.jpg);
JLabel label = new JLable (icon);
Container contentPane = frame.getContentPane();

contentPane.add(label);
frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
frame.pack();
frame.setVisible(true);

}

}
 
L

Lew

K said:
Here is my script

It's not a script.
does anyone know why this isn't working

What is "working" for this program?

What happens instead?

Copy and paste your error messages so we don't have to
tire out our crystal balls.

You should place your program in a package, rather than the
"default" (empty) package.
import javax.swing.JFrame;
import javax.swing.ImageIcon;
import javax.swing.JLabel;
import javax.swing.Continer;
^
You should get a compilation error on this import.
class Jframe {

Get in the habit of declaring classes 'public', and not naming them so
similarly to standard API types.
public static void main(String[] args) {
JFrame frame = new JFrame();

You failed to put the GUI work on the Event Dispatch Thread (EDT).
ImageIcon icon = new ImageIcon(puffin.jpg);
^
This line should raise a compilation error.
JLabel label = new JLable (icon);
^
This line should raise a compilation error.
Container contentPane = frame.getContentPane();
contentPane.add(label);
frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
frame.pack();
frame.setVisible(true);
}
}

You are getting compilation errors that tell you exactly where you are being
careless.

Use 'invokeLater()' to do GUI things (like defining your frame and all that).
Doing GUI off the EDT and doing non-GUI on the EDT are the twin sins of Swing.

Repent your sins.
 
E

Eric Sosman

Here is my script does anyone know why this isn't working

import javax.swing.JFrame;
import javax.swing.ImageIcon;
import javax.swing.JLabel;
import javax.swing.Continer;

Because there's no Continer class in javax.swing.
class Jframe {

public static void main(String[] args) {
JFrame frame = new JFrame();

Because Swing components should be constructed and
manipulated on the Event Dispatch Thread.
ImageIcon icon = new ImageIcon(puffin.jpg);

Because puffin.jpg is a syntax error.
JLabel label = new JLable (icon);

Because no JLable class has been declared.
Container contentPane = frame.getContentPane();

Because no Container class has been declared.
 
A

Arne Vajhøj

Here is my script does anyone know why this isn't working

import javax.swing.JFrame;
import javax.swing.ImageIcon;
import javax.swing.JLabel;
import javax.swing.Continer;

class Jframe {

public static void main(String[] args) {
JFrame frame = new JFrame();
ImageIcon icon = new ImageIcon(puffin.jpg);
JLabel label = new JLable (icon);
Container contentPane = frame.getContentPane();

contentPane.add(label);
frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
frame.pack();
frame.setVisible(true);

}

}

You need to learn to look at the error messages and fix
bugs following the information in those.

Then you can help yourself in the future.

Arne
 

Ask a Question

Want to reply to this thread or ask your own question?

You'll need to choose a username for the site, which only take a couple of moments. After that, you can post your question and our members will help you out.

Ask a Question

Staff online

Members online

Forum statistics

Threads
473,767
Messages
2,569,571
Members
45,045
Latest member
DRCM

Latest Threads

Top