File APIs for Java Developers
Manipulate DOC, XLS, PPT, PDF and many others from your application.
http://aspose.com/file-tools
Win a copy of Clojure in Action this week in the Clojure forum!
  • Post Reply
  • Bookmark Topic Watch Topic
  • New Topic

Thread Doubt

 
NDP Prasad
Ranch Hand
Posts: 177
  • 0
  • Mark post as helpful
  • send pies
  • Quote
  • Report post to moderator
public class newqw implements Runnable {
public static void main(String[] args) {
new Thread(new newqw("Wallace")).start() ;
new Thread(new newqw("Gromit")).start();
}
private String name;
public newqw(String name) { this.name = name; }
public void run() {
message(1);
message(2);
}
private synchronized void message(int n) {
System.out.print(name + "-" + n + " ");
}
}

why is the above code printing different answers at different times? what should i do to get a same answer always?
 
Joe Harry
Ranch Hand
Posts: 10045
3
Eclipse IDE Mac PPC Ubuntu
  • 0
  • Mark post as helpful
  • send pies
  • Quote
  • Report post to moderator
Hi,

why is the above code printing different answers at different times? what should i do to get a same answer always?


Dismantle your system and change your processor. Just kidding. This is what expected regarding the behaviour of thread. Thereads dont run the way you like, it is totally under the control of the JVM. You don't have to worry about this.

Regards,
Jothi Shankar Kumar. S
 
catherine powell
Greenhorn
Posts: 26
  • 0
  • Mark post as helpful
  • send pies
  • Quote
  • Report post to moderator
Hi,

I believe the problem is that you created a new Runnable object for each thread and you're using these different objects for synchronization. Since each thread is looking to obtain a different object lock (each thread is looking to obtain it's own Runnable object's lock) none of them will encounter a situation where they are blocked and cannot obtain the lock they're looking for, so really there's no sychronization.

I think one possible solution would be to create a static reference variable in the Runnable class (so just 1 lock is available) and sychronize the code in your run method on that (see below).

 
I agree. Here's the link: http://aspose.com/file-tools
  • Post Reply
  • Bookmark Topic Watch Topic
  • New Topic