Parallel infinite loop

Question

I'm using threads for the first time in Java. Let me know what here is causing an infinite loop and how I can resolve it. I think the issue is being caused by the fact that I'm using a GUI along with threading.

What happens is that infinite copies of the GUI pop up.

public static void main(String[] args) {
    java.awt.EventQueue.invokeLater(new Runnable() {
        public void run() {
                new RollplayGUI();
        }
    });
}

public RollplayGUI() {
    createGUI();
    RollplayGUI rg = new RollplayGUI();
    Thread trg = new Thread(rg);
    trg.setDaemon(true);
    trg.run();
}

public void run() {
        //Some fun stuff my daemon thread is supposed to do 
    }

The reason I'm doing this is that the run() method does some net code that blocks until it gets a connection from another program.

Have any of the comments below helped? You should provide some feedback as to how it turned out on your end and if you found a solution. — Jerry Saravia
– Jerry Saravia, Commented Nov 28, 2012 at 21:30

Justin Beckwith · Accepted Answer · 2011-08-07 05:59:00Z

7

Your code is failing because the RollplayGUI constructor is instantiating a new RollplayGUI object. This will result in an infinite loop and likely a ... stack overflow.

answered Aug 7, 2011 at 5:59

Justin Beckwith

7,8761 gold badge38 silver badges57 bronze badges

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

1 Comment

dlev Over a year ago

A smile to my face when someone's first SO question is about... an SO.

Jerry Saravia · Accepted Answer · 2011-08-07 06:01:47Z

2

One reason you get an infinite number of gui's is that you have each RollplayGUI creates a new RollplayGUI right after it is instantiated.

Additionally, you should call start() on Thread object, not run. If you call run then it's as if you're not doing threading. start() handles the threading properly.

answered Aug 7, 2011 at 6:01

Jerry Saravia

3,8573 gold badges27 silver badges40 bronze badges

Comments

Spencer · Accepted Answer · 2011-08-07 06:00:01Z

0

This is an infinite loop because you have your RollbackGUI() create a new RollbackGUI(). Remove this line:

RollplayGUI rg = new RollplayGUI();

answered Aug 7, 2011 at 6:00

Spencer

1,4961 gold badge14 silver badges26 bronze badges

Comments

hmakholm left over Monica · Accepted Answer · 2011-08-07 06:00:44Z

0

What Justin said, but once that is fixed you're never starting your thread. You're just creating a Thread object (which will represent the thread when and if it starts) and then executing its run() method in the original thread. Use trg.start() instead of trg.run() to actually launch the thread.

answered Aug 7, 2011 at 6:00

hmakholm left over Monica

23.4k3 gold badges55 silver badges75 bronze badges

Comments

foxy · Accepted Answer · 2011-08-07 06:01:43Z

0

When you create your first RollplayGUI, the constructor is called. Inside that constructor you create a new RollplayGUI, whose constructor will again call RollplayGUI. This will occur indefinitely, which is the infinite loop.

The line is

RollplayGUI rg = new RollplayGUI();

answered Aug 7, 2011 at 6:01

foxy

7,6722 gold badges32 silver badges34 bronze badges

Collectives™ on Stack Overflow

Parallel infinite loop

5 Answers 5

1 Comment

Comments

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

5 Answers 5

1 Comment

Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related