Discussion :

[®om]

Salut,

J'ai une classe qui définit entre autres 2 méthodes:

Code :

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

1
2
public boolean canPlay(...) { ... }
public void play(...) {...}

Pour faire play(...) il faut tester d'abord canPlay(...).

Ayant plusieurs threads pouvant appeler ces méthodes, je cherche une solution pour être sûr de conserver un état "stable".

Code :

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

1
2
public synchronized boolean canPlay(...) { ... }
public synchronized void play(...) {...}

Si je rajoute le mot-clé "synchronized", il y aura toujours un problème, car aucune intervention d'un autre thread ne doit s'éxécuter entre canPlay() et play() (sinon le résultat de canPlay() risque de changer).

Avez-vous une solution?

EDIT: Je n'ai pas trouvé la DEUXIÈME PARTIE de ce tuto:
http://alwin.developpez.com/tutorial/JavaThread/

[herve91]

Bonsoir,
peut-être qq chose comme ça :

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
private boolean playable;
 
public synchronized boolean canPlay(...) {
  boolean canPlay = false;
  if (!playable) {
    playable = canPlay = canPlayImpl(...);
  }
  return canPlay;
}
 
private boolean canPlayImpl(...) {
}
 
public synchronized void play(...)  {
  if (playable) {
    ...
    playable = false;
  }
}

[the-gtm]

Il faut que le thread prenne le verrou sur l'objet, execute canPlay() puis (éventuellement) play() et enfin libère le verrou.

Tu peux pour ça créer un bloc de code synchronisé sur l'objet (que j'appelle player) :

Code :

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

1
2
3
4
5
6
 
synchronized(player) {
   if (player.canPlay()) {
      player.play();
  }
}

Dans ce cas plus besoin que ces deux méthodes soient elles même synchronisées.