Execution de fichiers .bat sous java

Bonjour,

J'ai un programme sous format de deux fichier ".bat" et je veux l'exécuter à travers une application Java. Les détails des deux fichiers (setgen.bat et XMLGenerator.bat) sont les suivants:

1. Fichier "setgen.bat":

Code:

---

1 2 3 4 5 6 7 8 9

@echo off echo this release was tested with JDK 1.2, XML4Java 2_0_15. if "%xmlgen-root%" == "" for /f %%f in ('cd') do set xmlgen-root=%%f if "%xmlgen-classpath%" == "" set xmlgen-classpath=%classpath% set classpath=%xmlgen-root%\XMLGenerator.jar;%xmlgen-root%\xml4j.jar;%xmlgen-root%\DDbe.jar;%xmlgen-root%\jfaceall.jar;%xmlgen-classpath% if "%xmlgen-savedpath%" == "" set xmlgen-savedpath=%path% set path=%xmlgen-root%;%xmlgen-savedpath% echo Your classpath is %classpath%

---

2. Fichier "XMLGenerator.bat":

Code:

---

1 2 3 4 5 6 7 8

@echo off if ""=="%xmlgen-root%" goto fail java com.ibm.XMLGenerator.XMLGenerator %1 %2 %3 %4 %5 %6 %7 %8 %9 goto done :fail echo please run setgen.bat from the install directory. :done

---

Le premier fichier a pour but de définir certaines variables d'environnement qui sont utilisées par le deuxième. Le deuxième lance un certain fichier jar. Sous un terminal, je lance setgen.gat et puis XMLGenerator param_1 param_2....param_n et ça s'exécute sans aucun problème.

Par contre je ne sais pas comment faire exécuter ces deux fichiers en utilisant un programme java.

J'ai essayé avec le code suivant:

Code:

---

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

private void executeCommand(String directory){ try { //Execution du premier fichier. File dir = new File(directory); Process p = Runtime.getRuntime().exec("cmd /c setgen", null, dir); p.waitFor(); BufferedReader reader=new BufferedReader(new InputStreamReader(p.getInputStream())); String line1=reader.readLine(); while(line1 != null){ System.out.println(line1); line1=reader.readLine(); }   //Execution du deuxième fichier. p = Runtime.getRuntime().exec("cmd /c XMLGenerator param1 param2", null, dir); p.waitFor(); reader=new BufferedReader(new InputStreamReader(p.getInputStream())); line1=reader.readLine(); while(line1 != null){ System.out.println(line1); line1=reader.readLine(); } } catch (Exception ex){ System.err.println("Error: "+ex.getMessage()); } }

---

Après l'exécution de cette méthode j’obtiens le message d'erreur: "please run setgen.bat from the install directory.", puisque chacun des deux fichier s'exécute dans un terminal différent de l'autre, et donc les variables d'environnement crées par le premier fichier ne sont plus utilisées par le deuxième, donc ce sont deux processus différents et indépendant ce qui n'est plus le cas du terminal où les variables crées peuvent être utilisées tant que le terminal est encore ouvert.

Y a t-il une astuce pour combiner les deux fichiers ensemble ? ou pour les exécuter dans un même processus ?

Merci d'avance.

C'est bon j'ai trouvé la solution, il fallait juste combiner les deux commandes avec un "&". La méthode devient donc comme suit:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16

private void executeCommand(String directory){ try { File dir = new File(directory); Process p = Runtime.getRuntime().exec("cmd /c setgen.bat & XMLGenerator param1 param2", null, dir); p.waitFor(); BufferedReader reader=new BufferedReader(new InputStreamReader(p.getInputStream())); String line1=reader.readLine(); while(line1 != null){ System.out.println(line1); line1=reader.readLine(); } } catch (Exception ex){ System.err.println("Error: "+ex.getMessage()); } }

---

N'hésitez pas à me faire part de vos remarques.

Merci et bonne journée.

Salut,

Citation:

---

Envoyé par Laurent_3154

N'hésitez pas à me faire part de vos remarques.

---

J'ai remarqué plusieurs petits problèmes dans ton code :

• Tu utilises waitFor() juste après avoir lancer ton process.
  Du coup ton code va bloquer tant que le process ne sera pas terminé.
  
  C'est problématique car tu lis son flux de sortie après.
  Si les données envoyées sont de taille réduite ca passera sans problème, mais si le process écrit "trop" de données cela va remplir le buffer entre ton process et ton appli java, et tu va te retrouver dans un double-lock :
  
  • Ton appli Java attendra la fin du process (waitFor()).
  • Ton process sera bloqué en écriture à cause du buffer plein.
  
  
  :arrow: Il faut bien entendu utiliser waitFor() après la lecture du flux de sortie.
  
• Tu ne traites pas les flux d'entrée et d'erreur du process. Encore une fois tu risques de bloquer le process s'il les utilise.
  Il faut soit les traiter (en parallèle), soit les fermer (ce qui évitera les problèmes de blocage).

Bref pour moi ton code devrait plutôt ressembler à ceci :

Code:

---

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

private void executeCommand(String directory) { try { File dir = new File(directory); Process p = Runtime.getRuntime() .exec("cmd /c setgen.bat & XMLGenerator param1 param2", null, dir); try { p.getOutputStream().close(); p.getErrorStream().close();   BufferedReader reader = new BufferedReader( new InputStreamReader(p.getInputStream())); String line1 = reader.readLine(); while (line1 != null) { System.out.println(line1); line1 = reader.readLine(); } p.waitFor(); } finally { // perso j'aime bien forcé le destroy "au cas où" p.destroy(); }   } catch (Exception ex) { System.err.println("Error: " + ex.getMessage()); } }

---

Et si tu utilises Java 5.0 ou plus, il serait préférable d'utiliser ProcessBuilder à la place de Runtime.exec(), cela permet entre autre de séparer proprement les arguments de la commande, et surtout de rediriger le flux d'erreur dans le flux standard (ce qui permet de ne traiter qu'un flux sans perdre d'éventuel message d'erreur) :

Code:

---

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

private void executeCommand(String directory) { try { Process p = new ProcessBuilder("cmd", "/c", "setgen.bat & XMLGenerator param1 param2") .directory(new File(directory)) .redirectErrorStream(true) // on redirige le flux d'erreur vers le flux de sortie .start(); try { p.getOutputStream().close(); p.getErrorStream().close();   BufferedReader reader = new BufferedReader( new InputStreamReader(p.getInputStream())); String line1 = reader.readLine(); while (line1 != null) { System.out.println(line1); line1 = reader.readLine(); } p.waitFor(); } finally { // perso j'aime bien forcé le destroy "au cas où" p.destroy(); }   } catch (Exception ex) { System.err.println("Error: " + ex.getMessage()); } }

---

Et avec Java 7 on peut même utiliser inheritIO() pour hériter des flux de l'application Java, et du coup plus besoin de les gérer :

Code:

---

1 2 3 4 5 6 7 8 9 10 11

private void executeCommand(String directory) { try { Process p = new ProcessBuilder("cmd", "/c", "setgen.bat & XMLGenerator param1 param2") .directory(new File(directory)) .inheritIO() .start(); p.waitFor(); } catch (Exception ex) { System.err.println("Error: " + ex.getMessage()); } }

---

Note : on peut même paramétrer individuellement chaque flux avec les méthodes redirectXXX()


a++

Citation:

---

Envoyé par adiGuba

Salut,



J'ai remarqué plusieurs petits problèmes dans ton code :

• Tu utilises waitFor() juste après avoir lancer ton process.
  Du coup ton code va bloquer tant que le process ne sera pas terminé.
  
  C'est problématique car tu lis son flux de sortie après.
  Si les données envoyées sont de taille réduite ca passera sans problème, mais si le process écrit "trop" de données cela va remplir le buffer entre ton process et ton appli java, et tu va te retrouver dans un double-lock :
  
  • Ton appli Java attendra la fin du process (waitFor()).
  • Ton process sera bloqué en écriture à cause du buffer plein.
  
  
  :arrow: Il faut bien entendu utiliser waitFor() après la lecture du flux de sortie.
  
• Tu ne traites pas les flux d'entrée et d'erreur du process. Encore une fois tu risques de bloquer le process s'il les utilise.
  Il faut soit les traiter (en parallèle), soit les fermer (ce qui évitera les problèmes de blocage).

Bref pour moi ton code devrait plutôt ressembler à ceci :

Code:

---

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

private void executeCommand(String directory) { try { File dir = new File(directory); Process p = Runtime.getRuntime() .exec("cmd /c setgen.bat & XMLGenerator param1 param2", null, dir); try { p.getOutputStream().close(); p.getErrorStream().close();   BufferedReader reader = new BufferedReader( new InputStreamReader(p.getInputStream())); String line1 = reader.readLine(); while (line1 != null) { System.out.println(line1); line1 = reader.readLine(); } p.waitFor(); } finally { // perso j'aime bien forcé le destroy "au cas où" p.destroy(); }   } catch (Exception ex) { System.err.println("Error: " + ex.getMessage()); } }

---

Et si tu utilises Java 5.0 ou plus, il serait préférable d'utiliser ProcessBuilder à la place de Runtime.exec(), cela permet entre autre de séparer proprement les arguments de la commande, et surtout de rediriger le flux d'erreur dans le flux standard (ce qui permet de ne traiter qu'un flux sans perdre d'éventuel message d'erreur) :

Code:

---

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

private void executeCommand(String directory) { try { Process p = new ProcessBuilder("cmd", "/c", "setgen.bat & XMLGenerator param1 param2") .directory(new File(directory)) .redirectErrorStream(true) // on redirige le flux d'erreur vers le flux de sortie .start(); try { p.getOutputStream().close(); p.getErrorStream().close();   BufferedReader reader = new BufferedReader( new InputStreamReader(p.getInputStream())); String line1 = reader.readLine(); while (line1 != null) { System.out.println(line1); line1 = reader.readLine(); } p.waitFor(); } finally { // perso j'aime bien forcé le destroy "au cas où" p.destroy(); }   } catch (Exception ex) { System.err.println("Error: " + ex.getMessage()); } }

---

Et avec Java 7 on peut même utiliser inheritIO() pour hériter des flux de l'application Java, et du coup plus besoin de les gérer :

Code:

---

1 2 3 4 5 6 7 8 9 10 11

private void executeCommand(String directory) { try { Process p = new ProcessBuilder("cmd", "/c", "setgen.bat & XMLGenerator param1 param2") .directory(new File(directory)) .inheritIO() .start(); p.waitFor(); } catch (Exception ex) { System.err.println("Error: " + ex.getMessage()); } }

---

Note : on peut même paramétrer individuellement chaque flux avec les méthodes redirectXXX()


a++

---

Bonjour,

Je vous remercie infiniment, je vais prendre en considération toutes vos remarques pour améliorer mon code.

Cordialement.

Citation:

---

Envoyé par Laurent_3154

Bonjour,

Je vous remercie infiniment, je vais prendre en considération toutes vos remarques pour améliorer mon code.

Cordialement.

---

Rebonjour,

Savez-vous comment exécuter le même code sous Linux ? Y a t-il un moyen pour convertir mes deux fichiers .bat en fichiers sh ?

Merci d'avance.