Почему synchronized метод правильно работает только с модификатором static?

Question

[Илья Борисенко]

Всем доброго дня, прошу прощения за столь, возможно, глупый вопрос. Изучаю тему потоков и в ходе экспериментов выяснил, что при указании метода как synchronized все равно выводится неверный результат. Если же добавить модификатор static к методу, то все работает так, как надо. Но у человека в примере, курс которого изучаю, все работает и без static. В чем моя ошибка?

Вот код класса и метода:

public class Traning {

	public static void main(String[] args) throws InterruptedException {
		MyThread myT1 = new MyThread();
		MyThread myT2 = new MyThread();
		myT1.start();
		myT2.start();
		myT1.join();
		myT2.join();

		System.out.println(MyThread.a);
	}
}

class MyThread extends Thread {
	static int a = 0;

	public void run() {
		for (int i = 0; i < 1000; i++) {
			incrementA();
		}
		System.out.println("myT is ended");
	}
	// Если в данный метод добавить static, то все работает.
	synchronized void incrementA() {
		MyThread.a++;
	}
	
}

Comments

[Dmitry Roo]

Может быть стоит добавить в вопрос пример, где все работает?

Answer 1

[Dmitry Roo]

Дело в том, что synchronized над методом указывает, что объектом синхронизации будет this, у вас этого this два: myT1 и myT2.
Внутри метода производятся операции над переменной static int a, которая только одна.
Получается, что в каждом потоке ничего не блокируется, т.к. каждый поток синхронизируется только сам с собой.

Когда вы добавляете static к методу - this становится одним на уровне всего приложения и потоки, выполняя этот метод сталкиваются с необходимостью синхронизации.

Comments

[Илья Борисенко]

Спасибо, понял почему так происходит. Только почему тогда у автора курса все работает без указания static (код из урока на скрине)? На скрине пример немного другой (в main методе создается 3 потока), но с аналогичным примером, который указан у меня в вопросе, тоже все прекрасно работает. Что я не догоняю?)

[Dmitry Roo]

Илья Борисенко, можно ссылку на источник?

[Илья Борисенко]

Курс покупался на Udemy, а среди уроков этого автора на Ютубе, данной темы нет, к сожалению.
Канал на Ютуб: Програмания.
Автор: Заур Трегулов.

[Dmitry Roo]

Илья Борисенко, ну тогда полный листинг хотя бы

[Илья Борисенко]

Dmitry Roo, Я прошу прощения, в рабочем примере у автора static переменная и метод, изменяющий ее тоже static. Переменная и метод находятся в объекте, который в своем методе main создает два потока, которые изменяют эту самую переменную.

Еще раз прошу прощения за невнимательность и благодарю за подробный ответ!