Help Plz! AWT to Swing Conversion

Question

Hey everyone, I'm new to java and trying to convert this AWT program to Swing, I got no errors, the menu shows but they dont work !! Could somebody please help me? thanks alot!
Current Under-Construction Code: (no errors)
import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
class zipaint extends JFrame implements ActionListener,      ItemListener,
MouseListener, MouseMotionListener, WindowListener {
static final int WIDTH = 800;     // Sketch pad width
static final int HEIGHT = 600;     // Sketch pad height
int upperLeftX, upperLeftY;     // Rectangle upper left corner coords.
int width, height;          // Rectangle size
int x1, y1, x2, y2, x3, y3, x4, y4; // Point coords.
boolean fillFlag = false; //default filling option is empty
boolean eraseFlag = false; //default rubber is off
String drawColor = new String("black"); //default drawing colour is black
String drawShape = new String("brush"); //default drawing option is brush
Image background; //declear image for open option
private FileDialog selectFile = new FileDialog( this,
"Select background (.gif .jpg)",
FileDialog.LOAD );
// Help
String helpText = "Draw allows you to sketch different plane shapes over a " +
"predefined area.\n" + "A shape may be eother filled or in outline, " +
"and in one of eight different colours.\n\n" + "The position of the " +
"mouse on the screen is recorded in the bottom left-hand corner of the " +
"drawing area. The choice of colour and shape are disoplayed also in the " +
"left-habnd corner of the drawing area.\n\n" + "The size of a shape is " +
"determined by the mouse being dragged to the final position and " +
"released. The first click of the mouse will generate a reference dot on " +
"the screen. This dot will disapear after the mouse button is released.\n\n" +
"Both the square and the circle only use the distance measured along the " +
"horizontal axis when determining the size of a shape.\n\n" + "Upon " +
"selecting erase, press the mouse button, and move the mouse over the " +
"area to be erased. releasing the mouse button will deactivate erasure.\n\n" +
"To erase this text area choose clearpad from the TOOLS menu.\n\n";
// Components
JTextField color = new JTextField();
JTextField shape = new JTextField();
JTextField position = new JTextField();
CheckboxGroup fillOutline = new CheckboxGroup();
JTextArea info = new JTextArea(helpText,0,0/*,JTextArea.JSCROLLBARS_VERTICAL_ONLY*/);
JFrame about = new JFrame("About Zi Paint Shop");
// Menues
String[] fileNames = {"Open","Save","Save as","Exit"};
String[] colorNames = {"black","blue","cyan","gray","green","magenta","red","white","yellow"};
String[] shapeNames = {"brush","line","square","rectangle","circle","ellipse"};
String[] toolNames = {"erase","clearpad"};
String[] helpNames = {"Help","about"};
public zipaint(String heading) {
super(heading);
getContentPane().setBackground(Color.white);
getContentPane().setLayout(null);
/* Initialise components */
initialiseTextFields();
initializeMenuComponents();
initializeRadioButtons();
addWindowListener(this);
addMouseListener(this);
addMouseMotionListener(this);
/* Initialise Text Fields */
private void initialiseTextFields() {
// Add text field to show colour of figure
color.setLocation(5,490);
color.setSize(80,30);
color.setBackground(Color.white);
color.setText(drawColor);
getContentPane().add(color);
// Add text field to show shape of figure
shape.setLocation(5,525);
shape.setSize(80,30);
shape.setBackground(Color.white);
shape.setText(drawShape);
getContentPane().add(shape);
// Add text field to show position of mouse
position.setLocation(5,560);
position.setSize(80,30);
position.setBackground(Color.white);
getContentPane().add(position);
// Set up text field for help
info.setLocation(150,250);
info.setSize(500,100);
info.setBackground(Color.white);
info.setEditable(false);
private void initializeMenuComponents() {
// Create menu bar
JMenuBar bar = new JMenuBar();
// Add colurs menu
JMenu files = new JMenu("Files");
for(int index=0;index < fileNames.length;index++)
files.add(fileNames[index]);
bar.add(files);
files.addActionListener(this);
// Add colurs menu
JMenu colors = new JMenu("COLORS");
for(int index=0;index < colorNames.length;index++)
colors.add(colorNames[index]);
bar.add(colors);
colors.addActionListener(this);
// Add shapes menu
JMenu shapes = new JMenu("SHAPES");
for(int index=0;index < shapeNames.length;index++)
shapes.add(shapeNames[index]);
bar.add(shapes);
shapes.addActionListener(this);
// Add tools menu
JMenu tools = new JMenu("TOOLS");
for(int index=0;index < toolNames.length;index++)
tools.add(toolNames[index]);
bar.add(tools);
tools.addActionListener(this);
// Add help menu
JMenu help = new JMenu("HELP");
for(int index=0;index < helpNames.length;index++)
help.add(helpNames[index]);
bar.add(help);
help.addActionListener(this);
// Set up menu bar
setJMenuBar(bar);
/* Initilalise Radio Buttons */
private void initializeRadioButtons() {
// Define checkbox
Checkbox fill = new Checkbox("fill",fillOutline,false);
Checkbox outline = new Checkbox("outline",fillOutline,true);
// Fill buttom
fill.setLocation(5,455);
fill.setSize(80,30);
getContentPane().add(fill);
fill.addItemListener(this);
// Outline button
outline.setLocation(5,420);
outline.setSize(80,30);
getContentPane().add(outline);
outline.addItemListener(this);
/* Action performed. Detects which item has been selected from a menu */
public void actionPerformed(ActionEvent e) {
Graphics g = getGraphics();
String source = e.getActionCommand();
// Identify chosen colour if any
for (int index=0;index < colorNames.length;index++) {
if (source.equals(colorNames[index])) {
drawColor = colorNames[index];
color.setText(drawColor);
return;
// Identify chosen shape if any
for (int index=0;index < shapeNames.length;index++) {
if (source.equals(shapeNames[index])) {
drawShape = shapeNames[index];
shape.setText(drawShape);
return;
// Identify chosen tools if any
if (source.equals("erase")) {
eraseFlag= true;
return;
else {
if (source.equals("clearpad")) {
remove(info);
g.clearRect(0,0,800,600);
return;
if (source.equals("Open")) {
selectFile.setVisible( true );
if( selectFile.getFile() != null )
String fileLocation = selectFile.getDirectory() + selectFile.getFile();
background = Toolkit.getDefaultToolkit().getImage(fileLocation);
paint(getGraphics());
if (source.equals("Exit")) {
System.exit( 0 );
// Identify chosen help
if (source.equals("Help")) {
getContentPane().add(info);
return;
if (source.equals("about")) {
displayAboutWindow(about);
return;
public void paint(Graphics g)
super.paint(g);
if(background != null)
Graphics gc = getGraphics();
gc.drawImage( background, 0, 0, this.getWidth(), this.getHeight(), this);
/* Dispaly About Window: Shows iformation aboutb Draw programme in
separate window */
private void displayAboutWindow(JFrame about) {
about.setLocation(300,300);
about.setSize(350,160);
about.setBackground(Color.cyan);
about.setFont(new Font("Serif",Font.ITALIC,14));
about.setLayout(new FlowLayout(FlowLayout.LEFT));
about.add(new Label("Author: Zi Feng Yao"));
about.add(new Label("Title: Zi Paint Shop"));
about.add(new Label("Version: 1.0"));
about.setVisible(true);
about.addWindowListener(this);
// ----------------------- ITEM LISTENERS -------------------
/* Item state changed: detect radio button presses. */
public void itemStateChanged(ItemEvent event) {
if (event.getItem() == "fill") fillFlag=true;
else if (event.getItem() == "outline") fillFlag=false;
// ---------------------- MOUSE LISTENERS -------------------
/* Blank mouse listener methods */
public void mouseClicked(MouseEvent event) {}
public void mouseEntered(MouseEvent event) {}
public void mouseExited(MouseEvent event) {}
/* Mouse pressed: Get start coordinates */
public void mousePressed(MouseEvent event) {
// Cannot draw if erase flag switched on.
if (eraseFlag) return;
// Else set parameters to 0 and proceed
upperLeftX=0;
upperLeftY=0;
width=0;
height=0;
x1=event.getX();
y1=event.getY();
x3=event.getX();
y3=event.getY();
Graphics g = getGraphics();
displayMouseCoordinates(x1,y1);
public void mouseReleased(MouseEvent event) {
Graphics g = getGraphics();
// Get and display mouse coordinates
x2=event.getX();
y2=event.getY();
displayMouseCoordinates(x2,y2);
// If erase flag set to true reset to false
if (eraseFlag) {
eraseFlag = false;
return;
// Else draw shape
selectColor(g);
if (drawShape.equals("line")) g.drawLine(x1,y1,x2,y2);
else if (drawShape.equals("brush"));
else drawClosedShape(drawShape,g);
/* Display Mouse Coordinates */
private void displayMouseCoordinates(int x, int y) {
position.setText("[" + String.valueOf(x) + "," + String.valueOf(y) + "]");
/* Select colour */
private void selectColor(Graphics g) {
for (int index=0;index < colorNames.length;index++) {
if (drawColor.equals(colorNames[index])) {
switch(index) {
case 0: g.setColor(Color.black);break;
case 1: g.setColor(Color.blue);break;
case 2: g.setColor(Color.cyan);break;
case 3: g.setColor(Color.gray);break;
case 4: g.setColor(Color.green);break;
case 5: g.setColor(Color.magenta);break;
case 6: g.setColor(Color.red);break;
case 7: g.setColor(Color.white);break;
default: g.setColor(Color.yellow);
/* Draw closed shape */
private void drawClosedShape(String shape,Graphics g) {
// Calculate correct parameters for shape
upperLeftX = Math.min(x1,x2);
upperLeftY = Math.min(y1,y2);
width = Math.abs(x1-x2);
height = Math.abs(y1-y2);
// Draw appropriate shape
if (shape.equals("square")) {
if (fillFlag) g.fillRect(upperLeftX,upperLeftY,width,width);
else g.drawRect(upperLeftX,upperLeftY,width,width);
else {
if (shape.equals("rectangle")) {
if (fillFlag) g.fillRect(upperLeftX,upperLeftY,width,height);
else g.drawRect(upperLeftX,upperLeftY,width,height);
else {
if (shape.equals("circle")) {
if (fillFlag) g.fillOval(upperLeftX,upperLeftY,width,width);
else g.drawOval(upperLeftX,upperLeftY,width,width);
else {
if (fillFlag) g.fillOval(upperLeftX,upperLeftY,width,height);
else g.drawOval(upperLeftX,upperLeftY,width,height);
/* Mouse moved */
public void mouseMoved(MouseEvent event) {
displayMouseCoordinates(event.getX(),event.getY());
/* Mouse dragged */
public void mouseDragged(MouseEvent event) {
Graphics g = getGraphics();
x2=event.getX();
y2=event.getY();
x4=event.getX();
y4=event.getY();
displayMouseCoordinates(x1,y1);
if (eraseFlag) g.clearRect(x2,y2,10,10);
else {
selectColor(g);
if(drawShape.equals("brush")) g.drawLine(x3,y3,x4,y4);
x3 = x4;
y3 = y4;
// ---------------------- WINDOW LISTENERS -------------------
/* Blank methods for window listener */
public void windowClosed(WindowEvent event) {}
public void windowDeiconified(WindowEvent event) {}
public void windowIconified(WindowEvent event) {}
public void windowActivated(WindowEvent event) {}
public void windowDeactivated(WindowEvent event) {}
public void windowOpened(WindowEvent event) {}
/* Window Closing */
public void windowClosing(WindowEvent event) {
if (event.getWindow() == about) {
about.dispose();
return;
else System.exit(0);
Code End
Thanks again!
class ziapp {
/* Main method */
public static void main(String[] args) {
zipaint screen = new zipaint("Zi Paint Shop");
screen.setSize(zipaint.WIDTH,zipaint.HEIGHT);
screen.setVisible(true);

Answer 1

First of all use the [url http://forum.java.sun.com/features.jsp#Formatting]Formatting Tags when posting code to the forum. I'm sure you don't code with every line left justified so we don't want to read unformatted code either.
Hey everyone, I'm new to java and trying to convert this AWT program to SwingInstead of converting the whole program, start with something small, understand how it works and then convert a different part of the program. You can start by reading this section from the Swing tutorial on [url http://java.sun.com/docs/books/tutorial/uiswing/components/menu.html]How to Use Menus.
From a quick glance of your code it looks like you are not adding an ActionListener to your JMenuItems. You are adding them to the JMenu.
Other observations:
1) Don't mix AWT with Swing components. You are still using CheckBox
2) Don't override paint(). When using Swing you should override paintComponent();
3) Use LayoutManagers. Using a null layout does not allow for easy maintenance of the gui. Right now you are using 800 x 600 as your screen size. Most people are probably using at least 1024 x 768 screen sizes. When using LayoutManagers you would suggest component sizes by using setPreferredSize(), not setSize().