L'objet local est utilisé comme paramètre de méthode, puis les objets ne sont pas thread-safe. Pourquoi?

Question

La seule exception est bien sûr si l'une des méthodes appelées en paramètre LocalObject stocke l'instance LocalObject d'une manière qui permet d'y accéder à partir d'autres threads. S'il te plait, calme-moi sur ça. comment c'est son travail.

public void someMethod(){ 

    LocalObject localObject = new LocalObject(); 

    localObject.callMethod(); 

// localObject is not threadsafe 
    method2(localObject); 
} 

public void method2(LocalObject localObject){ 
    localObject.setValue("value"); 
} 

Answer 1

Je ne vois pas de problème évident de sécurité MT avec ceci. 'localObject' ne semble pas fuir quelque part permettant aux autres threads de le voir.

Answer 2

Un scénario est pas thread-safe (Tn = filetage n):

T1 appelle someMethod premier; T2 appelle someMethod après cela.

Lors de l'exécution de method2 sur T1, la référence à localObject est modifiée par T2 entraînant un comportement indéfini.

Answer 3

Vous semblez faire référence à this article, qui contient presque exactement l'extrait de code que vous présentez. Toutefois, cet article prétend que le code que vous affichez est thread safe.

Donc, il semble que vous avez des raisons d'affirmer que ce code est pas thread sûr, mais ne nous ont pas dit la raison de cette assertion.

Pour le code, vous affichez la seule vulnérabilité de threading si LocalObject lui-même a des problèmes de threads lui-même, par exemple s'il a des éléments de données statiques qui ne sont pas protégés correctement.

Je pense que peut-être vous demandez ce genre d'exemple

public class UnsafeClass{ 
    static LocalObject bestWeHaveSeen = new LocalObject(); 

    public void someMethod(){ 

     LocalObject localObject = new LocalObject(); 

     localObject.callMethod(); 

     // localObject is not threadsafe 
     method2(localObject); 
    } 

public void method2(LocalObject localObject){ 
    if (localObject.getValue("value") > bestWeHaveSeen.getValue("value")){ 
     bestWeHaveSeen = localObject; 
    } 

    // some code here to compute an important value 
    bestWeHaveSeen.setValue(computedValue); 
} 

Ceci est maintenant pas thread-safe. Comme deux threads différents peuvent accéder bestWeHaveSeen

Answer 4

Voici un exemple comment faire pas threadsafe:

private List<LocalObject> list = new ArrayList<LocalObject>(); 
public List<LocalObject> getList(){return list}; 

public void someMethod(){ 

    LocalObject localObject = new LocalObject(); 

    localObject.callMethod(); 

// localObject is not threadsafe 
    method2(localObject); 
} 

public void method2(LocalObject localObject){ 
    list.add(localObject); 
    localObject.setValue("value"); 
} 

method2 a été modifié pour stocker le localObject d'une manière qui permet à d'autres threads d'accéder - vers une liste publique accessible. Un autre thread pourrait maintenant obtenir l'objet de la liste et définir la valeur en même temps. Donc, même si nous travaillons avec un objet local, il n'est plus thread-safe et nous devons penser à la synchronisation.