Многопоточность ленивая инициализация

Question

У меня есть вопрос о java multi-threading.

У меня есть класс, который имеет доступ к нескольким потокам.

Class A 
{ 
    private Object obj; 

    public Object returnObject() 
    { 
     if(condition) 
      return getObjectA(); 
     else 
      return getObjectB(); 
    } 

    public Object getObjectA() 
    { 
     obj = new Object() 
     obj.setProperty("prp1"); 
    } 

    public Object getObjectB() 
    { 
     obj = new Object() 
     obj.setProperty("prp2"); 
    } 
} 

Когда есть несколько потоков, обращающихся к getObj ..(). Это создает проблемы с состоянием «obj». Были ли нежелательные результаты?

Благодарности SLaks и Питер,

Я думаю, что следующий будет также решать проблемы безопасности нити:

public Object getObjectA() 
{ 
    Object obj = new Object() 
    obj.setProperty("prp1"); 
    return obj;                     
} 

Answer 1

Да.

Если два потока вызывают различные пути кода в returnObject() одновременно, она может работать getObjectB() между двумя высказываниями в getObjectA() и в конечном итоге вызова obj.setProperty("prp1"); на ObjectB.

Answer 2

Да, это создаст проблемы (попробуйте код, чтобы увидеть, какие проблемы, но данные будут потеряны). Простейшим решением является использование блока synchronized.

Answer 3

При условии, что if(true) не является жестко закодированным, так как это будет исправлять путь выполнения, где getObjA() всегда вызывается!

Да, этот код может работать в состоянии гонки, где значение obj может зависеть от порядка выполнения потока (то есть, какой порядок getObj ..() вызывается несколькими потоками).

Вы должны получить synchronize доступ к переменной obj.

Answer 4

Я думаю, что getObjectA() и getObjectB() должны быть объявлены как частные, а затем вы должны использовать синхронизированный блок внутри метода returnObject(), так что-то вроде этого:

class A 
{ 

private Object obj; 

public Object returnObject() 
{ 

    Object result; 
    synchronized (A.class) { 
    result = (true) ? getObjectA() : getObjectB(); 
    } 
    return result; 
} 

private Object getObjectA() 
{ 
    obj = new Object() 
    obj.setProperty("prp1"); 
} 

private Object getObjectB() 
{ 
    obj = new Object() 
    obj.setProperty("prp2"); 
} 

} 

Очевидно верно условия должны заменить на какое-то условие ...

Answer 5

Я бы предложил перейти к NOT имеет форму 'obj 'как элемент данных, так как требуется одновременное обращение к нему.

Есть несколько решений:

1. Создание объекта локально внутри функции
2. Использовать ThreadLocal объект, который может "обручи" The 'OBJ' Данные члены
3. Создание объекта один раз и пользователь clone() для каждого вызова локально локальному объекту и редактировать его.

Answer 6

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

Class A { 
    public Object returnObject() { 
     if (true) 
      return getObjectA(); 
     return getObjectB(); 
    } 

    public Object getObjectA() { 
     Object obj = new Object() 
     obj.setProperty("prp1"); 
     return obj; 
    } 

    public Object getObjectB() { 
     Object obj = new Object() 
     obj.setProperty("prp2"); 
     return obj; 
    } 
} 

Если это сработает для вас, рассмотрите возможность использования этих методов static.