Синхронизация метода с двумя потоками

@Андрей1981 · Регистрация: 05.06.2016

Студворк — интернет-сервис помощи студентам

Добрый день, уважаемые!
Только начал знакомиться с многопоточностью и пытаюсь синхронизировать метод для корректной работы с двумя потоками:

Java
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
public class ServiceCounter implements Runnable {
 
    Thread thread;
    private int result1 = 0;
    private int result2 = 0;
    private int first_value = 1;
 
    ServiceCounter() {
    }
 
    @Override
    public void run() {
        ToAdd();
    }
 
    synchronized void ToAdd (){
 
        for (int i = 0; i < 10; i++) {
            try {
                result2 = result1 + first_value;
                result1 = result2;
                System.out.println(result2);
                Thread.sleep((long) (Math.random() * 1000));
            } catch (InterruptedException e) {
                e.printStackTrace();
            }
 
        }
    }
 
    public void start() {
        if (thread == null) {
            thread = new Thread(this);
            thread.start();
        }
    }
    
}

Java
1
2
3
4
5
6
7
8
9
10
11
public class MainThread {
 
    public static void main(String[] args) throws InterruptedException {
 
        ServiceCounter thread1 = new ServiceCounter();
        ServiceCounter thread2 = new ServiceCounter();
        thread1.start();
        thread2.start();
 
    }
}

Выводит в консоли: 1 1 2 2 3 3 4 5 4 5 6 7 6 8 7 9 8 10 9 10

То есть ничегошеньки-то не синхронизирует.

@xoraxax · 18.04.2019, 12:40

Java
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
public class ServiceCounter implements Runnable {
 
    private final Object lock;
 
    public ServiceCounter(Object lock) {
        this.lock = lock;
    }
 
    @Override
    public void run() {
        toAdd();
    }
 
    private void toAdd() {
        synchronized (lock) {
            for (int i = 0; i < 10; i++) {
                try {
                    System.out.println(i);
                    Thread.sleep((long) (Math.random() * 1000));
                } catch (InterruptedException e) {
                    e.printStackTrace();
                }
            }
        }
    }
}
 
public class App {
    public static void main(String[] args) {
        Object lock = new Object();
        new Thread(new ServiceCounter(lock)).start();
        new Thread(new ServiceCounter(lock)).start();
    }
}

размышляй

@Андрей1981 · 18.04.2019, 15:46 **[ТС]**

xoraxax, спасибо, теперь заработало.

Только размышления не приводят меня к пониманию то, почему заработало.
Ведь я использовал синхронизацию метода, Вы - синхронизацию блока кода внутри метода.
С точки зрения результата же они должны быть эквивалентны...

@xoraxax · 18.04.2019, 16:02

не о том думаешь. Почитай, что конкретно делает synchronized и как оно это делает

Добавлено через 1 минуту
попробуй вместо lock синхронизироваться на this (что ты и делаешь в общем то)

@Андрей1981 · 18.04.2019, 17:29 **[ТС]**

Теперь, вроде, понял.
Это равно ценно

Java
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
public class ServiceCounter implements Runnable {
 
    public ServiceCounter() {}
 
    @Override
    public void run() {
        toAdd();
    }
 
    void toAdd() {
        synchronized (this) {
            for (int i = 0; i < 10; i++) {
                try {
                    System.out.println(i);
                    Thread.sleep((long) (Math.random() * 1000));
                } catch (InterruptedException e) {
                    e.printStackTrace();
                }
            }
        }
    }
}

блокируется экземпляр класса, который включает метод, содержание которого надо блокировать?

@xoraxax · 18.04.2019, 18:48

ты не пытайся угадать, а почитай что-нибудь, вот например https://habr.com/ru/post/143237/

Новые блоги и статьи Все статьи Все блоги /
Первый деплой lagorue 16.01.2026 Не спеша развернул своё 1ое приложение в kubernetes. А дальше мне интересно создать 1фронтэнд приложения и 2 бэкэнд приложения развернуть 2 деплоя в кубере получится 2 сервиса и что-бы они. . .	Расчёт переходных процессов в цепи постоянного тока igorrr37 16.01.2026 / * Дана цепь постоянного тока с R, L, C, k(ключ), U, E, J. Программа составляет систему уравнений по 1 и 2 законам Кирхгофа, решает её и находит токи на L и напряжения на C в установ. режимах до и. . .	Восстановить юзерскрипты Greasemonkey из бэкапа браузера damix 15.01.2026 Если восстановить из бэкапа профиль Firefox после переустановки винды, то список юзерскриптов в Greasemonkey будет пустым. Но восстановить их можно так. Для этого понадобится консольная утилита. . .	Изучаю kubernetes lagorue 13.01.2026 А пригодятся-ли мне знания kubernetes в России?
Сукцессия микоризы: основная теория в виде двух уравнений. anaschu 11.01.2026 https:/ / rutube. ru/ video/ 7a537f578d808e67a3c6fd818a44a5c4/	WordPad для Windows 11 Jel 10.01.2026 WordPad для Windows 11 — это приложение, которое восстанавливает классический текстовый редактор WordPad в операционной системе Windows 11. После того как Microsoft исключила WordPad из. . .	Classic Notepad for Windows 11 Jel 10.01.2026 Old Classic Notepad for Windows 11 Приложение для Windows 11, позволяющее пользователям вернуть классическую версию текстового редактора «Блокнот» из Windows 10. Программа предоставляет более. . .	Почему дизайн решает? Neotwalker 09.01.2026 В современном мире, где конкуренция за внимание потребителя достигла пика, дизайн становится мощным инструментом для успеха бренда. Это не просто красивый внешний вид продукта или сайта — это. . .

@Андрей1981 0 / 0 / 0 Регистрация: 05.06.2016 Сообщений: 128
	18.04.2019, 15:46 [ТС]
	xoraxax, спасибо, теперь заработало. Только размышления не приводят меня к пониманию то, почему заработало. Ведь я использовал синхронизацию метода, Вы - синхронизацию блока кода внутри метода. С точки зрения результата же они должны быть эквивалентны... 0

@xoraxax 3639 / 2971 / 918 Регистрация: 05.07.2013 Сообщений: 14,220
	18.04.2019, 16:02
	не о том думаешь. Почитай, что конкретно делает synchronized и как оно это делает Добавлено через 1 минуту попробуй вместо lock синхронизироваться на this (что ты и делаешь в общем то) 1

@xoraxax 3639 / 2971 / 918 Регистрация: 05.07.2013 Сообщений: 14,220
	18.04.2019, 18:48
	Сообщение было отмечено Welemir1 как решение Решение ты не пытайся угадать, а почитай что-нибудь, вот например https://habr.com/ru/post/143237/ 1

Синхронизация метода с двумя потоками

Решение