Discussion :

[Djobird]

Bonjour,

Dans une appli, un serveur va à chaque requête recu d'un client générer un thread pour la traiter.
Ce thread utilise entre autre l'instruction suivante :

Code :

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

1
2
 
XPathFactory factory = XPathFactory.newInstance();

XPathFactory.newInstance() n'étant pas synchronisé, il faut donc que je le fasse à mon niveau. Seulement je n'ai pas de mutex général.
JE voulais donc savoir, si effectuer

Code :

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

1
2
3
 
synchronised(XpathFactory.class)
        factory = XPathFactory.newInstance();

[/code]

permettait cette synchro ? Ou s'il y avait plus simple/propre ?
Utiliser par exemple une variable de class static servant de mutex est il plus propre ?

[adiGuba]

Salut,


Tu peux très bien utiliser la classe comme mutex. C'est même très propre à mon avis !
D'ailleurs c'est ce qui est utilisé lorsque tu définis une méthode static synchronized...


Par contre je ne pense pas qu'il y ait besoin de synchronisé les newInstance() !

a++

[Djobird]

D'accord merci.
Par contre pourquoi penses tu qu'une synchro n'est pas nécessaire ?

Je ne maitrise pas encore tous les concepts de synchro, mais quand je regarde le code de la fonction newInstance, elle me ne semble pas thread safe, si ?

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
 
 
 public static final XPathFactory newInstance(final String uri)
        throws XPathFactoryConfigurationException {
 
        if (uri == null) {
        	throw new NullPointerException(
        		"XPathFactory#newInstance(String uri) cannot be called with uri == null"
        	);
        }
 
		if (uri.length() == 0) {
			throw new IllegalArgumentException(
				"XPathFactory#newInstance(String uri) cannot be called with uri == \"\""
			);
		}
 
		ClassLoader classLoader = ss.getContextClassLoader();
 
        if (classLoader == null) {            
            //use the current class loader
            classLoader = XPathFactory.class.getClassLoader();
        } 
 
		XPathFactory xpathFactory = new XPathFactoryFinder(classLoader).newFactory(uri);
 
		if (xpathFactory == null) {
			throw new XPathFactoryConfigurationException(
				"No XPathFactory implementation found for the object model: "
				+ uri
			);
		}
 
		return xpathFactory;
    }

[tchize_]

Qu'est-ce qui te fait penser que cette méthode n'est pas ThreadSafe. Elle ne modifie que des variables locales, et ne fait que des appels à new ou des appels au classloader.

[adiGuba]

Qu'est-ce qui te fait penser que cette méthode n'est pas ThreadSafe. Elle ne modifie que des variables locales, et ne fait que des appels à new ou des appels au classloader.

+1


Un code thread-safe n'implique pas forcément une synchronisation.


a++

[Kirua12]

Salut,

la méthode newInstance() te renvoie une nouvelle instance (comme son nom l'indique) mais pas la même instance à chaque appel (ce n'est pas un singleton).
Il n'y a pas besoin de synchroniser cette méthode pour ce fonctionnement.
Après si tu veux un singleton, c'est à toi de gérer la création de cette instance mais aussi de vérifier que la classe XPathFactory est thread-safe