How can I make my buttons switch without a loop?

Question

I was wondering how I would be able to make my buttons switch in this class which I call on from a different class. But it goes in and does only one change of the buttons and that's it..

package code;

import java.awt.Font;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.util.Random;

import javax.swing.JButton;


public class Something implements ActionListener {

    private Game G;
    private int random;
    private JButton a;
    private JButton b;
    private JButton c;
    private Font i;

    public Something(Game g, int rand, JButton d, JButton e, JButton f, Font h) {
        G = g;
        random = rand;
        a = d;
        b = e;
        c = f;
        i = h;
    }

    public void actionPerformed(ActionEvent e) {
        // TODO Auto-generated method stub
        G.increment();
        if (random == 0) {
            a.setText("A");
            a.setEnabled(true);
            b.setEnabled(false);
            c.setEnabled(false);
            SuperSize(a);
            SmallerSize(b);
            SmallerSize(c);
            random = RandomNum();
            ;
        } else if (random == 1) {
            b.setText("B");
            a.setEnabled(false);
            b.setEnabled(true);
            c.setEnabled(false);
            SuperSize(b);
            SmallerSize(a);
            SmallerSize(c);
            random = RandomNum();
        } else if (random == 2) {
            c.setText("C");
            a.setEnabled(false);
            b.setEnabled(false);
            c.setEnabled(true);
            SuperSize(c);
            SmallerSize(a);
            SmallerSize(b);
            random = RandomNum();
        }
    }

    public int RandomNum() {
        Random r = new Random();
        int rand = 0;
        rand = r.nextInt(3);
        return rand;
    }

    public void SuperSize(JButton a) {
        Font myFont = i.deriveFont(Font.BOLD, i.getSize() * 4);
        a.setFont(myFont);
    }

    public void SmallerSize(JButton a) {
        a.setFont(i);
    }
}

            }

I don't know what to do , can you guys help me?

Don't create a new instance of Random each time you call RandomNum, instead, create a instance in the constructor and continue to re-use it — MadProgrammer
– MadProgrammer, Commented Nov 2, 2015 at 5:09

MadProgrammer · Accepted Answer · 2015-11-02 05:11:28Z

1

Don't create a new instance of Random each time you call RandomNum, instead, create a instance in the constructor and continue to re-use it.

public class Something implements ActionListener {

    //...

    private Random rnd;

    public Something(Game g, int rand, JButton d, JButton e, JButton f, Font h) {
        //...
        rnd = new Random();
    }

    public int RandomNum() {
        return rnd.nextInt(3);
    }

answered Nov 2, 2015 at 5:11

MadProgrammer

348k22 gold badges243 silver badges379 bronze badges

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

7 Comments

Andriy Ukraine Over a year ago

It's still the same thing , after the second switch of the buttons, it doesn't work anymore

MadProgrammer Over a year ago

I'll need more code to test with. Consider providing a runnable example which demonstrates your problem. This is not a code dump, but an example of what you are doing which highlights the problem you are having. This will result in less confusion and better responses

Andriy Ukraine Over a year ago

I'm still new to this site, so I should just provide like the same problem in the smallest code but including all the classes?

MadProgrammer Over a year ago

Check the linked in the previous comment to understand which a "runnable example' should provide. I stripped out all the UI code and run your RandomNum method through a 100 iteration loop and it works fine, the problem is likely somewhere else

MadProgrammer Over a year ago

@AndriyUkraine We need something is compilable, which can be run with out modification and only includes those parts which demonstrate your general workflow and problem

|

Collectives™ on Stack Overflow

How can I make my buttons switch without a loop?

1 Answer 1

7 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

7 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related