【Java】waitメソッドでスレッドを停止後にnotifyメソッドでスレッドの処理を再開されない

Question

以下のスレッドのコードで、waitメソッドを呼び出した後に、notifyメソッドを呼び出しても、停止したスレッドが再開されておりません。

waitメソッドをThread2で呼び出し、別のスレッドThread1でnotifyメソッドを呼び出しているので、停止していた処理が再開するという想定でした。

このようになった原因をご教示願いたいです。

■実行結果
wait開始
notify開始
notify終了
push

public class Stack {
	private boolean flag = false;
	synchronized public void push() {
		System.out.println("notify開始");
		notify();
		System.out.println("notify終了");
		if(flag) {
			try {
				wait();
			}catch(InterruptedException e) {
				System.out.println("push例外");
			}
		}
		flag = true;
		System.out.println("push");
	}
	
	synchronized public void pop() {
		if(!flag) {
			try {
				System.out.println("wait開始");
				wait();
				System.out.println("wait終了");
			}catch(InterruptedException e) {
				System.out.println("pop例外");
			}
		}
		System.out.println("pop ");
	}
}

public class Main{

	public static void main(String[] args) throws InterruptedException {

		Thread1 thread1 = new Thread1();
		Thread2 thread2 = new Thread2();
		thread1.start();
		thread2.start();
		
	}
}

class Thread1 extends Thread{
	public void run() {
		new Stack().push();
	}
}
class Thread2 extends Thread{
	public void run() {
		new Stack().pop();		
	}
}

■Mainクラス修正後

public class Main{

	public static void main(String[] args) throws InterruptedException {
		Stack stack = new Stack();
		Thread1 thread1 = new Thread1(stack);
		Thread2 thread2 = new Thread2(stack);
		thread1.start();
		thread2.start();
		
	}
}

class Thread1 extends Thread{
	Stack stack = null;
	
	public Thread1(Stack stack) {
		this.stack = stack;
	}
	
	public void run() {
		stack.push();
	}
}
class Thread2 extends Thread{
	Stack stack = null;
	public Thread2(Stack stack) {
		this.stack = stack;
	}
	
	public void run() {
		stack.pop();		
	}
}

Answer 1

synchronized メソッドは, それを定義しているクラスのオブジェクトを使用して同期を行っています.
Thread1 と Thread2 は共に new Stack() していますので, push / pop は別々のオブジェクトに対して行っており, synchronized は意味が有りません.

Comments

[kazu0630]

ご回答いただき、ありがとうございます。
元の質問に記載した修正後のコードで正しく再開されました。
とはいえ、元のソースコードでwaitメソッドで停止していたスレッドのnotifyメソッドで再開しなかった理由がよくわかっておりません。
よろしければ、もう少し理由を教えていただけますでしょうか。

[jimbe]

回答の通りなのですが...
Stack オブジェクトを家に例えますと, 宅配便を待っている(wait)家と宅配便が呼び鈴を鳴らした(notify)家が違うのでは, wait はいつまでも待ち続けます.

[kazu0630]

つまり、Thread1とThread2が異なるオブジェクトに対して処理を行っており、Thread1のStackオブジェクトに対してwaitメソッドを実行し、Thread2のStackオブジェクトに対してnotifyメソッドを呼び出しているため、うまく再開しなかったということですね。

ご回答いただき、ありがとうございました。