Résutat d'une commande en Java

Version imprimable

19/07/2011, 17h12
alamako

Résutat d'une commande en Java

Bonjour,
Je développe une application en java dans laquelle j’exécute la commande suivate:

Code:

rsh site -l user sqlplus rep/rep@DataBase @paramgen.

avec Runtime.exec. Mais le problème est que le processus créer envoie la valeur 1 et je n'ai pas de retour de ma commande. Quand j’exécute en ligne de commande tout va bien. Merci pour toute suggestion.

Utilise la classe ProcessBuilder pour lancer ta commande avec quelque chose du genre :
Code:

1 2 3 ProcessBuilder processBuilder = new ProcessBuilder("rm", "-f", "*"); Process process = processBuilder.start(); OutputStream oStream = process.getOutputStream();

19/07/2011, 17h46
tchize_

On pourrait voir le code aussi?

Voici le 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
32
33
34
35
36
37
Runtime rt2 = Runtime.getRuntime();
try{
 
    	proc2 = rt2.exec("rsh "+site+" -l user sqlplus rep/rep@DataBase @paramgen");
 
    }catch(IOException ex){
      System.out.println("une erreure s'est produit ");
      ex.printStackTrace();
    }
DataInputStream in = null;
    try{  
      in = new DataInputStream(proc2.getInputStream());
      BufferedReader buf = new BufferedReader(new InputStreamReader(in));
      String st = buf.readLine();
 
      while(st != null){
        fic.write(st+"\n");
        System.out.println(st);
        st = buf.readLine();
      }
 
    }catch(IOException ex){
      ex.printStackTrace();
    }
 
try{
      in.close();	
      fic.close();
    }catch(IOException ex){
      ex.printStackTrace();
    }
try{
      str = proc2.exitValue();
      System.out.println("la cmd à ete bien execute avec la valeur "+str);
    }catch(IllegalThreadStateException ex){
      System.out.println("la cmd est mal termine");
    }

à l'exécution j'ai ceci "la cmd à ete bien execute avec la valeur 1"

Salut,

Une valeur de retour supérieur à 0 indique généralement une erreur...

Mais bon ton code a un grand nombre de défaut, voir d'erreur :

Il est préférable de séparer les paramètres de la commande, via un tableau. D'ailleurs il faudrait privilégier ProcessBuilder à la place de Runtime.exec() !
Tu ne traites pas le flux d'erreur (du coup tu dois passer à coté des messages d'erreurs), et tu ne traites pas le flux d'entrée.
:arrow: Il faut impérativement les traiter (en parallèle) ou fermer ces flux, sous peine d'interblocage entre les deux programmes !
La gestion des exceptions est exécrable : plein de bloc catch qui se contente d'afficher un message et de continuer le traitement !?!?
Du style tu n'arrives pas à lancer le process, mais tu tentes quand même de lire son résultat !?!?
Le bloc try/catch devrait contenir tout ton code. Une erreur arrêtant tout le traitement. C'est inutile de vouloir continuer après une erreur...
Il manque les try/finally pour les close()... (si j'avais eu 1 € à chaque fois que j'ai dit cela je serait riche :D !)
A quoi sert le DataInputStream ? Tout comme les Readers !?!? Si tu veux simplement copier le tout dans un fichier autant le faire directement via un buffer de byte. C'est plus simple et plus performant (pas de conversion inutile en char).

A noter qu'avec ProcessBuilder tu peux regrouper les flux standard et d'erreur, ce qui permet de n'en lire qu'un seul et de ne pas s'embêter à paralléliser tout cela ;)

Quelque chose comme cela serait déjà plus correcte :
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 32 33 34 35 36 37 38 39 40 41 42 43 44 // On utilise ProcessBuilder, et on passe chaque paramètre indépendammment : ProcessBuilder pb = new ProcessBuilder( "rsh", site, "-l", "user", "sqlplus", "rep/rep@DataBase", "@paramgen"); // On redirige le flux d'erreur dans le flux standard // (pour pas s'embêter à traiter les deux en parallèle) pb.redirectErrorStream(true); // On crée le process : Process process = pb.start(); try { // On ferme les flux inutilisés : process.getOutputStream().close(); // on n'a pas besoin d'écrire process.getErrorStream().close(); // il est redirigé // On lit le flux principal InputStream input = process.getInputStream(); try { // Ouverture du fichier FileOutputStream output = new FileOutputStream("file.txt"); try { // Copie par buffer : byte[] buf = new byte[8192]; int len; while ( (len=input.read(buf)) >= 0) { output.write(buf, 0, len); } output.flush(); } finally { // fermeture du flux output.close(); } } finally { // fermeture du flux input.close(); } int result = process.waitFor(); } finally { // garde-fou : on détruit le process en cas de problème /// il devrait déjà être terminée dans une exécution normale process.destroy(); }
a++

20/07/2011, 14h11
alamako

Merci pour tes remarques, je sais maintenant où se trouve mon erreur. En redirigeant le flux d'erreur sur la sortie standard.
21/07/2011, 09h38
alamako

Je viens encore vers vous; Est -il possible de lancer une série de commande différentes avec un même processus.
21/07/2011, 09h42
adiGuba

C'est à dire ? Quel intérêt ?

a++
21/07/2011, 10h56
alamako

cad je veux lancer une série de commande tel que: "ls -l" puis "mkdir rep"
puis "cd rep" sans passer par un script shell.
21/07/2011, 12h22
sinok

Bah en même temps, tout ça tu peux le faire directement en java sans passer par des commandes shell.
La classe File est là pour toi.

Puis appeler SQLplus en java alors que tu as JDBC à disposition...
21/07/2011, 12h41
adiGuba

Citation:

Envoyé par sinok

Bah en même temps, tout ça tu peux le faire directement en java sans passer par des commandes shell.
La classe File est là pour toi.

Puis appeler SQLplus en java alors que tu as JDBC à disposition...

Pas mieux !!!

Sinon il faut appeler le shell système, puis communiquer avec lui avec les flux d'entrée/sortie... mais bon courage pour faire quelque chose de propre !

a++
21/07/2011, 15h19
alamako

Vous voulez dire en faisant des write sur les output on peut lancer une autre commande dans le même process ????
21/07/2011, 15h30
adiGuba

Si tu lances le shell oui (c'est l'objectif même du shell que de lancer des commandes). Mais franchement ce doit être galère à gérer.

Que cherches-tu à faire précisément ? Car cela me semble être une bien mauvaise solution !

a++
22/07/2011, 09h44
alamako

Je dois transformer un programme shell qui exécute des tâches (séries des commandes) en séquentiel en des tâches parallèles, pour optimiser l'exécution et ainsi exploiter la capacité du serveur.
22/07/2011, 19h41
tchize_

et pourquoi utiliser java pour ça plutot que de paralléliser directement dans le shell??