Discussion :

[dkbaier]

Bonjour,

Je rencontre le même problème que sur ce topic (http://www.developpez.net/forums/d41...ipe-unix-exec/). Je veux exécuter une commande dans mon programme Java (qui contient un pipe)

J'ai essayé cette solution

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
Process ls = Runtime.getRuntime().exec("ls -l");
Process wc = Runtime.getRuntime().exec("wc -l");
 
InputStream lsOutput = ls.getInputStream();
OutputStream wcInput = wc.getOutputStream();
 
// On 'copie' la sortie de ls dans l'entrée de wc :
int c;
while ( (c = lsOutput.read()) != -1 ) {
      wcInput.write(c);
}
 
lsOutput.close();
wcInput.close();

Mais ça ne marche pas car
le InputStream n'a pas de write et le OutputStream n'a pas de read

Comment faire?

[dinobogan]

Dans ton code, le InputStream utilise un read et le OutputStream utilise un write. Ce code est correct.

[Mandraxx]

Bonjour,

Le principe de l'algo est tout à fait correct : m'est avis que c'est un problème de protocole....

En effet, vu ton code, il y a de fortes chances que ton protocole d'échange soit binaire (ou du moins caractère par caractère, ce qui revient au même vu ce que tu cherches à faire...).

Je te propose d'essayer de décorer ta lecture avec un BufferedReader et ton écriture avec un PrintStream :

Code :

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

1
2
BufferedReader lsOutput = new BufferedReader(new InputStreamReader(ls.getInputStream()));
PrintStream wcInput = new PrintStream(wc.getOutputStream());

Du coup, ton algo va pouvoir implanter un vrai protocole texte :

Code :

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

1
2
3
4
5
6
7
8
String line = lsOutput.readLine();
while (line != null) {
    wcInput.println(line);
    line = lsOutput.readLine();
}
lsOutput.close();
wsInput.flush();
wsInput.close();

Ca peut éviter les problèmes de broken pipe ou de dead locks sur attente de caractères.

@+

[adiGuba]

Salut,

Le principe de l'algo est tout à fait correct : m'est avis que c'est un problème de protocole....

En effet, vu ton code, il y a de fortes chances que ton protocole d'échange soit binaire (ou du moins caractère par caractère, ce qui revient au même vu ce que tu cherches à faire...).

Non il vaut mieux éviter d'utiliser des Reader ici. Tu veux transmettre les données d'un process à l'autre autant recopier le flux directement. Sinon tu vas effectuer un décodage/encodage totalement inutile qui risque en plus de poser des problèmes potentiels d'encodage...


La copie des flux est tout à fait correct... Même s'il aurait été préférable d'utiliser un buffer.

Toutefois il manque quand même la fermeture des flux inutilisés, et l'utilisation des try/finally pour la libération des ressources...


Par contre le code ne sert à rien car la sortie du "wc -l" n'est pas lu...



a++

[Mandraxx]

Non il vaut mieux éviter d'utiliser des Reader ici. Tu veux transmettre les données d'un process à l'autre autant recopier le flux directement. Sinon tu vas effectuer un décodage/encodage totalement inutile qui risque en plus de poser des problèmes potentiels d'encodage...

Oui, autant pour moi, j'avais pas pensé à ça avec mon raisonnement purement système

Même s'il aurait été préférable d'utiliser un buffer.

Tout à fait d'accord

[DarkSeiryu]

Salut,

Juste une 'tite question : dans le sujet que tu as donné (pour un problème identique), il y a cette solution :

Code :

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

Process process = Runtime.getRuntime().exec("/bin/sh", "-c", "ls -l | wc -l");

Elle ne te convient pas ? Ca me semble quand même plus simple que d'utiliser des buffer...

DarkSeiryu