Discussion :

[grunk]

Bonjour à tous,

Je développe une application android qui se compose de plusieurs Thread.
L'un de ces Thread est appelé comme ceci :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

MyThread thread = new MyThread(handler,monObjet);

monObjet est un objet déclaré dans mon activité principale.
Selon les données reçue par le Thread (via socket) il va modifier une des propriété de monObjet.
La modification de monObjet est donc répercuté dans l'activité principale.

Sachant que monObjet peut être utilisé (mais pas modifié) dans d'autre thread dois je mettre en place une sécurité particulière dans mon thread lors de la modification de mon objet ?

Est ce correcte de se reposer sur le passage par référence de l'objet pour les modification ou devrais je faire quelque chose de plus explicite comme par exemple un setter dans l'activité qui recevrait l'objet modifié par le thread ?

Merci

[Rei Ichido]

Sur la question de la synchronisation, cela dépend fortement des risques liés à la modification de ton objet.

Imaginons par exemple que monObjet contienne une liste non synchronisée (genre, ArrayList). Dans un thread "utilisateur", tu entames une itération, mais ton thread modificateur rajoute un élément. Pam, fail-fast, ton thread utilisateur t'envoie une ConcurrentModificationException.

Donc à moins d'être certain que tu n'auras pas ce genre de configuration, il va te falloir synchroniser !

[grunk]

Merci de ta réponse.

Mon thread modificateur ne fait que modifier un boolean donc a priori pas de problème de ce côté.

[thelvin]

Dans ce cas, pas besoin de "sécurité" particulière. Mais les modifications de l'objet ne seront pas forcément visibles d'un thread à l'autre s'il n'y a pas une forme de synchronisation.

Si la seule chose qui concerne les deux threads est ce fameux booléen, alors il suffit de le déclarer volatile : dans ce cas le compilateur ne gardera jamais sa valeur en cache et retournera le chercher dans l'état de la mémoire, à chaque lecture. Ce qui rend les changements visibles pour tous les threads.