Discussion :

[ploxien]

Hello,

J'ai fait un programme JAVA qui execute un commande linux et qui affiche le résultat...

La fonction:

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
public static void executeCommand(String command){
        Process p = null;
        try {
            p = Runtime.getRuntime().exec(command);
        } catch (IOException e) {
            e.printStackTrace();
        }
        java.io.BufferedReader out = new java.io.BufferedReader( new java.io.InputStreamReader( p.getInputStream() ) );
 
        String line = "";
        try {
            while((line=out.readLine())!=null){
                System.out.println(line);
            }
        } catch (IOException e) {
            e.printStackTrace();
        }
    }

Si je l'appèle comme ceci:

Code :

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

executeCommand("pwd");

Pas de souci! Ca m'affiche bien le chemin courant!

Si je l'appele comme ceci:

Code :

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

executeCommand("ls ./plugins/*.*");

, rien ne s'affiche (pas d'erreur générée non plus...) Alors que la même commande depuis le même chemin mais tapée dans une invite de commande m'affiche bien tous les fichiers du dossier!

Pourquoi??

C'est vraiment bizarre!

Merci d'avance

[iohack]

Fais le exec sur un tableau de String contenant les mots de ta commande.
La question a déjà été posée des dizaines de fois...
EDIT ton code est sale, utiliser la fonction rechercher pour en savoir d'avantage notamenent pourquoi il faut géré les entrées/sorties dans des threads séparés.

[Yannick_from_31]

Utilise plutot un Vector (ou autre objet permis) et fait :

Code :

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

1
2
3
4
5
 
Vector command = new Vector();
command.add("ls");
command.add("./plugins/*.*");
executeCommand(command);

Oups grillé

[iohack]

Il Ne Faut Plus Utiliser Vecteur Lorsque Cela Est Possible !

[Yannick_from_31]

Il Ne Faut Plus Utiliser Vecteur Lorsque Cela Est Possible !

Ah bon, pourquoi (hormis le fait que c'est plus lourd q'un simple tableau) ?

[adiGuba]

Ah bon, pourquoi (hormis le fait que c'est plus lourd q'un simple tableau) ?

Parce qu'il faut lui préférer ArrayList

a++

[ploxien]

Merci, avec Shell ,c'est parfait!

A+

[adiGuba]

Salut,

Si je l'appele comme ceci:

Code :

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

executeCommand("ls ./plugins/*.*");

, rien ne s'affiche (pas d'erreur générée non plus...) Alors que la même commande depuis le même chemin mais tapée dans une invite de commande m'affiche bien tous les fichiers du dossier!

Normal : les messages d'erreurs s'affichent dans le flux d'erreur que tu ne lis pas (getErrorStream()).

Rappel : il faut traiter tous les flux d'E/S dans des threads différents (ou au moins les fermer si on ne les utilises pas).

Et ici tu dois avoir une erreur qui précise que le fichier "./plugins/*.*" n'existe pas. En effet sous les système Unix c'est le shell qui "transforme" les caractères * en nom de fichier... Or Runtime n'est pas un shell...




Si tu es sous Java 5.0 tu peux utiliser cette librairie qui tente de gérer la plupart des problèmes rencontré avec Runtime : [url=http://blog.developpez.com/index.php?blog=51&title=runtime_exec_n_est_pas_des_plus_simple&more=1&c=1&tb=1&pb=1]Runtime.exec() n'est pas des plus simple...[/quote]/applic/
Ce qui reviendrait à faire dans ton cas :

Code :

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

Shell.system("ls ./plugins/*.*");

Les sorties standard et d'erreurs sont reporté vers la console (System.out et System.err), alors que l'entrée standard est ignoré...



Sinon jettes un coup d'oeil à ce tutoriel : Exécuter une application externe en Java
Ou fais des recherches sur le forums (ce genre de problème est assez fréquent).


a++


PS : Tes try/catch ne sont pas très cohérent : Si un exception survient lors du lancement de l'application, tu tentes quand même de lire la sortie du programme (qui n'a pas pu être lancé), ce qui provoquera un joli NPE
Tu devrais avoir un seul et unique bloc try/catch qui englobe tous le traitement...