Discussion :

[Le Marlou]

Bonjour,
J'ai un client lourd Java qui peut être lancé en double-cliquant sur un jar exécutable. Pour diverses raisons, je veux que ce client ne puisse pas être lancé plusieurs fois en même temps (pour faire bref : l'exécution de l'interface plusieurs fois en parallèle peut provoquer des comportements bizarres, voire le plantage de l'application).
Pour cela, j'ai mis en place la solution suivante :
- Au premier lancement, l'application crée un fichier monappli.log (dans windows\temp par exemple), dans lequel je mets juste la date et l'heure de lancement de l'appli
- Lorsque l'appli fermée en cliquant sur la croix en haut à droite, le fichier est supprimé comme ceci:

Code :

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

1
2
3
4
5
6
7
8
9
 
public void processWindowEvent(WindowEvent e)
{
   if (e.getID() == WindowEvent.WINDOW_CLOSING)
   {
       // suppression du fichier
   }
   super.processWindowEvent(e);
}

- A chaque démarrage de l'application, on vérifie si ce fichier existe, si c'est le cas, c'est que l'appli est déjà lancée et j'affiche un message adéquat.

Le problème est que cette solution ne marche que si on ferme l'application "normalement" (via la croix en haut à droite). Si l'application est lancée par un script (qui exécute java -jar monjar.jar) et que ce script est stoppé, le fichier monappli.log ne sera pas effacé. De même, si on kill violemment le processus, le fichier ne sera pas effacé. Du coup, au redémarrage de l'appli, ça dit qu'elle est déjà lancée alors que ce n'est pas le cas.
C'est très moyen. Auriez-vous une autre solution à proposer ?
Merci d'avance
Le marlou

[fr1man]

De mémoire, tu as des techniques expliquées dans la FAQ.

[Le Marlou]

j'ai regardé dans le FAQ Java GUI mais je n'ai rien trouvé qui corresponde à mon problème.

[fr1man]

http://java.developpez.com/faq/java/...nceApplication

Tu peux également adapter ta méthode de lock de fichier en utilisant la méthode addShutdownHook() de la classe Runtime. Ce code sera alors appelé même en cas de fermeture brutale de ton application.

[adiGuba]

Salut,


Tu peux tout simplement utiliser deleteOnExit() juste après la création du fichier. Comme cela il sera supprimé à la fermeture du programme quoi qu'il arrive (sauf méchant plantage de la JVM).


Sinon tu peux aussi utiliser les FileLock...

a++

[edit] PS : mais la solution de la socket permet un dialogue simple entre les deux applications. On peut par exemple passer les arguments de la nouvelle application à l'application d'origine afin qu'elle les traitent...

[Le Marlou]

merci pour vos réponses. alors :
1) pour la solution avec la socket, ça marche bien mais le fait de devoir utiliser un port ne n'emballe pas et je suis a peu pres sur que ça sera critiqué lors du prochain audit de sécurité sur notre produit.
2) le deleteOnExit() marche bien mais pas quand on kille violemment le processus java
3) idem pour le addShutdownHook(), ça marche bien mais pas quand on kille violemment le processus java
4) Je n'ai pas besoin de FileLock dans mon cas.

Donc pour résumer les 2 solutions deleteOnExit() et addShutdownHook() me plaisent bien mais ne marchient pas si on kille le processus java.
Voici mon code avec le hook :

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
 
public class ShutdownThread extends Thread 
{
   private File file;
 
   ShutdownThread(File file) 
   {
      this.file = file;
   }
 
   public void run() 
   {
      if (file != null && file.exists())
      {
         file.delete();
      } 
   }
}

et l'init dans ma classe principale :

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
 
   ...
   final File file = new File(filename);
   if (file.exists())
   {
      // l'appli est deja lancée
      isStarted = true;
   }
   else
   {
      // creation du fichier
      ...
      Runtime.getRuntime().addShutdownHook(new ShutdownThread(file));
   }
   ...

Si qq'un a une idée pour supprimer le fichier à coup sûr qd l'appli est arrêtée (de manière correcte ou pas)...

[Uther]

4) Je n'ai pas besoin de FileLock dans mon cas.

Je ne suis pas sur que tu aie compris ce qu'adiGuba a proposé.

En fait l'idée est de faire un lock sur un fichier quelconque de la même manière que l'on ouvre un socket. Si le fichier est déjà verrouillé, c'est que l'application a déjà été lancée.
Si l'application est tuée violemment le système va lever le verrou sur le fichier.

[Le Marlou]

Je ne suis pas sur que tu aie compris ce qu'adiGuba a proposé.
En fait l'idée est de faire un lock sur un fichier quelconque de la même manière que l'on ouvre un socket. Si le fichier est déjà verrouillé, c'est que l'application a déjà été lancée.
Si l'application est tuée violemment le système va lever le verrou sur le fichier.

Bonjour,
J'ai essayé le FileLock avec ce code :

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
 
      ...
      final File file = new File(filename);
      try
      {
         final FileChannel channel = new RandomAccessFile(file, "rw").getChannel();
         // Use the file channel to create a lock on the file.
         // This method blocks until it can retrieve the lock.
         FileLock lock = channel.lock();
         // Try acquiring the lock without blocking. This method returns
         // null or throws an exception if the file is already locked.
         try
         {
            lock = channel.tryLock();
         }
         catch (OverlappingFileLockException e)
         {
            isStarted = true;
         }
      }
      catch (Exception e)
      {
         logger.error("error while lock file : " + e.getMessage());
      }

Mais il passe tout le temps dans le catch OverlappingFileLockException, alors que je viens juste de créer le fichier...

[adiGuba]

Tu lockes deux fois le fichier. Une fois avec lock() et une autre fois avec tryLock()...

Utilises seulement tryLock() !

Code :

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

1
2
final FileChannel channel = ...
lock = channel.tryLock();

a++

[Le Marlou]

Tu lockes deux fois le fichier. Une fois avec lock() et une autre fois avec tryLock()...

Utilises seulement tryLock() !

Code :

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

1
2
final FileChannel channel = ...
lock = channel.tryLock();

a++

Si je fais juste le tryLock, il ne passe jamais dans le catch OverlappingFileLockException...

[Uther]

C'est normal car si le fichier est locké, il n'y a pas de OverlappingFileLockException, tryLock() retourne juste null

[Le Marlou]

ah ok, c'est bon ça marche. Donc pour résumer, il ne faut pas utiliser la méthode lock() mais plutôt tryLock(), et ne pas utiliser la OverlappingFileLockException. Il faut juste regarder si tryLock renvoie null ou pas :

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
 
      .....
      final File file = new File(filename);
      try
      {
         final FileChannel channel = new RandomAccessFile(file, "rw").getChannel();
         final FileLock lock = channel.tryLock();
         if (lock == null)
         {
            // le fichier est déjà locké
            channel.close();
            isStarted = true;
         }
         else
         {
            // je peux démarrer mon appli
            .......
         }
      }
      catch (Exception e)
      {
         .......
      }

Merci à tous pour vos réponses.