Discussion :

[NicoTheDude]

Bonjour,

Contexte : Application Web JAVA / Serveur Tomcat

Je suis étonné de la lenteur d'exécution des commandes lancées par mon appli avec la méthode Runtime.exec()
En effet, une commande lancée en shell directement me prend trois secondes au lieu de 10 lors de l'exécution du programme JAVA.

Auriez vous une explication voire une solution pour améliorer la vitesse d'exécution de ma commande shell?

Merci

[sinok]

Gères tu correctement les flux standard d'entrée/sortie/erreur de ton Runtime.exec (sont ils bien consommés)?

http://ydisanto.developpez.com/tutoriels/j2se/runtime/
http://blog.developpez.com/adiguba/p...s-plus-simple/

[NicoTheDude]

Est-il nécessaire de traiter ces flux?
Je lance simplement l'exec puis j'attends que le Process soit terminé avec waitfor() pour le moment, je ne m'occupe pas de ce qui se passe en console.

[NicoTheDude]

J'ai ajouté le traitement des flux comme indiqués dans tes liens et j'obtiens toujours le même temps. J'ai pourtant bien deux Threads séparés qui traitent le flux standard et le flux d'erreur.
Le problème ne semble donc pas venir de là mais ca semblait etre une bonne idée, merci.

[adiGuba]

Salut,


Il faut TOUJOURS traiter les flux (ou les fermer) sous peine d'avoir un dead-lock entre les deux programmes.

Quand à ton problème, il faudrait en savoir plus sur ta commande ? Comment tu la lances dans le shell, le code java correspondant, ce qu'elle fait, etc...


a++

[NicoTheDude]

Bonjour,

Voilà le morceau de code principal qui me sert à lancer un commande.
J'ai donc deux threads pour gérer les flux :

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
final Process p;
        try {
            // Consommation de la sortie standard et de la sortie d'erreur des scripts
            p = Runtime.getRuntime().exec(command);
            new InputThread(p.getInputStream(), session).start();
            new InputThread(p.getErrorStream(), session).start();
            p.waitFor();
            exitValue = p.exitValue();
        }
        catch (IOException ex) {
            java.util.logging.Logger.getLogger(ExecScript.class.getName()).log(Level.SEVERE, null, ex);
        }
        catch (InterruptedException ex) {
            //ex.printStackTrace();
            logger.logTraceMessage(session,"<h5 class=\"error\">Unexpected error while executing" + StringEscapeUtils.escapeHtml(command.toString()) + "</h5> \n");
            exitValue = 1;
            return;
        }

Je lance une commande shell (donc un script ".sh") via Java en utilisant le code ci-dessus qui est dans une fonction que j'appelle a chaque fois que je veux exécuter un script.
la commande shell elle-même appelle un script en C...

Est ce que cela vous suffit comme info? Voyez vous des pistes de réflexion sur lesquelles me pencher?

Merci

[adiGuba]

Je lance une commande shell (donc un script ".sh")

Lequel ? Et que fait-il exactement ! Cela peut rentrer en compte.



Sinon pour le code :

• Il faut fermer les flux inutilisés (getOutputStream() dans ton cas).
• Depuis Java 5 il faut privilégier ProcessBuilder à Runtime.exec(), en séparant bien les différents paramètres de la commande.
• Pas besoin de créer forcément deux threads : tu peux lire un des flux dans le thread courant
• Mieux avec ProcessBuilder tu peux cumuler les flux de sorties standard et d'erreur avec redirectErrorStream() pour n'en avoir qu'un seul à lire sans avoir besoin de créer de thread supplémentaire...

Bref ton code pourrait ressembler à ceci :

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
	final Process p = new ProcessBuilder("cmd", "/C", "dir")
		.redirectErrorStream(true)
		.start();
 
	// On ferme les flux inutiles :
	p.getOutputStream().close();
	p.getErrorStream().close();
 
	InputStream input = p.getInputStream();
	try {
		// Copie sur stdout :
		byte[] buf = new byte[8192];
		int len;
		while ( (len=input.read(buf)) >= 0 )
			System.out.write(buf, 0, len);
	} finally {
		input.close();
	}
	p.waitFor();

a++