Discussion :

[dams78]

bonjour, je souhaite récupérer le retour d'une fonction unix, voila le code que j'ai écris

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
import java.io.*;
 
public class Essai{
        public static void main(String[] args) throws Exception {
                String commande = "wc -l cheminFichier1 | awk '{print $1}'";
                Process processus = Runtime.getRuntime().exec(commande);
                BufferedReader d = new BufferedReader(new InputStreamReader(processus.getInputStream()));
                System.out.println(d.readLine());
        }
}

seulement bien que normalement ma commande me renvoi le nombre de ligne du fichier c'est à dire 10,
java lui renvoie " 10 cheminFichier1 ", alors effectivement je pourai extraire le 10 qui m'interesse, mais dans ce cas la le pipe de ma commande ne sert plus à rien.

merci de votre aide

[adiGuba]

Salut,


Deux remarques :

• Il faut lire tous les flux du process (stdout et stderr) depuis des threads séparés. Si tu l'aurais fais tu aurais surement vu un joli message d'erreur...
• Runtime.exec() lance un programme et non pas une ligne de commande. Du coup ici | awk '{print $1}' sont trois paramètres passé à la commande wc, comme si tu avais fais en ligne de commande :
  

  Code :

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

  wc -l cheminFichier1 "|" awk '{print $1}'

Si tu veux lancer une ligne de commande il faut lancer le shell (/bin/sh)

Plus d'info :

• Exécuter une application externe en Java
• Runtime.exec() n'est pas des plus simple...

Bonne lecture...

a++

[dams78]

merci pour les liens, ils sont vraiment bien
du coup voila ce que j'ai fais =>

Code :

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

1
2
3
4
5
6
 
Runtime runtime = Runtime.getRuntime();
Process process = runtime.exec(new String[] { "/bin/sh", "-c", "wc -l Fichier1 | awk '{print $1}'" } );
BufferedReader reader = new BufferedReader(new InputStreamReader(process.getInputStream()));
String line =  reader.readLine();
System.out.println("nombre de lignes : " + line);

par contre j'ai pas utilisé de thread, je sais pas si c'est "pas bien ou pas"

[adiGuba]

par contre j'ai pas utilisé de thread, je sais pas si c'est "pas bien ou pas"

Si tu n'utilises pas de thread il faut fermé le flux d'erreur (mais du coup il te sera impossible de détecter un éventuel problème).

Au passage je te conseille également de fermer le flux d'entrée si tu ne l'utilise pas, et de lire la totalité du flux de sortir (ou de le fermer rapidement) :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
Runtime runtime = Runtime.getRuntime();
Process process = runtime.exec(new String[] { "/bin/sh", "-c", "wc -l Fichier1 | awk '{print $1}'" } );
// On ferme le flux d'entrée
process.getOutputStream().close();
// Et le flux d'erreur :
process.getErrorStream().close();
BufferedReader reader = new BufferedReader(new InputStreamReader(process.getInputStream()));
try {
    String line =  reader.readLine();
    System.out.println("nombre de lignes : " + line);
} finally {
    reader.close();
}

• Pourquoi fermer le flux d'entrée ? Car si la commande lit dans son stdin cela la bloquera en attente de lecture. En le fermant cela lui génèrera un EOF...
• Pourquoi fermer le flux d'erreur ? Si tu ne le lis pas et que le programme écrit dans le flux d'erreur cela peut aussi le bloquer. En effet il écrit en réalité dans un petit buffer et si ce dernier n'est pas vidé par ton application la commande appelée sera bloquée en écriture... Tant qu'à ignorer les erreurs autant que cela ne pose pas d'autres problèmes...
• Pourquoi fermer le flux standard ? Pour la même raison que ci-dessus si la commande écrit plus de ligne que prévu.

Toutes ces remarques ne s'appliquent peut-être pas au cas présent, mais permettent plus de sécurité lors de l'appel d'un programme externe...


a++

[Grimly_old]

Et la commande suivante ne convient-elle pas ?

Code :

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

java Essai `"wc -l cheminFichier1 | awk '{print $1}'"`

Si c'est pour l'inclure en plein milieu d'un de tes codes java, tu perds la portabilité de ton code entier. Ce serait dommage