appel de méthode synchronized

Bonjour,

j'ai une méthode :

Code:

---

1 2 3 4

public synchronized void changeNote( int index, String note ) { bodyList.set( index, note ); }

---

j'ai lancer mon application web qui contient se code dans le serveur et j'ai ouvert deux navigateur j'ai connecter un utilisateur dans chaque navigateur et je fait appel dans chaque navigateur.

normalement l'accès à cette méthode est exclusif ( si un utilisateur est entraint d'exécuter cette méthode l'autre ne peut l'exécuter que si le premier à fini ) ( si je comprend bien la notion de synchronized)

dans mon cas j'ai lancer le serveur en mode debug j'ai mis un point d'arret sur la première instruction de cette méthode j'ai exécuter la méthode avec le premier utilisateur (donc le thread de cette utilisateur possède le verrou sur cette méthode), puis j'ai enlevé le point d'arret et j'ai lancé la méthode avec le second utilisateur " normalement il reste en attente d'exécuter cette méthode mais c'est pas le cas l'exécution continue normalement.

est ce que ma comprérhension de la notion de synchronized est fausse?
sinon comment assurer la synchronisation de la contention du cette méthode.

merci d'avance

Ta notion de ce qu'est une méthode synchronized est fausse ;)

Une méthode synchronized garanti simplement que quand on l'exécute sur un objet, aucune autre méthode synchronized ne pourra être utilisée *sur le même objet*. Le but étant de prévenir des accès concurrents sur un même objet, avec notamment des modifications sur l'état d'un objet qui feraient bugger une autre méthode (par exemple, sur une liste si tu retires des objets alors qu'une autre méthode est en train de la parcourir, tu peux prendre un risque de tomber sur un index hors limite).

Par conséquent, si ta méthode s'applique sur des objets différents, ce n'est pas près de marcher ^^

Citation:

---

Envoyé par Rei Ichido

Ta notion de ce qu'est une méthode synchronized est fausse ;)

Une méthode synchronized garanti simplement que quand on l'exécute sur un objet, aucune autre méthode synchronized ne pourra être utilisée *sur le même objet*. Le but étant de prévenir des accès concurrents sur un même objet, avec notamment des modifications sur l'état d'un objet qui feraient bugger une autre méthode (par exemple, sur une liste si tu retires des objets alors qu'une autre méthode est en train de la parcourir, tu peux prendre un risque de tomber sur un index hors limite).

Par conséquent, si ta méthode s'applique sur des objets différents, ce n'est pas près de marcher ^^

---

merci pour la réponse.

dans mons cas par exemple la méthode

Code:

---

1 2 3 4 5 6 7

public void deleteNote( int index ) {   titleList.remove( index ); bodyList.remove( index ); } }

---

est entrain d'etre exécuter par un autre utilisateur alors qu'un premier est entrain de modifier l'index j'aurais dans ce cas une erreur d'exécution alors que la méthode changenote est synchronized et utilise l'objet bodylist

Comment je peut eviter ce confilit ?

merci

Citation:

---

Envoyé par Sfaxiano

Comment je peut eviter ce confilit ?

---

En utilisant synchronized et le même objet pour tous les utilisateurs.

Citation:

---

Envoyé par thelvin

En utilisant synchronized et le même objet pour tous les utilisateurs.

---

mon problème c que chaque utilisateur crée une instance de ces objets (j'ai vérifier cela avec le mode debug)

est ce que l'utilisation static de ces objets peut résoudre le problème ?

merci

Les objets que créent les utilisateurs, s'ils en créent un chacun ou un pour tous, tout ça est censé être sous ton contrôle.

C'est à toi de décider ce qui est commun à tous les utilisateurs et ce qui est créé pour chaque. Ton objectif est que des appels à une méthode soient fait en exclusion mutuelle. Le mot-clé synchronized est fait pour ça, mais il ne fait l'exclusion mutuelle que sur un objet, ou sur une classe pour les méthodes statiques.

À partir de là, à toi de voir la solution qui s'adapte à ton problème. On ne va pas faire ton travail.

Pour information, static peut plus ou moins aider au problème, mais on en conseille pas en général. Il vaut mieux créer un objet commun à tous les utilisateurs.

Citation:

---

Envoyé par thelvin

Les objets que créent les utilisateurs, s'ils en créent un chacun ou un pour tous, tout ça est censé être sous ton contrôle.

C'est à toi de décider ce qui est commun à tous les utilisateurs et ce qui est créé pour chaque. Ton objectif est que des appels à une méthode soient fait en exclusion mutuelle. Le mot-clé synchronized est fait pour ça, mais il ne fait l'exclusion mutuelle que sur un objet, ou sur une classe pour les méthodes statiques.

À partir de là, à toi de voir la solution qui s'adapte à ton problème. On ne va pas faire ton travail.

Pour information, static peut plus ou moins aider au problème, mais on en conseille pas en général. Il vaut mieux créer un objet commun à tous les utilisateurs.

---

Merci pour la réponse

le problème c'est que c'est pas que j'ai écrit la solution et pour la comprendre il me faux beaucoup de temps, c'est pour cela je cherche la solution la plus rapide qui concerne juste ma tache.

merci une autre fois

C'est une chose que je peux comprendre, mais nous ne savons pas à quoi ressemble ton programme. On ne peut donc pas voir comment il est architecturé et quelle est la méthode la plus rapide.

Le plus rapide, je pense, est de jouer un peu avec le mot-clé synchronized pour s'y habituer, et voir comment intégrer ça dans ton projet une fois que tu connais le sujet. (Soit dit en passant, c'est comme ça pour tout ce qui n'est pas trivial.)

Ca ressemble a un besoin pour l'utilisation d'un singleton cette affaire.

Tu crés ton singleton au démarrage du serveur par exemple et les utilisateurs se partage l'objet, si toutes les méthodes de cet objet sont synchronisées, il n'y a plus de soucis.

tuto ici : http://christophej.developpez.com/tu...n/multithread/

c'est quoi le but? Ton moyen c'est d'empecher deus personnes d'utiliser ta méthode en même temps, mais quel est le but recherché? A partir de ça, tu saura déterminer quel est le meilleur moyen d'y arriver (singleton, méthode synchronisée, lock, etc)

Citation:

---

Envoyé par tchize_

c'est quoi le but? Ton moyen c'est d'empecher deus personnes d'utiliser ta méthode en même temps, mais quel est le but recherché? A partir de ça, tu saura déterminer quel est le meilleur moyen d'y arriver (singleton, méthode synchronisée, lock, etc)

---

mon objectif c'est d'éviter les conflits, les objets bodyList et titleList soit à accès exclusif par exemple si un utilisateur est entrain de modifier une note (modifier l'état des objets bodyList ou titleList) tous les autres utilisateur qui accède a cette note seront mise en attente jusqu'à ce que cette modification soit fini.

j'ai essayer avec synchronized mais dés fois je tombe sur un deadlock.

merci d'avance

Citation:

---

Envoyé par Jimmy_

Ca ressemble a un besoin pour l'utilisation d'un singleton cette affaire.

Tu crés ton singleton au démarrage du serveur par exemple et les utilisateurs se partage l'objet, si toutes les méthodes de cet objet sont synchronisées, il n'y a plus de soucis.

tuto ici : http://christophej.developpez.com/tu...n/multithread/

---

merci pour l'article c'est intéressent.:ccool:

Citation:

---

Envoyé par Sfaxiano

j'ai essayer avec synchronized mais dés fois je tombe sur un deadlock.

merci d'avance

---

Ca c'est toujours le probleme quand on joue avec des verrous. Le meilleur moyen d'éviter ca, c'est de soit prendre un seul verrou global pour l'ensemble des opération (mais peut impacter les performances) soit de prendre des verrous dans un ordre précis que tout le monde respectera.

Si A a besoin de verrous 1 5 3 2 et B a besoin des verrous 3 9 2 1, il faut prendre les verrous strictement dans l'orre 1 2 3 5, et 1 2 3 9. Pour faire ca, faut savoir a l'avance dans ton traitement de quel verrous tu aura besoin pour les ordonner et les prendre....