Re: unable to synchronize on primitive wrapper class?

Eniac Zhang wrote:


It's often a good idea to synchronize on a private object created
specially for this purpose. It's a common pattern and would prevent
issues like this this, i.e. some other code does something you didn't
expect, and your locking scheme goes to heck. Also make the protected
resource private, then you have complete control over the resource.

I think this works, although perhaps something should be done with count
as well:

public class VolatileVar {
     static class Worker implements Runnable {
         private static Integer total = 0;
         private static Object lock = new Object();
         public int count = 0;

         public void run() {
             while (!Thread.interrupted()) {
                 synchronized (lock) {
                     total++;
                 }
                 count++;
             }
         public int getTotal() {
             int i;
             sychronized (lock) {
                i = total;
             }
             return i;
         }
     }