Опрос по синхронизации Java

Question

Вот код, который я видел один раз. Вы видите, что с ним не так?

[обновлено]

public class ResourceManager1 
{ 
    private final String mutex = ""; 
    Object resource = null; 

    public Object getResource() 
    { 
     synchronized (mutex) 
     { 
      if (resource == null) 
      { 
       resource = new Object(); 
      } 
     } 

     return resource; 
    } 
} 

public class ResourceManager2 
{ 
    private final String mutex = ""; 
    Object resource = null; 

    public Object getResource() 
    { 
     synchronized (mutex) 
     { 
      if (resource == null) 
      { 
       resource = new Object(); 
      } 
     } 

     return resource; 
    } 
} 

Answer 1

Не синхронизировать строки, в частности строковые литералы, которые интернированы. У вас в основном есть только один замок.

В общем, никогда не синхронизируйтесь ни на одной ссылке, которая может быть видна вне вашего класса (включая «это») , если только цель внешней видимости точно для целей блокировки. Обычно я использую переменную private final, созданную исключительно для блокировки.

Answer 2

мьютекс не является окончательным и ресурс не является частным.

Кроме того, вам нужен метод getResource, который возвращает ресурс, но я полагаю, что это просто опечатка.

Answer 3

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