Question

我试图编写一个代码，其中多个线程调用共享对象上的方法来递增/递减/打印存储在其中的计数器。我还想要的是这些数字在0到8之间进行血管分离。这是输出可能如下所示： 0123234567654566677877666655 ....

有人可以看看我已经提出的问题并给我一些关于我是否走在正确轨道上的指示：

我的共享对象：

public class SyncObj{
        private int i;
        public synchronized void inc(){
                if(i<8)
                  i++;
        }
        public synchronized void dec(){
                if(i > 0)
                   i--;
        }
        public synchronized void print(){
                System.out.print(i);
        }
}

为了防止印刷品出现饥饿并确保打印每个inc / dec，我可以拥有一个名为hasPrinted的私有变量，并按如下方式重写该类：

public class SyncObj{
            private int i;
            //Changed Boolean to boolean as Keith Randall pointed out
            private boolean hasPrinted = false;
            public synchronized void inc(){
                    if(i<8 && hasPrinted){
                      i++;
                      hasPrinted = false;
                    }
            }
            public synchronized void dec(){
                    if(i > 0 && hasPrinted){
                       i--;
                       hasPrinted = false;
                    }
            }
            public synchronized void print(){
                    System.out.print(i);
                    hasPrinted = true;
            }
    }

有人可以查看上面的代码片段并查看它是否有陷阱和陷阱？

由于

Answer 1

Boolean - ＆gt; boolean，没有必要拥有一个对象而不是一个原始类型。

你的第一个代码很好。您的第二个代码无法解决您防止饥饿或确保每个inc / dec打印的要求。为什么不让inc / dec打印出值呢？

Answer 2

您应该习惯使用队列进行打印。

public class SyncObj {
  private volatile int i;
  private BlockingQueue<Integer> q = new LinkedBlockingQueue<Integer>();
  public synchronized void inc() {
    if (i < 8) {
      i++;
      q.add(i);
    }
  }
  public synchronized void dec() {
    if (i > 0) {
      i--;
      q.add(i);
    }
  }
  public void print() {
    for (Integer i = q.poll(); i != null; i = q.poll()) {
      System.out.print(i);
    }
  }
  private static volatile boolean stop = false;
  public static void main(String[] args) throws InterruptedException {
    final SyncObj o = new SyncObj();

    new Thread(new Runnable() {
      @Override
      public void run() {
        while (!stop) {
          o.inc();
        }
      }
    }).start();

    new Thread(new Runnable() {
      @Override
      public void run() {
        while (!stop) {
          o.dec();
        }
      }
    }).start();

    new Thread(new Runnable() {
      @Override
      public void run() {
        while (!stop) {
          o.print();
        }
      }
    }).start();

    Thread.currentThread().sleep(1000);
    stop = true;
  }
}

我的输出如下：

1012345678765432101234567876543210123456787654321012345678765432101234567876543210123456787654321012345678

使用多个线程递增/递减/打印（代码审查）

2 个答案: