[Système][Runtime] pb exec une commande ls ds runtime

Version imprimable

bonjour ,
je suis sr linux , j'exécute une commande ls simple ds un runtime et tout marche bien (j'affiche bien le résultat)
mais lorsque j'exécute ls D* rien ne s'affiche !

je ne comprends pas pkoi

voilà le bout de code que j'utilise :
Code:

1 2 3 4 5 6 7 8 9 cmd = "ls D* ;"; System.out.println("commande est : "+cmd); Process process = Runtime.getRuntime().exec(cmd); BufferedReader ds = new BufferedReader (new InputStreamReader(process.getInputStream())); while ((line = ds.readLine()) != null) { System.out.println("line result est : "+line); } ds.close();
merci

28/02/2006, 10h20
adiGuba
Salut,

Quelques remarques :
- Tu ne lis pas le flux d'erreur, et tu ne peux pas voir les eventuelles erreurs renvoyé par "ls". Si tu utilises Java 5.0 tu peux utiliser ProcessBuilder pour regrouper les flux stdout et stderr dans un seul... Sinon il faut que tu les lises en parallèle pour éviter les interblocages. Voir ce message pour plus de détail : http://www.developpez.net/forums/vie...570460#2570460
- Les caractères génériques *? sont interprétés de manière différentes selon les shells :
  [list:b7fd7a59c3]
- Sous Windows ils sont passé tel quel au programme, qui se chargera alors de les interpréter.
- Sous Unix/Linux, c'est le shell qui les interprètes et qui les remplace par les noms de fichier correspondant avant d'exécuter la commande. Donc les programmes qu monde Unix/Linux ne gère pas les caractères spéciaux puisque c'est déjà fait.
Or Runtime.exec() lance une application sans passer par le shell donc il passe les paramètres tels quel...
[*] Pour les mêmes raison (pas de shell), le caractère ";" est passé tels quel et ne correspond pas à une fin de commande mais à un paramètre...[/list:u:b7fd7a59c3]

En fait si tu lis le flux d'erreur tu devrais y trouver quelques chose du style :
Code:

1 2 ls: D*: Aucun fichier ou répertoire de ce type ls: ;: Aucun fichier ou répertoire de ce type
Si tu veux vraiment appeller la commande ls comme cela, il te faut passer par le shell :
Code:

1 2 3 cmd = "/bin/sh -c \"ls E* ;\""; System.out.println("commande est : "+cmd); Process process = Runtime.getRuntime().exec(cmd);
Maintenant si ton but est de lister les fichiers, il serait préférable de passer par les méthodes adéquates de la classe File (et au moins tu restes multiplateforme...)

a++
28/02/2006, 10h39
peppena

bonjour ,

le but n'est pas de lister ce que contient le repertoire ,
je dois récupérer le nbr de fichiers dont le nom contient des chaines particulières , donc ma commande réélle est sous cette forme

Code:

cmd = "ls /home/Dir"+chaine+"*_log_"+an+"* |wc -l ;";

et puis la commande n'est pas unique , elle est dynamique selon une chaine passée 'chaine'
vraiment je sèche :fou:
mais le code que tu m'as passé ne marche pas aussi

Citation:

cmd = "/bin/sh -c \"ls E* ;\"";
System.out.println("commande est : "+cmd);
Process process = Runtime.getRuntime().exec(cmd);

:roll:
28/02/2006, 10h47
adiGuba
Citation:
Envoyé par peppena

bonjour ,

le but n'est pas de lister ce que contient le repertoire ,
je dois récupérer le nbr de fichiers dont le nom contient des chaines particulières , donc ma commande réélle est sous cette forme

Code:

1 2 cmd = "ls /home/Dir"+chaine+"*_log_"+an+"* |wc -l ;";
Justement. Tu peux très bien utiliser la méthode File.list(FilenameFilter) pour cela...

Citation:

Envoyé par peppena

mais le code que tu m'as passé ne marche pas aussi

Citation:

cmd = "/bin/sh -c \"ls E* ;\"";
System.out.println("commande est : "+cmd);
Process process = Runtime.getRuntime().exec(cmd);

:roll:

C'est à dire ? Quel est l'erreur ? Tu lis bien le flux d'erreur en sortie de la commande ???

a++
28/02/2006, 10h51
peppena

la commande retourne null null pas d'erreur ni de résultat ,
j'ai l'impression que son exécution n'a pas eu lieu
28/02/2006, 10h56
adiGuba

Comment ca null ? Je ne vois pas ce qui peut te renvoyer null là dedans !

On pourrait voir le code exact ???

voilà tout le code :
ds l'affichage ca me renvoit null , il n'entre meme pas ds la boucle d'affichage de ds.readline ...

Code:

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
 
String an =  Integer.toString(year) ;
an = an.substring(2); 
String mois = Integer.toString(month) ; 
 int nbrfile = 0;
                 String line ;   
                 for(int i=1 ; i <= month ; i++ )
                 {
                if (i.length() < 2)
                i = "0"+i ;
                 String cmd = null ;
                  cmd = "/bin/sh -c \"/home/Dir/"+chaine+"*_log_"+an+"* |wc -l  ;\""; 
                  System.out.println("commande est : "+cmd);
                  Process process = Runtime.getRuntime().exec(cmd); 
                  BufferedReader ds  = new BufferedReader (new InputStreamReader(process.getInputStream()));
 
                  while ((line = ds.readLine()) != null) {
                       nbrfile = line ;
                       System.out.println("line est "+line);
                 }
                 ds.close(); 
 
 
                  System.out.println("mle nbr de file log pour l'an "+an+"  est  : "+nbrfile); 
                }

merci :oops:

28/02/2006, 11h37
adiGuba
Désolé mais ton code ne compile même pas chez moi :
Code:

1 2 if (i.length() < 2) i = "0"+i ;
i est un int !

Code:

nbrfile = line ;

nbrfile est un int et line une String !!

Ensuite tu ne lit toujours pas le flux d'erreur de ton Process, comme je te l'ai indiqué dans mon premier message...

Enfin il n'y a pas de "ls" dans ta commande...

a++

désolée c juste des fautes de frappes ,

voilà le bout de code du runtime :oops:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
 
 
String an =  Integer.toString(year) ;
an = an.substring(2);
String mois = Integer.toString(month) ;
String nbrfile = null;
String cmd = null ;
String line ;   
          for(int i=1 ; i <= month ; i++ )
            {
               cmd = "/bin/sh -c \"ls /home/Dir/"+chaine+"*_log_"+an+"* |wc -l  ;\"";
               System.out.println("commande est : "+cmd);
               Process process = Runtime.getRuntime().exec(cmd);
               BufferedReader ds  = new BufferedReader (new InputStreamReader(process.getInputStream()));
 
               while ((line = ds.readLine()) != null)
                {
                   nbrfile = line ;
                   System.out.println("line est "+line);
                 }
                 ds.close();
 
                  System.out.println("le nbr de file log pour l'an "+an+"  est  : "+nbrfile);
         }

mais je ne sais pas comment lire le flux d'erreur en sortie de la commande ??? :roll:

Pourquoi ne pas utiliser File.list(FilenameFilter) comme adIguba l'a proposé?

Sa solution est vraiment très simple et elle est multi-platforme. ;-)

Code:

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
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
 
/*
 * Created on 30.11.2004
 *
 * To change the template for this generated file go to
 * Window>Preferences>Java>Code Generation>Code and Comments
 */
 
import java.io.File;
import java.io.FileNotFoundException;
import java.io.IOException;
import java.io.FilenameFilter;
 
import java.util.regex.Matcher;
import java.util.regex.Pattern;
 
 
/**
 * @author marfab001
 * 
 * To change the template for this generated type comment go to
 * Window>Preferences>Java>Code Generation>Code and Comments
 */
public class Test {
 
    public static void main(String[] args) throws Exception {
        File file = new File("C:\\projects");
        String[] files;
 
        RegExFilenameFilter filter = new RegExFilenameFilter("*.txt");
        files = file.list(filter);
        System.out.println(" " + files.length);
 
    }
}
 
class RegExFilenameFilter implements FilenameFilter {
 
    private Pattern pattern;
 
    public RegExFilenameFilter(String regex) {
        pattern = Pattern.compile(buildRegEx(regex));
    }
 
    public boolean accept(File dir, String name) {
        return pattern.matcher(name).matches();
    }
 
    /* Algo: un peu simplet à améliorer */
    private String buildRegEx(String regex) {
        StringBuffer sb = new StringBuffer();
        for (int i = 0; i < regex.length(); ++i) {
           char c = regex.charAt(i);
           if (c == '*') {
              sb.append(".*");
           }
           else if (c == '.') {
              sb.append("\\.");
           }
           else {
              sb.append(c);
           }
        }
        sb.insert(0, "^");
        sb.append("$");
        return sb.toString();
    }
}

28/02/2006, 15h23
adiGuba

Citation:

Envoyé par peppena

mais je ne sais pas comment lire le flux d'erreur en sortie de la commande ??? :roll:

getErrorStream()... à lire en parallèle de getInputStream()... comme je l'indique dans le lien de mon premier message !!!

Citation:

Envoyé par adiGuba

Enfin il n'y a pas de "ls" dans ta commande...

Je me demande si tu lis mes messages en entier... :roll:

a++

Citation:

Envoyé par jowo

Sa solution est vraiment très simple et elle est multi-platforme. ;-)

+1 et une suggestion pour buildRegEx() :
Code:

1 2 3 4 5 6 7 8 9 10 private String buildRegEx(String regex) { return "^" + // On entoure le tout par \Q et \E pour éviter l'interprétation des caractères quote(regex) // On ferme/ouvre les \Q \E autour des * et ? --> .* .? .replaceAll("[*?]","\\\\E.$0\\\\Q") // On supprime les \Q\E inutile .replaceAll("\\\\Q\\\\E","") + "$"; }
Cela permet de ne prendre en compte que les ? * des expressions du shell dans la génération de l'expression régulière. Cela permet de ne pas avoir de conflit avec d'éventuelle caractères spéciaux que l'on aurait pas traité (du style un "$" dans le nom du fichier...)

Plus d'info sur Pattern.quote() dans la FAQ : Comment empêcher les expressions régulières d'interpréter une sous-chaînes ?

a++

[edit] correction du lien

28/02/2006, 15h46
peppena

ne te fache pas adiGuba :mouarf:
je t'assure que j'ai tout lu :wink:
et je vais prendre le tps pour tout relire et tester