Whats wrong with this loop code?

wordone

Sometimes it works, sometimes it doesn't but I can't find the bug:

Expand|Select|Wrap|Line Numbers

 
package Loop.Test;
 
import java.util.concurrent.ExecutorService;

import java.util.concurrent.Executors;
 
public class TheLoopTest {
 
        private final ExecutorService executorService = Executors.newFixedThreadPool(100);

        private static final int MAX = 1000;

        private int zeroToTop = 0;
 
        TheLoopTest() throws InterruptedException {

                for (int i = 0; i < MAX; i++) {

                        executorService.execute(new Runnable() {

                                public void run() {

                                        incrementAllTheWay();

                                }

                        });

                }

                executorService.shutdown();

                while (!executorService.isTerminated()) {

                        Thread.sleep(500);

                }

                System.out.println(zeroTotop);

        }
 
        private void incrementAllTheWay() {

                int obfusticatedIncremental = zeroToTop;

                obfusticatedIncremental = obfusticatedIncremental + 1;

                zeroToTop = obfusticatedIncremental;

        }
 
        /**

         * @param args

         */

        public static void main(String[] args) throws InterruptedException {

                new TheLoopTest();

        }
 
}

JosAH

Can you tells us what you think this thing is supposed to do and what it actually does? This is not a guessing forum.

kind regards,

Jos

wordone

Still not working, any other suggestions?

JosAH

Quote:

Originally Posted by wordone

Still not working, any other suggestions?

Please answer my question (see my previous reply). If not I'm out of here; this is not a guessing forum.

kind regards,

Jos

wordone

It's about a race condition and it should print out '1000' every time but it doesn't always. I tried to change Executors.newFixedThreadPool(100); to match the Thread.sleep(500); so the times are the same but that didn't seem to fix it.

Whats wrong with this loop code?

re: Whats wrong with this loop code?

re: Whats wrong with this loop code?

re: Whats wrong with this loop code?

re: Whats wrong with this loop code?

Similar Java bytes

/bytes/development

/bytes/it

/bytes/about