probleme de recuperation de données

Version imprimable

bonjour,

lorsqu' a partir de mon appli java je souhaite executer une commande windows, je n'arrive pas a recuperer de resultats.

voici des morceaux de code:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
try {
process = Runtime.getRuntime().exec("cmd /K dir");		} catch (IOException e) {
		e.printStackTrace();
	}
 
	try {
		inn=new InputStreamReader(process.getInputStream(),"CP850");
		} catch (UnsupportedEncodingException e) {
			e.printStackTrace();
		}
 
		br= new BufferedReader(inn);
		err = process.getErrorStream();
		readClearCaseClient(buf, br);//ici ca bloque alors qu'avant pas de probleme

Code:

1
2
3
4
5
6
7
8
9
10
11
12
public  void readClearCaseClient(StringBuffer buf,BufferedReader br){
		String ligne;
		try {
			while((ligne=br.readLine())!="hello"  || ligne=br.readLine())!=null ){
				buf.append(ligne).append("\n");
			}
 
		} catch (IOException e) {
			// TODO Auto-generated catch block
			e.printStackTrace();
		}
	}

Voila je vous remercie

14/03/2007, 15h43
in

Je crois qu'il faut mieux utiliser un tableau de String pour les paramètres ... t'as essayé ?

Sinon, quelle est l'erreur, tu parles d'un blocage dans les commentaires, ça tourne en boucle ou ça plante (dans ce cas dis nous quelle est l'erreur) ?
14/03/2007, 15h52
kenny49

ca tourne en boucle la où j'ai mis le commentaire...
14/03/2007, 15h54
kenny49

je viens d'avoir une erreur:

Exception in thread "Thread-0" java.lang.OutOfMemoryError: Java heap space
14/03/2007, 16h02
iohack

dans ton while, c'est un "&&", pas un "||" !
et +1 pour utiliser un tableau de string pour la commande à passer, si tu continues de faire comme ça, ça risque fortement de ne pas fonctionner.
14/03/2007, 16h03
kenny49

je vais redemarrer ca me parait vraiment trop bizarre
14/03/2007, 16h11
iohack

Remarque 1:
Je ne vois de "waitFor" sur ton process dans ton code.
Pour info, j'ai déjà eu des soucis avec cette omission...

Remarque 2:
Si je me rappelle bien, le traitement des flux doit se faire dans des Threads séparés, il doit y avoir un article qui en parle sur ce site ( FAQ, sources... ).
14/03/2007, 17h43
kenny49

Citation:

et +1 pour utiliser un tableau de string pour la commande à passer, si tu continues de faire comme ça, ça risque fortement de ne pas fonctionner.

Comment ca? :oops:

encore plus simple!!

voilà pour résoudre un pb, diminuons le nb de lignes:

voici tout mon code; mon appli ne s'arrete malhereusement pas.

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
 
public class TestCmd {
 
 
	public TestCmd() {
		System.out.println("begin");
		try {
			Runtime r = Runtime.getRuntime();
                        Process p = r.exec("cmd /c dir");
                        p.waitFor();
		}catch(Exception e) {
			System.out.println("erreur d'execution "  + e.toString());
        }
 
 
		System.out.println("fin");
 
	}
 
	public static void main(String[] args) {
		// TODO Auto-generated method stub
		new TestCmd();
	}
 
}

Donc 'begin' s'affiche mais apres on peut toujours attendre... Des idées?

14/03/2007, 22h07
iohack

tu ne m'écoutes pas...
Tableau de string plutot qu'une contenant tout
String [] s={"cmd","arg1","arg2} != String s="cmd arg1 arg2"

OK je t'écoute..

bon j'ai fait comme effectivement j'aurais du faire; cependant le pb n'est toujours pas résolu!!

Je te redonne le code, dis moi ce que tu en penses:
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 package test; public class TestCmd { public TestCmd() { System.out.println("begin"); try { Runtime r = Runtime.getRuntime(); String [] s={"cmd","/c","dir"}; Process p = r.exec(s); p.waitFor(); }catch(Exception e) { System.out.println("erreur d'execution " ); e.printStackTrace(); } System.out.println("fin"); } public static void main(String[] args) { new TestCmd(); } }
Comme d'habitude, 'begin' s'affiche à la console mais après, on peut toujours attendre...

15/03/2007, 21h00
iohack

et en mettant "dir" tout seul sans le "cmd /c" ?
15/03/2007, 21h35
Scorpyosis

Ce n'est pas "vraiment" un problème de code mais plutôt d'agencement. Toi tu t'attends qu'une fois que ta commande est finie d'être exécutée, que le bufferedReader renvoit null et que ton programme continue. Mais voilà pas de bol, le bufferedReader ne renverra jamais null car il n'y a rien qui lui indique qu'il n'y aura plus rien a lire. Dans un fichier c'est facile il y a EOF, mais pour une commande système comment le deviner? Donc comme il a été dit sépare les traitements

si ca peut aider, quand j'enlève le p.waitFor(), mon appli fonctionne correctement.

Rappel du code:

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
28
29
30
31
package test;
 
 
public class TestCmd {
 
 
	public TestCmd() {
		System.out.println("begin");
		try {
			Runtime r = Runtime.getRuntime();
			System.out.println("r="+r);
			String [] s={"cmd","/c","dir"};
            Process p = r.exec(s);
            System.out.println("p="+p);
            p.getOutputStream().close();
//            p.waitFor(); si cette ligne est omise, tout fonctionne bien;
// mais c'est pas propre!
		}catch(Exception e) {
			System.out.println("erreur d'execution " ); 
			e.printStackTrace();
        }		
 
		System.out.println("fin");
 
	}
 
	public static void main(String[] args) {
		new TestCmd();
	}
 
}

16/03/2007, 15h59
adiGuba

Salut,

Lorsque tu lances un Process, il faut traité tous ses flux (input, output et error) dans des threads séparés...

En effet : ici le programme que tu lances (cmd /K) reste actif et attend des données sur son flux d'entrées, mais comme tu n'écrit rien dedans il reste bloqué en attente de lecture...

Et si toi tu restes en attente de la fin du process, tu te retrouves en "deadlock" car les deux programmes s'attendent mutuellement...

:arrow: lis tous les flux d'E/S (ou fermes les dans le pire des cas).

a++

PS : Pour info, je suis en train de développer une classe utilitaire simplifiant l'exécution de Process... je pourrais poster ici lorsqu'elle sera utilisable...
16/03/2007, 17h18
kenny49

Ouais je veux bien recuperer ton utilitaire.

N'empeche que lorsque je le mets inactif cmd /c (si c'est bien ce que ca veut dire) , ca ne change rien au problème.
16/03/2007, 17h19
adiGuba

Citation:

Envoyé par kenny49

Ouais je veux bien recuperer ton utilitaire.

Je rajouterais un lien ici lorsque ce sera publiable ;)

Citation:

Envoyé par kenny49

N'empeche que lorsque je le mets inactif cmd /c (si c'est bien ce que ca veut dire) , ca ne change rien au problème.

Tu traites bien tous les flux dans des threads séparés ?
On pourrait voir ton code ?

a++

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
28
package test;
 
 
public class TestCmd {
 
 
	public TestCmd() {
		System.out.println("begin");
		try {
			Runtime r = Runtime.getRuntime();
			String [] s={"cmd","/k","dir"};
            Process p = r.exec(s);
            p.getOutputStream().close();
            p.waitFor();
		}catch(Exception e) {
			System.out.println("erreur d'execution " ); 
			e.printStackTrace();
        }		
 
		System.out.println("fin");
 
	}
 
	public static void main(String[] args) {
		new TestCmd();
	}
 
}

Voila..

19/03/2007, 09h35
adiGuba

Il faut fermer ou lire tous les flux...

Ici tu te contentes de fermer le flux d'entrée du programme, mais tu ne lis pas les flux de sorties :arrow: il y a de forte chances que le buffer de communication entre les deux process soit plein, ce qui bloque le process.

Et comme ton application est elle aussi bloqué en attente, tu es dans un cas typique de deadlock !

:arrow: Il faut fermer ou lire tous les flux...

a++
19/03/2007, 16h49
adiGuba

Citation:

Envoyé par adiGuba

PS : Pour info, je suis en train de développer une classe utilitaire simplifiant l'exécution de Process... je pourrais poster ici lorsqu'elle sera utilisable...

Je l'ai posté sur mon blog avec un billet explicatif : Runtime.exec() n'est pas des plus simple...

a++