0

Написал код, ожидал что lock в методах addToList будет происходить на одном и том же объекте а точнее worker, но судя по тестам lock происходит на разных объектах. Вопрос на каких? Не на тредах же

public class Main {

    public static void main(String[] args) throws InterruptedException {
       Worker worker = new Worker();

        Thread thread1 = new Thread(() -> {
            worker.doWork();
        });

        Thread thread2 = new Thread(() -> {
            worker.doWork();
        });

        thread1.start();
        thread2.start();

    }

}

class Worker {

    List<Integer> list1 = new ArrayList<>();
    List<Integer> list2 = new ArrayList<>();
    ThreadLocalRandom random = ThreadLocalRandom.current();

    public synchronized void addToList1(){
            list1.add(random.nextInt(100));
    }

    public synchronized void addToList2(){
            list2.add(random.nextInt(100));
    }

    public void doWork()  {
        for (int i = 0; i < 1000; i++) {
                addToList1();
                addToList2();
        }
    }
}
7
  • судя по тестам lock происходит на разных объектах как вы это определили?
    – IR42
    20 апр 2020 в 22:56
  • по времени выполнении программы 21 апр 2020 в 1:29
  • Это как? У вас есть эталонное значения времени, которое должна выполняться данная программа на вашем конкретном компе и на конкретной jre при синхронизированных методах на одном объекте и на нескольких да ещё и на таком маленьком тесте? В любом случая синхронизация происходит на объекте в котором вызывают нестатические synchronized методы (worker в данном случае). К тому же вы забыли thread1.join(); thread2.join();, ваша программа завершается, не успев даже запустить потоки...
    – IR42
    21 апр 2020 в 2:13
  • @IR42 она не завершается и здесь не нужны join'ы. Это же не потоки-демоны
    – NarasuOo
    21 апр 2020 в 5:21
  • @ВасилийПупкин lock происходит на одном и том же объекте. Как уже сказали выше, когда syncronized метод не статический, ему в качестве объекта lock'a передается this
    – NarasuOo
    21 апр 2020 в 5:25

0

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Посмотрите другие вопросы с метками или задайте свой вопрос.