Discussion :

[gl0be]

Bonjour,

Lorsque j'exécute le processus "cmd" et que je récupère son inputStream je peux accéder aux donnée reçues. Par contre quand j'utilise un autre exécutable impossible de récupéré ce que renvoie l'inputStream (à la main il affiche bien qqch) voici le 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
 
ProcessBuilder b = new ProcessBuilder(	"C:/openocd-2007re204/bin/openocd-ftd2xx",
										"-f",
										"D:/at91r40008_jtagkeyARMEBS.cfg") ;
 
//ProcessBuilder b = new ProcessBuilder("cmd") ;
 
try 
{
	Process process = b.start() ;
	BufferedReader ds = new BufferedReader(new InputStreamReader(process.getInputStream()));
 
	while (!ds.ready()){}
 
	String lineCourante = ds.readLine() ;
	while (lineCourante != null) {
	System.out.println(lineCourante);
	lineCourante = ds.readLine();
	}
}
catch (Exception e8){System.out.println(e8.getMessage());}

Avec l'exécutable 'openOCD' la boucle while (!ds.ready()){} devient infinie, par contre avec 'cmd' la console affiche bien le résultat.

peut-être l'autre exécutable de gère pas bien son stdOut ?

Merci de vos commentaires.

gl0be

[adiGuba]

Salut,


A mon avis tu peux déjà supprimer la boucle sur le ready() qui est non seulement complètement inutile mais également potentiellement problématique car il peut ne pas être implémenté (bref c'est juste à titre informatif).

De plus c'est une attente active qui risque de prendre tout le temps CPU pour rien, alors que le readLine() ou n'importe quel read() bloquera proprement s'il n'y a pas de données à lire...


De plus tu ne libères pas tes ressources !!! Comment libérer proprement les ressources ?


Enfin il faut impérativement traiter TOUS les flux, même stdin et stderr (surtout que ce dernier contient les éventuels messages d'erreurs du process), ou dans le pire des cas tu dois fermer ces flux... sinon tu risque un interblocage entre ton programme et le process lancé (chacun s'attend mutuellement).

Plus d'info :

• Bonnes pratiques pour lancer des applications
• Runtime.exec() n'est pas des plus simple...

Bonne lecture

a++

[gl0be]

Merci beaucoup de ta réponse très complète ! :-)

[adiGuba]

De rien

PS : pour la prochaine fois, tu peux utiliser directement le bouton en bas de page plutôt que d'éditer le titre du premier message

a++