Discussion :

[Sfaxiano]

Bonjour,

j'ai une méthode :

Code :

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

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

[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 ^^

[Sfaxiano]

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 :

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

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

[thelvin]

Comment je peut eviter ce confilit ?

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

[Sfaxiano]

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

[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.