Optimisation de code

Version imprimable

Bonjour je cherche comme mon titre l'indique à optimiser ma fonction

Elle a pour but dans un repertoire donné de recupérer les noms de fichiers PDF et uniquement PDF

Voici mon code :

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
 
    public static String[] getListFichierNom()
    {
        File repertoire = new File(Informations.rootPath); File []listeFilePath = repertoire.listFiles();
        int i, j, num;
 
        //recuperation du nombre de fichiers valides
        for (i = 0, num = 0; i < listeFilePath.length ; i++)
        {
            int longueur = listeFilePath[i].getName().length();
            int index = listeFilePath[i].getName().lastIndexOf(".pdf");
            if((longueur-index == 4 && longueur > 3))
            {
                num++;
            }
        }
        //Enregistrement dans l'objet liste des fichiers valides
        String[] liste = new String[num];
        for (i = 0, j = 0 ; i < listeFilePath.length||j <num ; i++)
        {
            int longueur = listeFilePath[i].getName().length();
            int index = listeFilePath[i].getName().lastIndexOf(".pdf");
            if((longueur-index == 4 && longueur > 3))
            {
                liste[j] = listeFilePath[i].getName();
                j++;
            }
        }
        return liste;
    }

Je pense qu'il doit y avoir bien plus simple que ce que fais en sachant que ce code fonctionne

Merci :D

Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 public static String[] getListFichierNom() { File[] listeFilePath = new File(Informations.rootPath).listFiles(); List<String> list = new ArrayList<String>(); String nameFile; for (File file:listeFilePath) { nameFile = file.getName(); if(nameFile.contains(".pdf")) { list.add(nameFile); } } return list.toArray(new String[list.size()]); }
Ce que tu fais semble correct , je me dis qu'il est plus simple quand même d'utiliser la fonction contains(".pdf").

Et si tu as pas absolument besoin d'un tableau de String , tu peux renvoyer une ArrayList.

Mais bon ici c'est pas vraiment de "l'optimisation" , c'est juste pour rendre le code plus "lisible". De plus de l'optimisation sur ce genre de portion de code sert strictement à rien.

Allez bon courrage !

26/10/2010, 20h07
snay13

Merci de ta reponse

Je n'ai pas utilisé contains pour eviter d'ajouter un fichier nommé par exemple : fichier.pdf.doc

Salut,

Pourquoi ne pas utiliser la classe FileNameFilter et la méthode associée de la classe File, ainsi que endsWith ? ce qui donne quelque chose du genre:
Code:

1 2 3 4 5 6 7 8 9 10 public static String[] getListFichierNom() { File repertoire = new File(Informations.rootPath); return repertoire.list(new FilenameFilter() { @Override public boolean accept(File rep, String name) { return name.endsWith(".pdf"); } }); }
C'est plus concis, plus clair, plus adapté je pense

A plus

27/10/2010, 09h16
adiGuba

Salut,

@George7 : +1

Toutefois il serait préférable de faire une comparaison en ignorant la case :

Code:

name.toLowerCase().endsWith(".pdf");

a++

Salut,

On peut aussi utiliser JFileChooser (sans faire un choix!!!):

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
JFileChooser chooser = new JFileChooser();
         chooser.setCurrentDirectory(new File(".")); // dans le répertoire courant
 
         chooser.setFileFilter
            (
               new javax.swing.filechooser.FileFilter()
               {
                  public boolean accept(File f)
                  {
                     if(f.getName().toLowerCase().endsWith(".pdf")) // si true
                        System.out.println(f.getName());  
                     return f.getName().toLowerCase().endsWith(".pdf");
                  }
 
                  public String getDescription()
                  {
                     return "";
                  }
               });

Cordialement,

Dan

Merci à tous pour vos reponse

Je ne connaissais pas cette classe
donc j'ai decidé d'integrer la solution de george7 en ignorant la casse
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 public static String[] getListFichierNom() { File repertoire = new File(Informations.rootPath); return repertoire.list(new FilenameFilter() { @Override public boolean accept(File rep, String name) { return name.toLowerCase().endsWith(".pdf"); } }); }
Par contre si maintenant je veux faire un tri par date (du plus récent au plus ancien)
Comment puisje proceder sur ce code?

Je pense qu'il faut tester le .lastModified(); de chaque fichier avant et de les comparer 1 à 1

Si vous avez des idées je suis preneur. Je me lance de mon coté

27/10/2010, 14h45
Invité

Dans ce cas, tu peux utiliser l'interface FileFilter qui te permet d'utiliser un objet de type File pour filtrer les fichiers acceptés. La classe File a une méthode listFiles acceptant un FileFilter en paramètre.

@adiGuba : c'est pas faux
@danimo : si on travaille sans interface graphique, je trouve la solution d'utiliser une classe swing suboptimale, surtout que la classe File prévoit des méthodes à cet effet
27/10/2010, 16h09
tchize_

Citation:

Envoyé par snay13

Par contre si maintenant je veux faire un tri par date (du plus récent au plus ancien)
Comment puisje proceder sur ce code?

Tu récupère la liste de fichiers de la même manière. Puis tu utilise Arrays.sort(listeFichier,comparateur)

avec comparateur une classe de ton cru implémentant Comparator<File>

Bonjour
je reviens vers vous car j'ai encore besoin d'un petit coup de main ;)

j'essaye de suivre les conseils de Tchize mais j'ai une erreur dans mon programme

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
    public static String[] getListFichierNom()
    {
	File repertoire = new File(Informations.rootPath);
        File[] test = repertoire.listFiles();

	String[] liste = repertoire.list(new FilenameFilter()
        {
            @Override
            public boolean accept(File rep, String name)
            {
		return name.toLowerCase().endsWith(".pdf");
            }
	});
        Arrays.sort(liste, TriFichierDate);      
        return liste;
    }

Code:

1
2
3
4
5
6
7
8
9
10
11
12
 
public class TriFichierDate implements Comparator<File>
{
    public int compare(File fichier1, File fichier2) 
    {
        long comparateur = fichier1.lastModified() - fichier2.lastModified();
        if(comparateur > 0)
            return -1;
        else
            return 1;
    }
}

l'erreur vient de la ligne surlignée dans la 1ere partie

27/10/2010, 21h24
Invité
Il faut passer une instance de TriFichierDate à la méthode Arrays.sort
donc du genre
Code:

1 2 Arrays.sort(liste, new TriFichierDate());
si tu n'as pas besoin de le réutiliser. Si tu réutilises souvent ta méthode, il pourrait peut-être être bien d'avoir un objet TriFichierDate instancié que tu réutilises à chaque fois.

De plus, liste est un tableau de String, or ton comparateur est pour les objets de type File.
Pour avoir un tableau de File, tu peux utiliser la méthode listFiles de la classe File avec un FileFilter au lieu d'un FileNameFilter (cf mon post du dessus pour la doc).

Effectivement j'ai reussi à corriger mon code pour le faire fonctionner juste avant que tu me reponde

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 
    public static String[] getListFichierNom()
    {
	File repertoire = new File(Informations.rootPath);
	String[] liste = repertoire.list(new FilenameFilter()
        {
            @Override
            public boolean accept(File rep, String name)
            {
		return name.toLowerCase().endsWith(".pdf");
            }
	});
 
        Arrays.sort(liste, new TriFichierDate());
        return liste;
    }

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
 
public class TriFichierDate implements Comparator<String>
{
    public int compare(String fichier1, String fichier2)
    {
        long comparateur = new File(Informations.rootPath + fichier1).lastModified()
                - new File(Informations.rootPath + fichier2).lastModified();
        if(comparateur > 0)
            return -1;
        else
            return 1;
    }
}

27/10/2010, 21h49
snay13

Si il reste des optimisations je suis à l'ecoute

Sinon merci à tous

:ccool:

:merci:

Citation:

Envoyé par snay13

Si il reste des optimisations je suis à l'ecoute

Sinon merci à tous

:ccool:

:merci:

Tu peux surment éviter ton if...else... en écrivant :
Code:

1 2 3 4 5 6 7 8 9 public class TriFichierDate implements Comparator<String> { public int compare(String fichier1, String fichier2) { return new File(Informations.rootPath + fichier2).lastModified() - new File(Informations.rootPath + fichier1).lastModified(); } }
Ça peut paraitre plus ou moins clair suivant les personnes.
Pour être sûr : tu as fichier1 < fichier2 si fichier1 a été modifié dernièrement après fichier2, c'est bien ça ?

28/10/2010, 05h58
snay13

euh non! Je crois que c'est l'inverse

plus le fichier est recent, plus le timestamp est elevé

mais le code que tu as posté convient sans le if et le else

:P