Thread: random card display problem

hey guys im trying to get my code to display 3 random cards from my images folder ( the cards are labeled 1.png to 54.png). Im confused how i would get my program to read and display my cards.

import javax.swing.*;
import java.awt.*;
import java.util.*;
public class DisplayCard extends JFrame {
	String[] images;
	for (int i = 1; i < 0 && i >=54;i++)
	{
	images[i] = i+".png";
	}
 
	Random rn = new Random();
	int maximum = 54;
	int minimum = 1;
	int range = maximum - minimum + 1;
	int randomNum =  rn.nextInt(range) + minimum;
 
  protected ImageIcon usIcon = new ImageIcon(images[randomNum]);
  protected ImageIcon myIcon = new ImageIcon(images[randomNum]);
  protected ImageIcon frIcon = new ImageIcon(images[randomNum]);
  protected ImageIcon ukIcon = new ImageIcon(images[randomNum]);
 
 
  public DisplayCard() {
    setLayout(new GridLayout(1, 4, 5, 5));
    add(new JLabel(usIcon));
    add(new JLabel(myIcon));
    add(new JButton(frIcon));
    add(new JButton(ukIcon));
  }
 
  /** Main method */
  public static void main(String[] args) {
	DisplayCard frame = new DisplayCard();
    frame.setTitle("TestImageIcon");
    frame.setSize(500, 125);
    frame.setLocationRelativeTo(null); // Center the frame
    frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    frame.setVisible(true);
  }
}

im confused it is giving me an error at ukicon = new imageIcon... basically its telling me to put a } after my semicolon.
here is the error message
Exception in thread "main" java.lang.Error: Unresolved compilation problem:

at DisplayCard.main(DisplayCard.java:32)
Appreciate the help guys.

Originally Posted by Norm

There may be statements in the code that should be inside of a method.

A full error message from the javac compiler could better explain what the problem is.

I revised the code.Now i have everything finished but I can't get my actionlistener to randomize my cards again when I click my jbutton.

import javax.swing.*;
import java.awt.*;
import java.awt.event.*;
import java.util.*;
public class DisplayCard extends JFrame {
	//generate random numbers
Random generator = new Random();
int i = 54 - generator.nextInt(54);
Random generator2 = new Random();
int p = 54 - generator.nextInt(54);
Random generator3 = new Random();
int t = 54 - generator.nextInt(54);
//defines images and button
protected ImageIcon card1 = new ImageIcon("C:/Users/bearcat/Desktop/computer science/book/image/card/"+i+".png");
protected ImageIcon card2 = new ImageIcon("C:/Users/bearcat/Desktop/computer science/book/image/card/"+p+".png");
protected ImageIcon card3 = new ImageIcon("C:/Users/bearcat/Desktop/computer science/book/image/card/"+t+".png");
JButton d= new JButton("Display Cards");
 
//displays 3 cards and button
  public DisplayCard() {
    setLayout(new GridLayout(1, 4, 5, 5));
    add(new JLabel(card1));
    add(new JLabel(card2));
    add(new JLabel(card3));
    add(d);
  }
 
public  void Button() {
	 setSize(200,200);
	   JButton startButton = new JButton("Start");
 
	   add(startButton);
	   startButton.addActionListener(new startButtonListener());
	   setDefaultCloseOperation(EXIT_ON_CLOSE);
	   setVisible(true);
}
private class startButtonListener implements ActionListener
{
	public void actionPerformed(ActionEvent e)
	{
 
	}
 
}
 
  /** Main method */
  public static void main(String[] args) {
	//calls method sets frame size, title
	DisplayCard frame = new DisplayCard();
    frame.setTitle("12-9");
    frame.setSize(500, 125);
    frame.setLocationRelativeTo(null); // Center the frame
    //close and show frame
    frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    frame.setVisible(true);
 
  }
}