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é