Discussion :

[alamako]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

[Robin56]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[tchize_]

On pourrait voir le code aussi?

[alamako]

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
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"

[adiGuba]

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.
  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 !)
• 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 :

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
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++

[alamako]

Merci pour tes remarques, je sais maintenant où se trouve mon erreur. En redirigeant le flux d'erreur sur la sortie standard.