Не синхронизироваться на разделяемых или подменяемых объектах: строковых литералах (интернированы), Integer и других обёртках (кэш значений), публичном this, изменяемом поле. Правильно — private final Object lock.
А ещё Монитор на виртуальных потоках до JDK 24 вызывал pinning
Так подсказку видит пользователь sotto на живом собеседовании — за пару секунд, пока вы думаете вслух.
Опасность в том, что монитор может оказаться общим с совершенно посторонним кодом: строковые литералы интернируются в один экземпляр, а Integer.valueOf возвращает объект из кэша для малых значений — в результате два независимых участка кода блокируют друг друга, и появляется загадочный deadlock. Синхронизация на this проблематична, когда объект публичен: внешний код может захватить тот же монитор и удерживать его сколь угодно долго. Отдельная классическая ошибка — синхронизироваться на изменяемом поле: если ссылку переприсвоить, потоки будут блокировать разные объекты, и защиты не будет вовсе. Отсюда идиома: private final Object lock = new Object() и блокировка на нём.
Несколько верных вариантов. Подумайте, потом откройте ответ.
Взаимное исключение и видимость через монитор, reentrancy, guarded blocks с wait/notify и причины deadlock.
sotto слышит вопрос интервьюера и показывает короткий ответ и план, как в блоке «Коротко». Оставьте контакт — пришлём доступ и промокод.
sotto