Discussion :

[snay13]

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 :

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

[Elendhil]

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

[snay13]

Merci de ta reponse

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

[Invité]

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 :

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

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

[adiGuba]

Salut,


@George7 : +1

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

Code :

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

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

a++

[danimo]

Salut,

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

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

[snay13]

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 :

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

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é

[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

[tchize_]

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>

[snay13]

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 :

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

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

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

[Invité]

Il faut passer une instance de TriFichierDate à la méthode Arrays.sort
donc du genre

Code :

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

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).

[snay13]

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

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

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

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;
    }
}

[snay13]

Si il reste des optimisations je suis à l'ecoute

Sinon merci à tous

[Invité]

Si il reste des optimisations je suis à l'ecoute

Sinon merci à tous

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

Code :

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

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 ?

[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