Simple Button Array doesn't work

Question

I need to create a simple Minesweeper Game at school. I want to make a JButton-Array for an easy use. However it doesn't work! I fell like i've searched the whole internet for a solution! Can you help me maybe? Here's the code:

public class Minesweeper extends Applet {

    public void init() {

        //Frameinitialiing

        JFrame frame = new JFrame("Minesweeper");
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        int width = 800;
        frame.setSize(width, width);
        frame.setResizable(false);
        frame.setLocation(0,0);
        frame.setVisible(true);


        //Game

        JPanel panel = new JPanel();
        panel.setLayout(null);
        frame.add(panel);


        //Buttons

        int w = 80;

        JButton[][] button = new JButton[10][10];
        for (int i = 1; i == 9 ; i++ ) {

            for (int j = 1; j == 9 ; j++ ) {

                button[i][j].setBounds(i*80 , j*80 , w , w);
                this.add(button[i][j]);

            }
        }
    }
}

I'm sure you get an error message, which and what did you already do yourself (Solution is quite simple!) — brummfondel
– brummfondel, Commented Nov 27, 2016 at 16:53

Sweeper · Accepted Answer · 2016-11-27 17:05:39Z

1

Your code has lots of places that needs to be changed. But I have changed them for you:

    JFrame frame = new JFrame("Minesweeper");
    frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    int width = 800;
    frame.setSize(width, width);
    frame.setResizable(false);
    frame.setLocation(0,0);
    frame.setVisible(true);

    int w = 80;

    JButton[][] button = new JButton[10][10];
    for (int i = 0; i < 10 ; i++ ) {

        for (int j = 0; j < 10 ; j++ ) {
            button[i][j]  = new JButton();
            button[i][j].setBounds(i*80 , j*80 , w , w);
            frame.add(button[i][j]);

        }
    }

First of all, the for loop is wrong. You seemed to have messed up the numbers. The two loops should both be = 0 and < 10. Also, I removed these three lines:

    JPanel panel = new JPanel();
    panel.setLayout(null);
    frame.add(panel);

You forgot the initialize the buttons using new JButton(), so I did it for you.

Finally, it is frame.add(button[i][j]) instead of this.add(button[i][j]).

It looks like this:

answered Nov 27, 2016 at 17:05

Sweeper

292k23 gold badges260 silver badges438 bronze badges

Sign up to request clarification or add additional context in comments.

3 Comments

Peaq Over a year ago

Thank you! However, this is what i see now: postimg.org/image/v3fkocdiz - the buttons appear when i go over their location....

Sweeper Over a year ago

Hmm... That is weird. Somehow I didn't get that kind of thing. Maybe this is something to do with the inner workings of JFrame layouts. You can always search on the internet or ask another question. I recommend you to not use coordinates. Instead, use one of the layouts that awt provided, like FlowLayout. @Peaq

Peaq Over a year ago

Thanks, i just tweaked the frame width and it worked :D!

Collectives™ on Stack Overflow

Simple Button Array doesn't work

1 Answer 1

3 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

3 Comments

Your Answer

Sign up or log in

Post as a guest

Related