PLease help. The error is: exception in thread "main" java.lang.OutOfMemoryError:Java

[Rasenganboy16]

heap space.

My codes:

import javax.swing.*;
import java.awt.*;
import java.awt.event.*;
public class board implements ActionListener{
	JButton y[]=new JButton[1000000];
	int xa, ya, i, l;
	JFrame Dave;
	
	public static void main(String[] args){
		board chuy = new board();
		chuy.setFrame();
	}
	
	public void setFrame(){
		xa=0;
		ya=0;
		l=0;
		i=1;
		Dave = new JFrame("Board");
		Dave.setLayout(new GraphPaperLayout(new Dimension(1000,1000)));
		while(l<9999){
			y[i]=new JButton("");
			Dave.add(y[i], new Rectangle(xa,ya,1,1));
			if(xa<=1000){
				if(xa==1000){
					xa=0;
					ya++;
				}
				else xa++;  
			}
		}
			i++;
		
		Dave.setVisible(true);
		Dave.setSize(360,400);
		Dave.setLocation(100,100);
		Dave.setResizable(false);
		Dave.setDefaultCloseOperation(Dave.DISPOSE_ON_CLOSE);
		}
		
		public void actionPerformed(ActionEvent e){
		
		}	
}

[Fubarable]

Have you tried giving your program more available heap memory on the command line (or via your IDE if using one)?

[DarrylBurke]

A comment on coding style: your button array would be more representative if declared as a two-dimensional array, and the while(...) loop would be more expressive coded as nested for(...) loops.

  JButton[][] buttons = new JButton[1000][1000];
  :
  :
    for (int x = 0; x < 1000; x++) {
      for (int y = 0; y < 1000; y++) {
        buttons[x][y] = new JButton();
        dave.add(buttons[x][y], new Rectangle(x, y, 1, 1));
      }
    }

Better still would be to declare a static final variable SIDE (or WIDTH and HEIGHT if these are not always to be equal) and use the variable/s in the layout parameters, button array declaration and loop limits. Then if you want to change from 1000 X 1000 to some other numbers, that change is in only one place.

db