Pipe UNIX et commande exec

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:

---

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?

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

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:

---

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:

---

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.

@+

Salut,

Citation:

---

Envoyé par Mandraxx

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

Citation:

---

Envoyé par adiGuba

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 :pastaper:

Citation:

---

Envoyé par adiGuba

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

---

Tout à fait d'accord :king:

Salut,

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

Code:

---

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