selecting every element in array. Java

Question

I am trying to build a TicTacToe game and to draw the game every element in my 2d array should be disabled. How to find out if all of my my JButtons in my array is disabled.Here I assigned I created JButton.

for(int row = 0; row < buttons.length; row++){
        for(int column = 0; column<buttons[row].length; column++){
            buttons[row][column] = new JButton();
            this.add(buttons[row][column]);
            buttons[row][column].addActionListener(new tickSquare());

        }
    }

and here I tried to do looping but it does if everything is disabled

    for(int row = 0; row < buttons.length; row++){
            for(int column = 0; column<buttons[row].length; column++){

                if(buttons[row][column].isEnabled() == false){
                    JOptionPane.showMessageDialog(null,labelD);
                }

            }
        }

BackSlash · Accepted Answer · 2017-04-11 16:42:54Z

1

Don't show the message dialog in the if itself. Use the loop to update a variable and use that variable outside the loop to decide if you have to show the popup or not:

boolean allDisabled = true;

for(int row = 0; row < buttons.length; row++){
    for(int column = 0; column<buttons[row].length; column++){

        if(buttons[row][column].isEnabled()){
            //This button is enabled, so we set allDisabled to false
            allDisabled = false;
        }

    }
}

if(allDisabled) {
    JOptionPane.showMessageDialog(null,labelD);
} else {
    // At least one button is enabled!
}

You can also break the loop to avoid unnecessary loops when an enabled button is found:

boolean allDisabled = true;

loop:
for(int row = 0; row < buttons.length; row++){
    for(int column = 0; column<buttons[row].length; column++){

        if(buttons[row][column].isEnabled()){
            //This button is enabled, so we set allDisabled to false
            allDisabled = false;
            break loop;
        }

    }
}

But I don't like labels and breaks, so if possible I try to avoid using them.

answered Apr 11, 2017 at 16:42

BackSlash

22.3k25 gold badges103 silver badges143 bronze badges

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

1 Comment

BackSlash Over a year ago

@AzamatAdylbekov You're welcome! Don't forget to accept the answer if it helped you solve the issue :)

Collectives™ on Stack Overflow

selecting every element in array. Java

1 Answer 1

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related