permaculture playing cards*
The moose likes Threads and Synchronization and the fly likes Thread Big Moose Saloon
  Search | Java FAQ | Recent Topics | Flagged Topics | Hot Topics | Zero Replies
Register / Login
JavaRanch » Java Forums » Java » Threads and Synchronization
Bookmark "Thread" Watch "Thread" New topic
Author

Thread

Tomm Schuman
Greenhorn

Joined: Mar 04, 2005
Posts: 11
Well this is the third time I ask this question, this time I try give you enough information and I have also read from the beginning of the java tutorial on sun's website to the lesson about threads. In my last post you said i should read about event listeners but i don't believe the problem is in there.

I want an Integer to rise by 1 every second while I have left arrow button pressed and then stop rising when I relase it. I tried somehow make an inner class which would be the tread which starts and stops when I press and relase the left arrow button.

But when I run the program (I have a println method which shows the value of heading360) it first prints out 2's, then when I press left arrow it prints 0's for like 1 sec and then prints 1's till I end the program. I believe the problem is that I don't really undertand how threads work. I read from the tutorial but I couldn't use the examples for this.

Below is the code I'm using. I have removed parts of the code. I'll post more if I have forgot to write a part you need to see.

You can download the whole source code here:Killers.java


[ March 09, 2005: Message edited by: Tomm Schuman ]
[ March 09, 2005: Message edited by: Tomm Schuman ]
Ilja Preuss
author
Sheriff

Joined: Jul 11, 2001
Posts: 14112
I see three problems with your code, on first sight:

- Thread.sleep gets the time in ms - 100ms is 0.1s, not 1s

- you don't print the value of the heading variable that gets changed in your thread, but the value that you copied in the keyDown method

- you are accessing the heading field from two different threads - the one that changes it, and the one that reads it to print it. You need to synchronize that access to work reliably. With other words, as written the access to heading is not threadsave.

Hope this helps.

Moving this to Threads and Synchronization, where there are more experts able to help you...


The soul is dyed the color of its thoughts. Think only on those things that are in line with your principles and can bear the light of day. The content of your character is your choice. Day by day, what you do is who you become. Your integrity is your destiny - it is the light that guides your way. - Heraclitus
Tomm Schuman
Greenhorn

Joined: Mar 04, 2005
Posts: 11
YAY!! IT WORKED

The problem was here:



I changed it to if (allDone == true) then the thread worked but i could only run it once. Then made Thread = null and it worked

Here is the code i use now. Could someone check it if I could do something better. Especially the synchronized methods because I have no idea if i put them to the right places.
 
I agree. Here's the link: http://aspose.com/file-tools
 
subject: Thread