Java ожидания оповещать общего целого объекта

user3157908:

Я пытаюсь напечатать цифры от 1 до 10 в последовательности с использованием общего целого объекта по нескольким потокам. При использовании разделяемого объекта как AtomicInteger, программа работает правильно, но при использовании нормальных объектов Integer, программа генерирует исключение, и я не знаю, почему это происходит.

Программа с AtomicInteger

import java.util.concurrent.atomic.AtomicInteger;  

public class ThreadingProblem {

  public static void main(String[] args) {  
    AtomicInteger sharedInt = new AtomicInteger(0);  
    Thread t1 = new Thread(new ThreadingPrintingTask(sharedInt), "PrinterThread");  
    Thread t2 = new Thread(new ThreadingIncrementingTask(sharedInt), "IncrementerThread");  
    t1.start();  
    t2.start();  
  }  
}  

class ThreadingPrintingTask implements Runnable {    

  private AtomicInteger sharedObject;  

  public ThreadingPrintingTask(AtomicInteger sharedObject) {  
    this.sharedObject = sharedObject;  
  }  

  @Override  
  public void run() {  
    try {  
      synchronized (sharedObject) {  
        while (true) {  
          sharedObject.wait();  
          System.out.println("Shared object value is: " + sharedObject);  
          sharedObject.notify();  
        }  
      }  
    }  
    catch (InterruptedException e) {  
    }  

  }  
}  

class ThreadingIncrementingTask implements Runnable {  

  private AtomicInteger sharedObject;  

  public ThreadingIncrementingTask(AtomicInteger sharedObject) {  
    this.sharedObject = sharedObject;  
  }  

  @Override  
  public void run() {  
    synchronized (sharedObject) {  
      while (this.sharedObject.get() < 10) {  
        this.sharedObject.incrementAndGet();  
        this.sharedObject.notify();  
        try {  
          this.sharedObject.wait();  
        }  
        catch (InterruptedException e) {  
          // TODO Auto-generated catch block  
          e.printStackTrace();  
        }  
      }  
    }  
  }  
}  

Выход

Shared object value is: 1
Shared object value is: 2
Shared object value is: 3
Shared object value is: 4
Shared object value is: 5
Shared object value is: 6
Shared object value is: 7
Shared object value is: 8
Shared object value is: 9
Shared object value is: 10

Программа с обычным объектом Integer

public class ThreadingProblem {

  public static void main(String[] args) {
    Integer sharedInt = new Integer(0);
    Thread t1 = new Thread(new ThreadingPrintingTask(sharedInt), "PrinterThread");
    Thread t2 = new Thread(new ThreadingIncrementingTask(sharedInt), "IncrementerThread");
    t1.start();
    t2.start();
  }
}

class ThreadingPrintingTask implements Runnable {

  private Integer sharedObject;

  public ThreadingPrintingTask(Integer sharedObject) {
    this.sharedObject = sharedObject;
  }

  @Override
  public void run() {
    try {
      synchronized (sharedObject) {
        while (true) {
          sharedObject.wait();
          System.out.println("Shared object value is: " + sharedObject);
          sharedObject.notify();
        }
      }
    }
    catch (InterruptedException e) {
    }

  }
}

class ThreadingIncrementingTask implements Runnable {

  private Integer sharedObject;

  public ThreadingIncrementingTask(Integer sharedObject) {
    this.sharedObject = sharedObject;
  }

  @Override
  public void run() {
    synchronized (sharedObject) {
      while (this.sharedObject < 10) {
        this.sharedObject++;
        this.sharedObject.notify();
        try {
          this.sharedObject.wait();
        }
        catch (InterruptedException e) {
          // TODO Auto-generated catch block
          e.printStackTrace();
        }
      }
    }
  }
}

Выход

Exception in thread "IncrementerThread" java.lang.IllegalMonitorStateException
    at java.lang.Object.notify(Native Method)
    at com.itiviti.apps.catalys.shared.mock.ThreadingIncrementingTask.run(ThreadingProblem.java:52)
    at java.lang.Thread.run(Unknown Source)

biziclop:

this.sharedObject++; не делать то, что вы предполагали, что это будет делать.

Так как Integerнеизменен, он не может изменить существующий общий объект. Что эта операция делает вместо этого распаковывать значения в int, увеличивает его, то окно его обратно в другую Integerинстанцию.

Так что ваш код (почти *) эквивалентно следующему:

int temp = this.sharedObject.intValue();
temp = temp + 1;
this.sharedObject = new Integer(temp);

На данный момент ваш объект больше не тот же экземпляр, ваши synchronizedблоки не будут выстраиваться с wait()/ notify()звонками.

Обратите внимание , что это не имеет ничего общего с атомарностью AtomicInteger, это просто с тем, как ++оператор работает по принципу Integer.

* На самом деле вы можете получить кэшированные экземпляр вместо new Integer(), но она все равно будет другой экземпляр, поскольку она представляет собой другое intзначение.